commit-classpath
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Patch #562] Fix 'configure --enable-debug'


From: nobody
Subject: [Patch #562] Fix 'configure --enable-debug'
Date: Tue, 15 Oct 2002 00:24:21 -0400

Patch #562 has been updated. 

Project: 
Category: None
Status: Open
Summary: Fix 'configure --enable-debug'

Follow-Ups:

Date: 2002-Oct-10 11:00
By: crawley

Comment:
The patch fixes the '--enable-debug' switch to work
as the comments suggest that it should,  The C 'DEBUG'
flag is now undefined by default and gets defined (as 1)
if you configure with '--enable-debug'.

This reverses an effect of a recent change that hardwired
'DEBUG' to '1'.
-------------------------------------------------------

Date: 2002-Oct-10 11:15
By: mkoch

Comment:
Shouldnt there NDEBUG be defined in the non-debug case ?

Some libc functions act on NDEBUG, e.g. assert(). See "man assert".
-------------------------------------------------------

Date: 2002-Oct-10 14:52
By: mark

Comment:
I notice another AC_DEFINED(DEBUG) at the top of the file.
Shouldn't that be removed?
-------------------------------------------------------

Date: 2002-Oct-15 00:24
By: None

Comment:
I don't think we currently use assert, but there is no harm in adding NDEBUG to 
the normal non-debug case in case someone would like to use it when debugging.  
No idea if assert behaves this way on every platform.
-------------------------------------------------------

-------------------------------------------------------
For more info, visit:

http://savannah.gnu.org/patch/?func=detailpatch&patch_id=562&group_id=85




reply via email to

[Prev in Thread] Current Thread [Next in Thread]