[vlc-devel] Re: vlc: svn commit r19278 (courmisch)

Rémi Denis-Courmont rem at videolan.org
Sat Mar 10 19:59:45 CET 2007


Le samedi 10 mars 2007 20:57, Jean-Paul Saman a écrit :
> Subversion daemon wrote:
> > r19278 | courmisch | 2007-03-10 17:57:51 +0100 (Sat, 10 Mar 2007) |
> > 2 lines Changed paths:
> >    M /trunk/modules/access_filter/dump.c
> >
> > Revert [19270]
> >
> >  > http://trac.videolan.org/vlc/changeset/19278
>
> Sorry courmisch, but I disagree with this revert. Line 159 checks for
> (len == 0), which makes assert(len>0) checking further in the code
> useless.

OF COURSE, assert() is useless. It is supposed to be. It's just a 
reminder in case someone reads/modifies the code.

-- 
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/f27b2008/attachment.sig>


More information about the vlc-devel mailing list