[MPlayer-cvslog] r18922 - in trunk: cfg-mplayer.h libmpcodecs/dec_video.c libmpcodecs/dec_video.h libmpcodecs/vd.h libmpcodecs/vd_ffmpeg.c libmpcodecs/vf.c libmpcodecs/vf.h libmpcodecs/vf_tfields.c libmpcodecs/vf_vo.c libmpcodecs/vf_yadif.c libmpdemux/demux_mkv.c libmpdemux/demuxer.c libmpdemux/demuxer.h libmpdemux/stheader.h mplayer.c

[Ivo]

On Thursday 06 July 2006 08:58, uau wrote:
> Author: uau
> Date: Thu Jul  6 08:58:17 2006
> New Revision: 18922
>
> Modified:
[..]
>    trunk/mplayer.c
>
> Log:
> Add a new video pts tracking mode, enabled by option -correct-pts.
> This mode has the following differences:
> - Video timing is correct for streams with B frames, at least with some
>   demuxers.
> - Video filters can modify frame timestamps and insert new frames, and
>   removing frames is handled better than before.
> - Some things are known to break, it's not usable as the default yet.
>
> Things should work as before when the -correct-pts option is not used.
[..]
> Modified: trunk/mplayer.c
> =========================================================================
>===== --- trunk/mplayer.c	(original)
> +++ trunk/mplayer.c	Thu Jul  6 08:58:17 2006
[..]
> @@ -3803,6 +3840,32 @@
>  	blit_frame=decode_video(sh_video,start,in_size,drop_frame,
> MP_NOPTS_VALUE); break;
>      }
> +    else while (1) {
> +	if (!generate_video_frame(sh_video, d_video)) {
> +	    eof = 1;
> +	    break;
> +	}
> +	sh_video->pts = vf_vo_data.pts;
> +	if (sh_video->pts == MP_NOPTS_VALUE) {
> +	    mp_msg(MSGT_CPLAYER, MSGL_ERR, "pts after filters MISSING\n");
> +	    sh_video->pts == last_pts;
> +	}

This doesn't seem right. Did you mean sh_video->pts = last_pts?

--Ivo