|
From: Miro J. <me...@me...> - 2003-08-25 23:54:57
|
>Summarize: Excellent summary. >The macro DEBUG, is used also in Debugging.h (in Mac OS system >headers) which will be often included in precompiled headers. The same holds of NDEBUG and _DEBUG. The way they are currently handled is that Whisper makes sure that NDEBUG and _DEBUG are synchronized with DEBUG, and then consistently uses DEBUG itself. I think that's a good way to handle the Mac OS DEBUG macro as well -- replace it with WHISPER_DEBUG and make sure that DEBUG, NDEBUG, and _DEBUG are all synchronized with it (see XDebug.h). >Draw back: macros may have lengthy names. I can live with that, but I would prefer if we did not add a category to those macros which we anticipate will be used often -- WHISPER_ASSERT et al. It's enough that we are lengthening them with WHISPER_, we don't need to add a category as well. meeroh -- <http://web.meeroh.org/> | KB1FMP A: Because it reverses the logical flow of conversation. Q: Why is top posting frowned upon? |