<!DOCTYPE html><html><head><title></title><style type="text/css">#qt p.qt-MsoNormal,#qt p.qt-MsoNoSpacing{margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px;}
p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body><div><br></div><div><br></div><div>On Sat, May 11, 2019, at 15:56, Roland Bewick wrote:<br></div><blockquote type="cite" id="qt"><p><br></p><div class="qt-moz-cite-prefix">On 11/05/2019 6:21 PM, Jean-Baptiste
Kempf wrote:<br></div><blockquote cite="mid:688e7b1b-1262-4dd2-bafa-5c291af52adf@www.fastmail.com" type="cite"><div><br></div><div><br></div><div>On Sat, 11 May 2019, at 13:15, Roland Bewick wrote:<br></div><blockquote id="qt-qt" type="cite"><div><div>On Sat, 11 May 2019 at 4:17 PM, Rémi Denis-Courmont <<a href="mailto:remi@remlab.net">remi@remlab.net</a>>
wrote:<br></div><blockquote class="qt-qt-gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-color:rgb(204, 204, 204);border-left-style:solid;border-left-width:1px;padding-left:1ex;"><div><div>Hi,<br></div><div><br></div><div>While it likely makes sense at the level of UI
widgets and user interactions, it seems a bit out of
place, standing alone, at the player API level.<br></div><div><br></div><div>In particular, I doubt that enabling multiple tracks
mode in Qt should interfere with the track selection of,
say, the HTTP remote control.<br></div><div><br></div><div>IMO, the API should just take the list of active
tracks, whether it be empty, a singleton, or larger.<br></div></div></blockquote><div dir="auto"><br></div><div dir="auto">Hi, Remi,<br></div><div dir="auto"><br></div><div dir="auto">I designed the changes so that none of the
current consumers of the API will be impacted.<br></div><div dir="auto"><br></div><div dir="auto">Enabling dual subtitles will not affect the
others (like the remote control or lua scripts), because
there is another requirement: to temporarily enable
multitrack selection using the new call I added. The UI and
hotkeys use this but none of the others do.<br></div><div dir="auto"><br></div><div dir="auto">I do agree however that your suggestion is
more fitting (but require a bit more logic from each
consumer that supports selecting multiple tracks)<br></div></div></blockquote><div><br></div><div>Make the API simple to maintain, for the core. UI will adapt.<br></div><div><br></div><div id="qt-sig60240713"><div class="qt-signature">--<br></div><div class="qt-signature">Jean-Baptiste Kempf - President<br></div><div class="qt-signature">+33 672 704 734<br></div><div class="qt-signature"> <br></div><div class="qt-signature"><br></div></div><div><br></div><div><br></div><pre class="qt-moz-quote-pre" wrap="">_______________________________________________
vlc-devel mailing list
To unsubscribe or modify your subscription options:
<a href="https://mailman.videolan.org/listinfo/vlc-devel" class="qt-moz-txt-link-freetext">https://mailman.videolan.org/listinfo/vlc-devel</a><br></pre></blockquote><p></p><div>Unfortunately it isn't just the SelectTrack function, but
SelectNextTrack and SelectPrevTrack too.<br></div><div> <br></div><div> I feel like making the consumer pass a list of already selected
tracks *plus* the new track to select is not right.<br></div><div> <br></div><div> I agree the api should be simple but it should also be simple to
use. Will this change make the core less complex?<br></div><p></p><p>What I've done isn't so good, yes. But I don't think Rémi's idea
is right either. We're basically searching for tracks and then
excluding the selected ones so that they don't get selected again.<br></p><p></p><div>Potential alternative option: <br></div><div> - Pass an extra parameter into the existing SelectTrack function -
selection type (Single / multiple).<br></div><div> - For SelectNextTrack/SelectPrevTrack functions I would need to
pass the "spu id" (1=primary, 2=secondary, 3=tertiary sub track)<br></div><p></p></blockquote><div><br></div><div>Next/Prev should be disabled if there are multpiple tracks selected. Indeed, this function is mostly used from a keyboard shortcut and there is no way to circle next/prev for a specified id from a keyboard.<br></div><div><br></div><blockquote type="cite" id="qt"><p></p><div> <br></div><div> Then I can remove the new API functions I added.<br></div><div> <br></div><div> Please correct me if I'm wrong - maybe I'm not seeing the full
picture, but I feel like this change is more complicated than its
worth.<br></div><p></p><p></p><div>Roland<br></div><div> <br></div><p></p><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div>https://mailman.videolan.org/listinfo/vlc-devel<br></div></blockquote><div><br></div></body></html>