CODE_STYLE: Use boolean conditions

Rather than use TRUE, FALSE, NULL, 0 or != 0 in if/while conditions.

Additionally, corrected some example code to adhere to the recommended
coding style.

diff --git a/docs/CODE_STYLE.md b/docs/CODE_STYLE.md
index 8cfb26e6fd0bb488e080718878a96c588bcfdc24..d77fc44764b4642a170f3da43360e3d061071933 100644 (file)
--- a/docs/CODE_STYLE.md
+++ b/docs/CODE_STYLE.md
@@ -101,6 +101,20 @@ between the keyword and the open parenthesis. Like this:
       /* loop forever */
     }
 
+## Use boolean conditions
+
+Rather than test a conditional value such as a bool against TRUE or FALSE, a
+pointer against NULL or != NULL and an int against zero or not zero in
+if/while conditions we prefer:
+
+CURLcode result = CURLE_OK;
+
+result = do_something();
+if(!result) {
+  /* something went wrong */
+  return result;
+}
+
 ## No assignments in conditions
 
 To increase readability and reduce complexity of conditionals, we avoid
@@ -112,7 +126,7 @@ assigning variables within if/while conditions. We frown upon this style:
 and instead we encourage the above version to be spelled out more clearly:
 
     ptr = malloc(100);
-    if(ptr == NULL)
+    if(!ptr)
       return NULL;
 
 ## New block on a new line
@@ -164,7 +178,7 @@ depending on a build-time conditional:
     #ifdef HAVE_MAGIC
     void magic(int a)
     {
-      return a+2;
+      return a + 2;
     }
     #else
     #define magic(x) 1