<div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hello,</div><div><br></div><div>Thank you all for helping me to write properly.</div><div>This is my very first post here and i'm not very familiar with git send-email command.</div><div>I did not know how to separate the subject from the explanation, I will do better next time.<br></div><div>Please accept my apology for not following the rules.</div><div><br></div><div>Best regards,</div><div><br></div><div>Pascal Souchet</div><div>ps: English is not my native language, you had guessed it :-)</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le jeu. 2 juil. 2020 à 22:44, Romain Vimont <<a href="mailto:rom1v@videolabs.io">rom1v@videolabs.io</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Jul 02, 2020 at 09:12:32PM +0200, Thomas Guillem wrote:<br>
> Hello,<br>
> <br>
> Thanks for the patch.<br>
> <br>
> Nitpicking, the commit title could be improved, cf. <a href="https://chris.beams.io/posts/git-commit/" rel="noreferrer" target="_blank">https://chris.beams.io/posts/git-commit/</a><br>
> The title should be short, and the message should contain the explanation (if the title is not self-explanatory).<br>
> <br>
> I would have written:<br>
> <br>
> "lib: fix libvlc_media_player_get_track_from_id() always using the VIDEO count"<br>
<br>
Or even "lib: fix track count in get_track_from_id()" to respect the<br>
50/72 rule :)<br>
<br>
Regards<br>
<br>
> <br>
> Best regards,<br>
> Thomas Guillem<br>
> <br>
> On Thu, Jul 2, 2020, at 19:09, Romain Vimont wrote:<br>
> > LGTM<br>
> > <br>
> > Thank you<br>
> > <br>
> > (ref: #24921)<br>
> > <br>
> > On Thu, Jul 02, 2020 at 06:32:55PM +0200, Pascal Souchet wrote:<br>
> > > ---<br>
> > >  lib/media_player.c | 2 +-<br>
> > >  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> > > <br>
> > > diff --git a/lib/media_player.c b/lib/media_player.c<br>
> > > index 2f390468c4..9c7a7a76aa 100644<br>
> > > --- a/lib/media_player.c<br>
> > > +++ b/lib/media_player.c<br>
> > > @@ -1871,7 +1871,7 @@ x( libvlc_media_player_t *p_mi,<br>
> > >      for (size_t i = 0; i < ARRAY_SIZE(cats); ++i)<br>
> > >      {<br>
> > >          enum es_format_category_e cat = cats[i];<br>
> > > -        size_t count = vlc_player_GetTrackCount(player, VIDEO_ES);<br>
> > > +        size_t count = vlc_player_GetTrackCount(player, cat);<br>
> > >  <br>
> > >          for (size_t j = 0; j < count; ++j)<br>
> > >          {<br>
> > > -- <br>
> > > 2.17.1<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><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></div></div>