[vlc-devel] Re: [PATCH] Allow configuration of JACK connections

[Jon Griffiths]

> If you follow that logic to its end, you don't need to check for 
> non-null-ity before invoking free() since free(NULL) is a no-op.

Thats true, but only on conforming systems
(http://www.gnu.org/software/autoconf/manual/html_node/Function-Portability.html).


I prefer to check: since its an error path efficiency isn't
important, and if VLC gets ported to some device with a
non-conforming libc, nothing breaks. I would expect gcc should be
able to remove the test when optimising, but a quick test shows it
doesn't.

Cheers
Jon






__________________________________________________
Do You Yahoo!?
Tired of spam?  Yahoo! Mail has the best spam protection around 
http://mail.yahoo.com 

-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html