added non-negated condition guideline

1 files changed, 28 insertions, 1 deletions

diff --git a/etc/ACE-guidelines.html b/etc/ACE-guidelines.html
index 28978806ca1..e339cdcb4f1 100644
--- a/etc/ACE-guidelines.html
+++ b/etc/ACE-guidelines.html
@@ -264,6 +264,33 @@ to Graham for providing the OSE tools!<p>
         declaration is seen.  Of course, we should avoid using
         static variables at all.<p>
 
+    <li>It is usually clearer to write conditionals that have
+        both branches without a negated condition.  For example,<p>
+
+        <pre>
+        if (test)
+          {
+            // true branch
+          }
+        else
+          {
+            // false branch
+          }
+        </pre><p>
+
+        is preferred over:<p>
+
+        <pre>
+        if (! test)
+          {
+            // false test branch
+          }
+        else
+          {
+            // true test branch
+          }
+        </pre><p>
+
     <li>If a cast is necessary, avoid use of ANSI-style casts,
         <em>e.g.</em>, <code>int (foo)</code>.  Instead, use
         the ACE_static_cast macro:
@@ -383,7 +410,7 @@ to Graham for providing the OSE tools!<p>
 
 <hr>
 <!-- hhmts start -->
-Last modified: Mon Nov 10 18:27:08 CST 1997
+Last modified: Tue Dec  2 09:54:01 CST 1997
 <!-- hhmts end -->
 
 </body>