On Tue, Jan 13, 2015 at 08:12:10AM +0100, Clément Bœsch wrote:
> On Tue, Jan 13, 2015 at 11:26:06AM +0530, Anshul Maheshwari wrote:
> [...]
> > > Also added roll-up functionality.
> > >
> > > patch is attached.
> > > Thanks
> > > Anshul
> > >
> > Attaching patch
> 
> > From ca66e917f45bd47dd797930423d488fa9c781c4c Mon Sep 17 00:00:00 2001
> > From: Anshul Maheshwari <anshul.ffm...@gmail.com>
> > Date: Mon, 12 Jan 2015 06:21:18 -0800
> > Subject: [PATCH] Addig Rollup functionality
> > 
> > Signed-off-by: Anshul Maheshwari <anshul.ffm...@gmail.com>
> > ---
> >  libavcodec/ccaption_dec.c | 222 
> > +++++++++++++++++++++++++++++-----------------
> >  1 file changed, 140 insertions(+), 82 deletions(-)
> > 
> 
> Please split your changes.
> 
> One patch for the cosmetics
> One patch for the CHAR_DEBUG/av_dlog
> One patch for the addition of av_log
> One patch for the reflow of init_decoder()
> One patch for the roll-up functionality
> One patchset to rule them all.
> 
> Thank you.
> 

hint: git add -p is your friend

-- 
Clément B.

Attachment: pgpSjCSFasQZR.pgp
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to