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

Jean-Paul Saman jean-paul.saman at planet.nl
Sat Mar 10 19:55:13 CET 2007


Jean-Paul Saman wrote:
> Rémi Denis-Courmont wrote:
>> Can you see the difference between: “assert (len > 0);” and “assert (len
>>> = 0);” ? At least, I can, and GCC can too (it prints a warning about 
>> always true statement in the second case).
>>
>> Regards,
>>
> Remi, I can see the difference but clearly my earlier observation was 
> wrong. Thanks for spotting it.
> 
Well I take those words back. Line 159 already checks for the case (len ==0)

line 159:
     if (len == 0)
         return; /* no useful data */

This makes the assert( len > 0 ) useless because it is guaranteed to be 
greater then zero.

gtz,
Jean-Paul Saman.

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



More information about the vlc-devel mailing list