<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>Thank you all,</p>
<p>I'll add vlc_player_SelectTrackList as requested.</p>
<p>Also: I can use this method to handle secondary subtitle track
cycling using the hotkeys too, so I'll remove the other API
function I added and return the original
SelectNextTrack/SelectPrevTrack functionality to support only
single selection (I'll handle secondary subtitle track cycling
from the hotkeys instead).</p>
<p>Roland<br>
</p>
<div class="moz-cite-prefix"><br>
</div>
<blockquote type="cite"
cite="mid:e44ebb4d-881d-4483-b23e-a32bd86da002@www.fastmail.com">
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<title></title>
<style type="text/css">#qt p.qt-MsoNormal,#qt p.qt-MsoNoSpacing,#qt p.qt-MsoNormal,#qt p.qt-MsoNoSpacing,#qt p.qt-MsoNormal,#qt p.qt-MsoNoSpacing,#qt #qt-qt p.qt-qt-MsoNormal,#qt #qt-qt p.qt-qt-MsoNoSpacing{margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px;}
p.MsoNormal,p.MsoNoSpacing{margin:0}</style>
<div><br>
</div>
<div>On Wed, May 15, 2019, at 15:44, Rémi Denis-Courmont wrote:<br>
</div>
<blockquote type="cite" id="qt">
<div>Hi,<br>
</div>
<div><br>
</div>
<div>If SelectTrackList sets the list (as opposed to, add tracks
to the list), then there is no use/meaning for
UnselectTrackList. A single function is enough, no?<br>
</div>
</blockquote>
<div><br>
</div>
<div>Indeed, that is what I answered to Steve. <br>
</div>
<div><br>
</div>
<blockquote type="cite" id="qt">
<div><br>
</div>
<div class="qt-gmail_quote">
<div>Le 15 mai 2019 13:39:59 GMT+03:00, Thomas Guillem
<a class="moz-txt-link-rfc2396E" href="mailto:thomas@gllm.fr"><thomas@gllm.fr></a> a écrit :<br>
</div>
<blockquote
style="margin-top:0pt;margin-right:0pt;margin-bottom:0pt;margin-left:0.8ex;border-left-color:rgb(204,
204,
204);border-left-style:solid;border-left-width:1px;padding-left:1ex;"
class="qt-gmail_quote">
<div>I think I prefer the following API:<br>
</div>
<div><br>
</div>
<div>VLC_API void<br>
</div>
<div>vlc_player_SelectTrackList(vlc_player_t *player, enum
es_format_category_e cat, vlc_es_id_t *const
es_id_list[]);<br>
</div>
<div><br>
</div>
<div>VLC_API void<br>
</div>
<div>vlc_player_UnselectTrackList(vlc_player_t *player, enum
es_format_category_e cat, vlc_es_id_t *const
es_id_list[]);<br>
</div>
<div><br>
</div>
<div>
<div>
<div>es_id_list is a NULL terminated array of
vlc_es_id_t<br>
</div>
</div>
<div>
<div><br>
</div>
</div>
<div>
<div>
<div><br>
</div>
</div>
</div>
</div>
<div>Example of use case:<br>
</div>
<div>{<br>
</div>
<div> vlc_es_id_t *spu_list[] = { spu_id_1, spu_id_2, NULL
};<br>
</div>
<div> vlc_player_SelectTrackList(player, SPU_ES, spu_list);<br>
</div>
<div>}<br>
</div>
<div><br>
</div>
<div>vlc_player_SelectTrackList will replace every tracks of
one category by the one specified in the array.<br>
</div>
<div><br>
</div>
<div>On Wed, May 15, 2019, at 11:30, Roland Bewick wrote:<br>
</div>
<blockquote id="qt-qt" type="cite">
<p><br>
</p>
<div class="qt-qt-moz-cite-prefix">On 15/05/2019 3:59 PM,
Thomas Guillem wrote:<br>
</div>
<blockquote type="cite"
cite="mid:a11febac-7c1e-4e1d-883d-9f460fd9b56a@www.fastmail.com">
<div>The new way to select multiple tracks should be
more generic since it could possibly work for video
and audio (in the future).<br>
</div>
<div><br>
</div>
<div>The player API is new and was not yet published, we
can modify it easily.<br>
</div>
<div><br>
</div>
<div>We could just add a "bool multiple" to
vlc_player_SelectTrack(); and specify in the
documentation that multiple work only for Video and
for 2 subtitles.<br>
</div>
<div><br>
</div>
<div>Or we can find a new function, like
vlc_player_SelectTrackMultiple().<br>
</div>
</blockquote>
<p>I'd prefer adding an extra parameter to the existing
vlc_player_SelectTrack, but it is not up to me.<br>
</p>
<p>Could the VLC team make a decision on this?<br>
</p>
<p>Thanks,<br>
</p>
<p>Roland<br>
</p>
<div><br>
</div>
<div>_______________________________________________<br>
</div>
<div>vlc-devel mailing list<br>
</div>
<div>To unsubscribe or modify your subscription options:<br>
</div>
<div><a class="moz-txt-link-freetext" href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div>
</blockquote>
<div><br>
</div>
</blockquote>
</div>
<div><br>
</div>
<div>-- <br>
</div>
<div>Envoyé de mon appareil Android avec Courriel K-9 Mail.
Veuillez excuser ma brièveté. <br>
</div>
<div>_______________________________________________<br>
</div>
<div>vlc-devel mailing list<br>
</div>
<div>To unsubscribe or modify your subscription options:<br>
</div>
<div><a class="moz-txt-link-freetext" href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div>
</blockquote>
<div><br>
</div>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<pre class="moz-quote-pre" wrap="">_______________________________________________
vlc-devel mailing list
To unsubscribe or modify your subscription options:
<a class="moz-txt-link-freetext" href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a></pre>
</blockquote>
</body>
</html>