Hi!
19-Янв-2005 15:06 [EMAIL PROTECTED] (Kenneth J. Davis) wrote to
freedos-kernel@lists.sourceforge.net:
>>>+++ task.c 17 Jan 2005 18:34:10 -0000 1.44.2.5
>>>-#if DEBUG
>>>+#ifdef DEBUG
>>Fix:
>>#if TST+0
>>-------^^
KJD> I'm confused.
KJD> How does #ifdef DEBUG (the new code)
Oops... :( Sorry, I'm dubiously blind man - I visually entangle/miss
"+" and "-" characters in your update. :(
KJD> On a tangential [only slightly related] note, for debug code I want in
KJD> the source but
KJD> almost never compiled (something that helped track down a specific
KJD> issue, but generally not needed for a debug build), should I used
KJD> #ifdef DDEBUG ... #endif
KJD> or simply
KJD> #if 0 ... #endif
KJD> or
KJD> #ifdef <?? suggestions ??> ... #endif
I think, for (most) specific-purposed debug fragments, better to use
general-purpose "#if 0" - usually, when you debug some fragment, you not
need to see other cross-kernel debug messages. The more so, I don't think
that you need to devise name for conditional compilation guard - debugging
of specific fragments is rare thing, and in this case you may manually
(temporarily!) edit source (0 -> 1), instead manually editing your
config.bat (but don't forget to re_build_ the kernel after/if you restore
original source file with older date).
After all, I think, DEBUG name is useful for places, where is perofmed
output of general-purpose and user-friendly information. And such places
should be small amount, else profit of debugging output (which can't be
redirected to separate device with permanent memory!) will be losted in big
amount of garbage.
-------------------------------------------------------
This SF.Net email is sponsored by: IntelliVIEW -- Interactive Reporting
Tool for open source databases. Create drag-&-drop reports. Save time
by over 75%! Publish reports on the web. Export to DOC, XLS, RTF, etc.
Download a FREE copy at http://www.intelliview.com/go/osdn_nl
_______________________________________________
Freedos-kernel mailing list
Freedos-kernel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freedos-kernel
