QT_NO_DEBUG, Q_ASSERT(), and release build types

[Armin Berres]

On Wed, 14 Jan 09 02:47, Andreas Hartmetz wrote:
> I've looked up the #defines and tested it. Removing the !QT_NO_DEBUG condition 
> from kDebug() and setting QT_NO_DEBUG yields the desired result: debug output 
> is enabled, asserts (and therefore some internal checks in Qt) are disabled. 
> The size of libkdeui with -O1 and debug symbols (my usual build setting) was 
> reduced by about 300 kBytes out of ~46 MB, too. I didn't do any performance 
> tests; the differencen is probably small.
> 
> Any objections to committing this to trunk and/or 4.2?

Don't forget, that you also want to disable debug output (at least for
release builds) by default in this case. See the two patches in one of my
other messages.
I guess because of these changes (little changes though, but no bugfixes)
this is 4.3 material anyway.

Greetings,
Armin