<div>On Wed, 24 Apr 2019 at 7:38 PM, Francois Cartegnie <<a href="mailto:fcvlcdev@free.fr">fcvlcdev@free.fr</a>> wrote:<br></div><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le 24/04/2019 à 13:19, Roland Bewick a écrit :<br>
> Because the change I made only effects the selection of subtitles I<br>
> think the change will not break any existing auto deselection<br>
> functionality.><br>
> For example, when I open a new video file, all the subtitles I<br>
> previously opened are still removed, leaving a blank state as expected.<br>
<br>
Then your tests are too simplistic, and you don't see the extent of the<br>
subtitle selection.<br>
<br>
If a demux forcefully (by scripting, ...) selects another subtitle, the<br>
core must disable the previous one.<br>
<br>
Switching the policy not only allows 2, but N simultaneous subtitles.<br>
<br>
Then now any non spu ES now overrides selection rules...<br>
<br>
     if( p_esprops && p_sys->i_mode == ES_OUT_MODE_AUTO && EsIsSelected(<br>
es ) )<br>
-        p_esprops->p_main_es = es;<br>
+    {<br>
+        if( p_esprops->p_main_es == NULL || es->fmt.i_cat != SPU_ES )<br>
+        {<br>
+            p_esprops->p_main_es = es;<br>
+        }<br>
<br>
>> And as François hinted, hard-coding an alignment will break existing<br>
>> settings.<br>
> He said it wouldn't as long as there wasn't an existing bug. The codecs<br>
> were designed to load settings from user config. The codecs themselves<br>
> however OR'd in the bottom position in the case the user only set<br>
> horizontal alignment. I simply removed that hard coding so that vertical<br>
> alignment can also be set.<br>
<br>
The auto is bogus from the start.<br>
<br>
I don't want my subtitles centered if no user value exists.<br>
<br>
-- <br>
Francois Cartegnie<br>
VideoLAN - VLC Developer<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Thanks Francois.</div><div dir="auto"><br></div><div dir="auto">I’ll revert the policy and ensure the Dual Subtitles support makes as little impact to the system as possible. </div><div dir="auto"><br></div><div dir="auto">Roland</div><div dir="auto"><br></div>