[vlc-devel] commit: lpcm: kill a warning (Jean-Baptiste Kempf )
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
Which doesn't prevent a developer to think about his code in release mode.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the vlc-devel