[Ffmpeg-devel] [PATCH] fix frame duration computation in find_stream_infos

[Michael Niedermayer]

Hi

On Fri, Mar 09, 2007 at 11:34:08PM +0100, Baptiste Coudurier wrote:
> Michael Niedermayer wrote:
> > Hi
> > 
> > On Fri, Mar 09, 2007 at 09:51:44PM +0100, Baptiste Coudurier wrote:
> >> Hi
> >>
> >> Michael Niedermayer wrote:
> >>> Hi
> >>>
> >>> On Fri, Mar 09, 2007 at 05:42:48PM +0100, Baptiste Coudurier wrote:
> >>>> Hi
> >>>>
> >>>> $subj.
> >>>>
> >>>> I guess there is a typo here. Is it correct ?
> >>>> last_dts should be updated when duration is present.
> >>> no its no typo, this improves accuracy IIRC
> >>>
> >> ok, I misunderstood the code then.
> >>
> >> wrong_frame_rate.mp4 in incoming:
> >> Stream #0.0(eng): Video: mpeg4, yuv420p, 352x288, 49.67 fps(r)
> >>
> >> should be 16.38 (quicktime),
> > 
> > just because QT says it, doesnt make it correct ...
> 
> well, just because ffmpeg says it, neither.

of course not, the real_frame_rate thing is just a guess and the
doxy says so IIRC


> 
> > 16.38 does not look like a likely framerate at all
> 
> 49.67 neither.

of course, 49.67 is nonsense

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

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070309/e5b6be5b/attachment.pgp>