[vlc-devel] Re: vlc: svn commit r14107 (xtophe)

[Rémi Denis-Courmont]

Le Mercredi 1 Février 2006 15:51, Christophe Mutricy a écrit :
> On Wed, Feb 01, 06 at 15:39 +0100, Subversion daemon wrote:
> > Test for assert() in configure.
>
> To start the controversy and because vlc-devel is too quiet and
> consensual compare to others.
>
> Should we allow assert() in our code or should it be only for our
> local tree ?

The standard assert() can be removed by defining NDEBUG (and the 
transcoding plugins already do that). IMHO, we should allow assert() 
but define NDEBUG when --enable-debug is not specified at configure 
time.

-- 
Rémi Denis-Courmont
http://www.simphalempin.com/home/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20060201/18792897/attachment.sig>