[MPlayer-dev-eng] [PATCH] further dvr-ms playback improvements

[John Donaghy]

> (especially the fact that in general the framerate must be set by the
> demuxer,
> but demux_asf hardcodes it to 1000 fps);

Why have such a rule if not all containers provide the framerate?

> on the other side delegating
> this assignment to the decoder can help to fix one of mplayer's limitation:
> when playing file formats that don't have a nominal framerate  (such as
> asf, mpeg*)

It seems to me that sometimes you just have to rely on the decoder -
that is what the lavf demuxer does.

On the subject of lavf, why not just always build mplayer with
libavcodec and force it to use the lavf demuxer when playing dvr-ms
files? It already works when you use '-demuxer 35' - including setting
the framesize properly.


> 2) use the attached patch: it's much more appropriate and much less
> intrusive,
Will this patch force the selection of the vd_ffmpeg decoder? -
because currently libmpeg2 gets selected as the mpeg video decoder.

>
> BTW, that file seems to be somewhat broken: the timestamps delta seem to be
> completely random: .8, .108, .26, etc. seconds; what the hell of a program
> can have made such a mess?

Which file? fps_sample1 was created using something called dCut on a
file recorded using Windows Media Center. dCut could be the problem.