[MPlayer-dev-eng] [PATCH] dvdnav - partial REVIEW
Nico Sabbi
nicola_sabbi at fastwebnet.it
Sun Jul 2 13:00:23 CEST 2006
Ötvös Attila wrote:
>
>
>Hi All!
>
>I modify:
>
>- fixed mute bug
>- fixed xvidix yv12 bug
>- implemented use ffmpeg codec
>- remove old dvdnav code in mplayer.c
>- covert all dvdnav functions and dvdnav_priv access to stream_control() in
>mplayer.c
>
>
sorry to say it, but this is nonsense: stream_control() functions should
implement only
what belongs to the stream layer, so everything that is related to user
input, menu and
subtitles had better be invoked by a direct function call.
The CTRLs that are related to time_length, eof, lockseek (is it really
needed?) are ok,
the others that are related to language selection are acceptable only if
called by the demuxer
(I mean that demuxer_control() should call stream_control());
all the rest can be implemented in dvdnav_stream.c but not as
stream_control() implementation.
>- new parameters: -dvdnav-still-repeat
>- updated current SVN version
>
>If mplayer works with "-dvdnav-still-repeat 0" then can use external libdvdnav
>lib.
>
>
>
please, synchronize your tree to mplayer svn.
I'd like to import your code -except libmpdvdnav, of course:) -, so what
we need now is:
- first of all getting rid of your stream_dvdnav.c: from now on patch
dvdnav_stream.c, please
- fix in this file the stream code that is still wrong (someone reported
that occasionaly some titleset
can't be played (while(1) syndrome)
- if you have some better seeking code post it
- add the functions that you need for the menu code
all of the above without cosmetics and in small and indipendent patches,
please :)
Thanks,
Nico
More information about the MPlayer-dev-eng
mailing list