[Bf-committers] MT_assert shenanigans

Jean-Luc Peurière bf-committers@blender.org
Mon, 14 Jun 2004 22:24:17 +0200


Le 14 juin 04, à 09:35, Alexander Ewering a écrit :

>
> On Sun, 13 Jun 2004, Chris Burt wrote:
>
>> After the MT_assert commit was done I did a compile and was sadly
>> introduced to seven compilation errors related to the fact that 
>> #include
>> "assert.h" was missing from seven individual header files.  (Thanks
>> Lukep for your help in finding this out).  Hmetalcowgirl and I went
>> through and added the asserts where necessary.  Here is a list of the
>

As this caused problems, I've done a rewind of the commit.

I have the correct behaviour by adding the define in release builds
in scons. can others confirms it works ?


Main Sconstruct file

if user_options_dict['BUILD_BINARY'] == 'release':
     cflags = extra_flags + release_flags + warn_flags
     if sys.platform == 'win32':
         defines += ['NDEBUG']
---------- add this line respecting indent ---------------
     defines += ['MT_NDEBUG']               #correct bugs #1292 and 1260
--------- end of changes ---------------
-- 
Jean-luc