[FFmpeg-devel] ffmpeg.c: Weird AVI special-case

[Michael Niedermayer]

On Sun, Feb 05, 2012 at 03:40:41PM +0100, Reimar Döffinger wrote:
> On Sun, Feb 05, 2012 at 03:33:36PM +0100, Michael Niedermayer wrote:
> > On Sun, Feb 05, 2012 at 03:23:07PM +0100, Reimar Döffinger wrote:
> > > On Sun, Feb 05, 2012 at 03:07:52PM +0100, Michael Niedermayer wrote:
> > > > On Sat, Feb 04, 2012 at 07:05:50PM +0100, Reimar Döffinger wrote:
> > > > > Hello,
> > > > > this is related to trac #978.
> > > > > Why does transcode_init double the codec time base for the destination
> > > > > file for AVI and only for AVI?
> > > > > And if you know the answer, could you please add a comment in the code?
> > > > > The commit message doesn't really help a bit either.
> > > > 
> > > > added some comment yestarday
> > > 
> > > Yes, I saw it. I am not sure whether the fact that for most codecs
> > > this leads to a time base of 1/2*fps instead of just 1/fps is really
> > > intended and sensible though...
> > 
> > the alternative is to fail with non monotone timestamps for some files
> 
> Well, at least when transcoding it could just drop those frames.

the code is under "if(stream_copy)"


> Maybe that's just me, but I also didn't find a way to override this
> choice, e.g. by specifying -r at the right location.
> I haven't tried, but I guess even the setpts filter will be of no
> help here.

i agree, we need a way to override it if its not possible ATM

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120205/220bacbc/attachment.asc>