[FFmpeg-devel] [PATCH] ffmpeg.c: copy chapters by default.

Anton Khirnov wyskas
Sat Mar 20 08:36:18 CET 2010


On Fri, Mar 19, 2010 at 09:38:18AM +0100, Michael Niedermayer wrote:
> On Thu, Mar 18, 2010 at 10:18:11PM +0100, Michael Niedermayer wrote:
> > On Thu, Mar 18, 2010 at 08:15:48PM +0100, Anton Khirnov wrote:
> > > On Thu, Mar 18, 2010 at 04:34:21PM +0100, Michael Niedermayer wrote:
> > > > On Thu, Mar 18, 2010 at 01:13:41PM +0100, Anton Khirnov wrote:
> > > > maybe these could be simplified with FFMIN/FFMAX
> > > indeed, done.
> > > 
> > > Anton Khirnov
> > 
> > >  ffmpeg.c |   51 +++++++++++++++++++++++++++++++++++++++++++++++++++
> > >  1 file changed, 51 insertions(+)
> > > baca1878fb209f8c401f6c6b047e78d75c3c33d4  0001-ffmpeg.c-copy-chapters-by-default.patch
> > > From c538b582f2478e0c21a577901f30e011a55f13a0 Mon Sep 17 00:00:00 2001
> > > From: Anton Khirnov <wyskas at gmail.com>
> > > Date: Wed, 24 Feb 2010 23:00:31 +0100
> > > Subject: [PATCH] ffmpeg.c: copy chapters by default.
> > > 
> > > Chapters are copied from the first input file that has them to all
> > > output files.
> > > ---
> > >  ffmpeg.c |   51 +++++++++++++++++++++++++++++++++++++++++++++++++++
> > >  1 files changed, 51 insertions(+), 0 deletions(-)
> > 
> > okif tested
> 
> also maybe you want to move this function into lavf
something like this?
btw isn't av_rescale_q supposed to handle overflows?
av_rescale_q(INT64_MAX, {1,1E6}, {1,1E9}) returns -1000.

>(new c file)
why a new file? utils.c seems like a perfect place for it.

Anton Khirnov
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-av_copy_chapters-function.patch
Type: text/x-diff
Size: 5923 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100320/8c8dcdbd/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100320/8c8dcdbd/attachment.pgp>



More information about the ffmpeg-devel mailing list