From bc17221570e7ef3a127370d29d851fecb71d7bce Mon Sep 17 00:00:00 2001
From: Peter Hutterer <peter.hutterer@who-t.net>
Date: Wed, 8 Jul 2015 08:38:11 +1000
Subject: Add more rules to CODING_STYLE

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
---
 CODING_STYLE | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/CODING_STYLE b/CODING_STYLE
index 3648a4e2..c5336cc0 100644
--- a/CODING_STYLE
+++ b/CODING_STYLE
@@ -62,6 +62,26 @@
   	useit(c);
   }
 
+- do not mix function invocations and variable definitions.
+
+  wrong:
+
+  {
+  	  int a = foo();
+  	  int b = 7;
+  }
+
+  right:
+  {
+  	  int a;
+  	  int b = 7;
+
+  	  a = foo();
+  }
+
+  There are exceptions here, e.g. tp_libinput_context(),
+  litest_current_device()
+
 - if/else: { on the same line, no curly braces if both blocks are a single
   statement. If either if or else block are multiple statements, both must
   have curly braces.
@@ -88,3 +108,9 @@
   #include <libevdev/libevdev.h>
 
   #include "libinput-private.h"
+
+- goto jumps only to the end of the function, and only for good reasons
+  (usually cleanup). goto never jumps backwards
+
+- Use stdbool.h's bool for booleans within the library (instead of 'int').
+  Exception: the public API uses int, not bool.
-- 
cgit v1.2.3