[vlc-devel] Re: vlc: svn commit r19276 (jpsaman)
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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 197 bytes
Desc: not available
More information about the vlc-devel