Might be a silly question, but why the goto instead of just freeing before the return NULL (in each block)? I know code duplication isn't a good idea, but neither is goto afaik. Please enlighten me :)<br><br>Marnik<br>
<br><div><span class="gmail_quote">2008/6/2, Pierre d'Herbemont <<a href="mailto:pdherbemont@free.fr">pdherbemont@free.fr</a>>:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br> On Jun 2, 2008, at 11:49 AM, Dominique Leuenberger wrote:<br> <br> > Quoting git version control <<a href="mailto:git@videolan.org">git@videolan.org</a>>:<br> ><br> >> vlc | branch: 0.8.6-bugfix | RĂ©mi Denis-Courmont<br>
 >> <<a href="mailto:rdenis@simphalempin.com">rdenis@simphalempin.com</a>> | Sat May 31 23:19:32 2008 +0300|<br> >> [bd6877f9e3e0738013521bc98fe6bd12697a59ce]<br> ><br> >> +        goto error;<br>
 ><br> > WOW! And I thought the times of 'goto' are over for good. I've seen<br> > that you can use it twice.. but is this sacrifice really worthy?<br> <br> Well, goto are not over for good, especially in C.<br>
 <br> In that precise use it is the most appropriate and common in C code.<br> Don't worry, it is being used wisely.<br> <br> But you can try to look for goto abuse in VLC code if you like :)<br> <br> Pierre.<br> _______________________________________________<br>
 vlc-devel mailing list<br> To unsubscribe or modify your subscription options:<br> <a href="http://mailman.videolan.org/listinfo/vlc-devel">http://mailman.videolan.org/listinfo/vlc-devel</a><br> </blockquote></div><br>