[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