From: Mark Taylor <[EMAIL PROTECTED]>
Subject: Re: [MP3 ENCODER] new common macro
Date: Fri, 23 Jun 2000 11:09:03 -0600

> > 
> > Hi all,
> > 
> > I added some macros in util.h.
> > 
> 
> These look great!  But one small complaint:  do we really
> need a LAME_ASSERT macro for assert?  assert is already
> a macro, and part of ANSI C.  assert() is disabled by default,
> and should only be used for tracking down bugs in the encoding
> library.  
> 
> Mark
> 

Maybe there are many developers who want to enable assert()
in LAME library.  He would want to know if his binary of
the LAME library works correctly.  I thought providing
alternative assertion macro will help in such case.
However, you are right.  If one want another assertion macro,
he can replace "#include <assert.h>" with "#include <myassert.h>".
I will remove LAME_ASSERT().

Regards,

Iwasa Kazmi
--
MP3 ENCODER mailing list ( http://geek.rcc.se/mp3encoder/ )

Reply via email to