[MPlayer-users] Problem when cycling internal subtitles

[Nico Sabbi]

Victor Farias wrote:
> Nico Sabbi wrote:
>
>> taken.
>> What is "b" supposed to do? disable subs?
>
> According to MPlayer's manual page:
>
> "b / j     -> Cycle through the available subtitles."
>
> Each time you press the "b" key, MPlayer cycles to the next available 
> subtitles, if there are not another available subtitles, then MPlayer 
> will turn off the subtitles (the OSD message is "Subtitles: off").
>
> The correct cycle should be:
>
> -> sub 01 -> sub 02 -> last sub -> sub off -> sub 01 -> sub 02 -> last 
> sub -> sub off...
>
> If you test the "test_01.mkv" file, pressing "b" (or "j") several 
> times, then you will see the following behavior:
>
> -> sub 01 -> sub 01 -> sub 01 -> sub 01 -> sub 01 -> sub 01...
>
> If you test the "test_02.mkv" file, pressing "b" (or "j") several 
> times, then you will see the following behavior:
>
> -> sub off -> sub 01 -> sub off -> sub 01 -> sub off -> sub 01...
>
> The behavior testing "test_02.mkv" is the correct behavior.
>
> The behavior testing "test_01.mkv" should be:
>
> -> sub 01 -> sub off -> sub 01 -> sub off -> sub 01 -> sub off...
>
> Reverting from mplayer.c,1.900 to mplayer.c,1899 solves the problem 
> (CVS diff 01).
>
> I just discovered that this incorrect behavior is also present in DVD 
> playback.
>
> If you only revert to mplayer.c,1.889; then another problem will be 
> triggered... MPlayer will start displaying "english" subtitles (second 
> available subtitles) even if you do not use the "slang" or "vobsubid" 
> command. Then pressing "b" until you turn off the subtitles (OSD 
> message: "Subtitles: off") MPlayer will display "chinese" subtitles 
> (first available subtitles). This problem is only triggered when 
> playing DVDs.
>
> So if you want to avoid both problems (for testing purposes), then you 
> must revert both commits (CVS diff 01 and CVS diff 02).
>
> CVS diff 01 (check for "Revision 1.900"):
> http://www1.mplayerhq.hu/cgi-bin/cvsweb.cgi/main/mplayer.c
>
> CVS diff 02 (check for "Revision 1.67"):
> http://www1.mplayerhq.hu/cgi-bin/cvsweb.cgi/main/libmpdemux/demux_mpg.c
>

the previous code was BROKEN, that's why I changed it.
Now I have to fix it properly

 
 
 --
 Email.it, the professional e-mail, gratis per te: http://www.email.it/f
 
 Sponsor:
 250 biglietti da visita premium GRATIS!
 Clicca qui: http://adv.email.it/cgi-bin/foclick.cgi?mid=4835&d=3-3