<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>