[vlc-devel] commit: lpcm: kill a warning (Jean-Baptiste Kempf )

Hugo Beauzee-Luyssen beauze.h at gmail.com
Wed Dec 16 12:52:17 CET 2009


2009/12/16 Rémi Denis-Courmont <remi at remlab.net>

>
> On Wed, 16 Dec 2009 11:58:13 +0100, Hugo Beauzee-Luyssen
> <beauze.h at gmail.com> wrote:
> > Isn't that supposed to be true only in debug mode ?
>
> Assertion are supposed to be true in release mode as well, you know. Just
> because they are checked does not turn them false.
>

I agree, however the point is the unreachable code, which isn't unreachable
if you're building in release mode, since assert is removed.


>
> > An unreachable code in debug mode seems better to me than an unset return
> > value in release :o
>
> And it seems much worse to me because developers should all be using debug
> builds.
>

Which doesn't prevent a developer to think about his code in release mode.

-- 
Hugo Beauzée-Luyssen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20091216/2a0cb78e/attachment.html>


More information about the vlc-devel mailing list