[MPlayer-cvslog] r19217 - trunk/libmpdemux/demux_avs.c

[Uoti Urpala]

On Fri, 2006-07-28 at 16:44 +0200, reimar wrote:
> -        d_video->pts=AVS->frameno / sh_video->fps; // OSD
> +        dp->pts=AVS->frameno / sh_video->fps;

If fps has float precision then this should have the (double) cast
mentioned on mplayer-dev-eng. If the value of fps is slightly inaccurate
that at most makes the timestamps become consistently too small or big
the further you go in the file, but if the division is also done with
float accuracy that causes similar size errors in the duration of
individual frames. Of course using double for fps might be a better
solution.