forget what i just said, didn't see the test before the switch.<br><br>Can you change it or should i sent another patch ?<br><br><div class="gmail_quote">On Sun, Oct 2, 2011 at 9:25 PM, XilasZ <span dir="ltr"><<a href="mailto:xilasz@gmail.com">xilasz@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">In theory yes, but since the value comes from the setup callback outside libvlc, we can't be sure. i think it's better to give an error message.<div>
<div></div><div class="h5"><br><br><div class="gmail_quote">2011/10/2 Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le dimanche 2 octobre 2011 22:17:33 XilasZ, vous avez écrit :<br>
<div>> No particular reason, it was done that way in the aout i copy/paste it from<br>
> (don't know which one, they are all different).<br>
><br>
> But your are right it makes more sense in 'case 2', 'default' should be a<br>
> "format not supported" case.<br>
<br>
</div>default should actually be assert(0). It's impossible.<br>
<div><div></div><div><br>
--<br>
Rémi Denis-Courmont<br>
<a href="http://www.remlab.net/" target="_blank">http://www.remlab.net/</a><br>
<a href="http://fi.linkedin.com/in/remidenis" target="_blank">http://fi.linkedin.com/in/remidenis</a><br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br>
</div></div></blockquote></div><br>