[FFmpeg-devel] [FFmpeg-cvslog] Adding Closed caption Support

[Clément Bœsch]

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.ffmpeg at gmail.com>
> > Date: Mon, 12 Jan 2015 06:21:18 -0800
> > Subject: [PATCH] Addig Rollup functionality
> > 
> > Signed-off-by: Anshul Maheshwari <anshul.ffmpeg at 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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150113/f5fe1d2e/attachment.asc>