[ds6-devel] Readme.maint update
Chris Leishman
chris at leishman.org
Mon Jan 27 14:13:12 CET 2003
Hi,
This rewording of Readme.maint clears up Mauros last addition:
--- README.Maint 25 Jan 2003 14:42:36 -0000 1.10
+++ README.Maint 27 Jan 2003 03:09:47 -0000
@@ -171,18 +171,18 @@
return ret;
}
-* Do not to use global (non static) variables. In the 100% of cases there
- is a smarter way to achieve the same result you would get using global
- variables.
+* Avoid declaring global variables, especially external (non-static) ones.
+ There is rarely a need for this and it can indicate a badly thought out design.
-* Try not to #define preprocessor constants when possible (especially constants
- that are used only inside a single module). The definition:
+* Try not to #define preprocessor constants when possible (usually this is when
+ the constant is only used in a single module and will be inlined by the
+ compiler). The definition:
- static const int VERY_VERBOSE_MODE = 0x02;
+ static const int SOME_NUMBER = 0x02;
is much better than:
- #define VERY_VERBOSE_MODE 0x02
+ #define SOME_NUMBER 0x02
Please consider making this change, and reverting the verbose_mode code.
Regards
More information about the ds6-devel
mailing list