<html><head></head><body>Okay but that just ends up breaking the normal return code path, so it seems wrong even then.<br><br><div class="gmail_quote">Le 20 avril 2020 09:56:13 GMT+03:00, Thomas Guillem <thomas@gllm.fr> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail">hello,<br><br>On Sun, Apr 19, 2020, at 10:55, Rémi Denis-Courmont wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Le sunnuntaina 19. huhtikuuta 2020, 11.05.02 EEST Jorge Bellon-Castro a écrit <br>:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;">Hello again,<br>There is an earlier check doing the same goto.<br></blockquote>That can't be right.<br>Existing code cannot goto to a label that does not exist yet.<br></blockquote><br>This was also what I thought when I first read the diff, then if you look again, you will see:<br><br>-invalid_track_id:<br>+invalid_track_id:<br><br>> <br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;">Also the patch seems to emit an error every time, even when there's no error.<br><br>-- <br>Rémi Denis-Courmont<br>Hagalund ny stad, Nylands republik<hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a><br></blockquote><hr>vlc-devel mailing list<br>To unsubscribe or modify your subscription options:<br><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a></pre></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>