[FFmpeg-devel] [PATCH] simplify ipmovie.c pts calculation

[Michael Niedermayer]

On Fri, Feb 27, 2009 at 02:27:06PM +0100, Reimar D?ffinger wrote:
> On Fri, Feb 27, 2009 at 02:18:12PM +0100, Reimar D?ffinger wrote:
> > On Fri, Feb 27, 2009 at 01:45:39PM +0100, Michael Niedermayer wrote:
> > > On Fri, Feb 27, 2009 at 08:59:14AM +0100, Reimar D?ffinger wrote:
> > > > On Fri, Feb 27, 2009 at 12:28:03AM +0100, Michael Niedermayer wrote:
> > 
> > [concerning code to guess r_frame_rate in libavformat/utils.c]
> > 
> > > > > also if you ignore the first 1-2 durations then this might fix the fps of
> > > > > h264_acc_in_flv.flv i think
> > > > 
> > > > I set the code to ignore the first 4, for that file it gives a frame
> > > > rate of 22.2222... which seems not really right either, though I don't
> > > > actually know what the right value is.
> > > 
> > > > It does not make a difference though, since the get_std_framerate-based
> > > > code after it overwrites it...
> > > 
> > > fix that prevents it from increasing the frame rate is welcome ...
> > 
> > Well, I thought the code was also there to prefer standard rates, not
> > just reduce the frame rate.
> > I have not extensively tested, but does attached patch fit the intended
> > purpose of the code?
> 
> Sorry, here is a fixed version. The "if" looks a bit ugly though.

ok if tested a little

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

Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090227/9f236b86/attachment.pgp>