[vlc-devel] Re: vlc: svn commit r19276 (jpsaman)

Rémi Denis-Courmont rem at videolan.org
Sat Mar 10 17:53:30 CET 2007


Le samedi 10 mars 2007 18:42, Subversion daemon a écrit :
> r19276 | jpsaman | 2007-03-10 17:42:21 +0100 (Sat, 10 Mar 2007) | 1
> line Changed paths:
>    M /trunk/modules/misc/playlist/m3u.c
>
> Sanity check

This is utterly useless. VLC assumes that table allocations succeeds, 
and will crash if they don't. And even if it did not, the right place 
to check for failure, is straight after the allocation, not at a 
completely differently place in the code.

Removing assertions and/or adding useless sanity checks only hides bugs; 
it does not solve them at all. That's very BAD.

-- 
Rémi Denis-Courmont
http://www.remlab.net/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20070310/514851d0/attachment.sig>


More information about the vlc-devel mailing list