<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, 17 May 2019 at 6:42 PM, Thomas Guillem <<a href="mailto:thomas@gllm.fr">thomas@gllm.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
On Fri, May 17, 2019, at 13:01, Roland Bewick wrote:<br>
> <br>
> On 17/05/2019 3:16 PM, Thomas Guillem wrote:<br>
> ><br>
> > On Thu, May 16, 2019, at 15:40, Roland Bewick wrote:<br>
> >> Hi,<br>
> >><br>
> >> I've chosen the following implementation for vlc_player_SelectTrackList:<br>
> >> 1. Deselect all selected tracks not matching the provided list.<br>
> >> 2. Sequentially select the tracks that aren't already selected.<br>
> >><br>
> >> When a track is selected in the QT GUI, I check the category of the track.<br>
> >> - If Not SPU_ES, use vlc_player_SelectTrack as usual.<br>
> >> - If SPU_ES:<br>
> >>       1. find which tracks are already selected by looping through all<br>
> >> tracks of the same category<br>
> >>       2. If more than one track is already selected, do not proceed with<br>
> >> the selection<br>
> >>       3. Call vlc_player_SelectTrackList using the ids of the new track<br>
> >> and the already selected tracks<br>
> >><br>
> >> To ensure that multiple tracks get selected in the core (es_out.c) I've<br>
> >> added a new parameter to EsOutSelect.<br>
> > Fine wih me, that is what I had in mind.<br>
> ><br>
> > For es_out implementation, I prefer to send directly the list of ES via a new control. This allow to select multiples tracks atomically (only one lock held).<br>
> <br>
> Great, I can do that.<br>
> <br>
> I'll create a new list in vlc_player_SelectTrackList containing only the <br>
> list of ids of ES that aren't selected yet, so that vlc_es_id_Hold isn't <br>
> called unnecessarily on any already selected ES.<br>
<br>
I think it's completely ok to call Hold one extra time. The more important is that release is called accordingly.</blockquote><div dir="auto"><br></div><div dir="auto">Ok, great.</div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
> <br>
> Since multiple ES Ids are getting sent it also might be better to add an <br>
> EsOutSelectList method rather than overloading the existing EsOutSelect <br>
> (partial and automatic selection modes don't apply).<br>
<br>
I agree<br>
<br>
> <br>
> I think I should be able to find my way from here and submit a new patch <br>
> set.<br>
> <br>
> Thanks Thomas,<br>
<br>
No, Thanks to you for doing this work ;)<br>
<br>
We don't have that many external devolopers that want to touch this part of the code. </blockquote><div dir="auto"><br></div><div dir="auto">I think I’m beginning to understand why :D</div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
> <br>
> Roland<br>
> <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><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>