[FFmpeg-devel] ffmpeg.c: Weird AVI special-case
michaelni at gmx.at
Sun Feb 5 17:21:37 CET 2012
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
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel