[MPlayer-cvslog] CVS: main mplayer.c,1.882,1.883

Rich Felker dalias at aerifal.cx
Mon Nov 7 05:19:09 CET 2005


On Sun, Nov 06, 2005 at 03:44:52PM +0100, Tobias Diedrich CVS wrote:
> CVS change done by Tobias Diedrich CVS
> 
> Update of /cvsroot/mplayer/main
> In directory mail:/var2/tmp/cvs-serv19930
> 
> Modified Files:
> 	mplayer.c 
> Log Message:
> Number of frames to show the OSD shouldn't be hardcoded, derive from fps instead
> 
> Index: mplayer.c
> ===================================================================
> RCS file: /cvsroot/mplayer/main/mplayer.c,v
> retrieving revision 1.882
> retrieving revision 1.883
> diff -u -r1.882 -r1.883
> --- mplayer.c	5 Nov 2005 17:25:34 -0000	1.882
> +++ mplayer.c	6 Nov 2005 14:44:49 -0000	1.883
> @@ -2859,7 +2859,7 @@
>      case MP_CMD_SEEK : {
>        float v;
>        int abs;
> -      osd_show_percentage = 25;
> +      osd_show_percentage = sh_video->fps;

Do you have any idea what will happen when fps is 1000000? This is
actually possible/likely since it's not a framerate but rather
1/timebase. Very bad! The osd needs to be fixed to work in terms of
time rather than number of frames.

Rich




More information about the MPlayer-cvslog mailing list