doc: add note about bool into coding style
authorMarko Kovacevic <marko.kovacevic@intel.com>
Mon, 17 Sep 2018 15:24:29 +0000 (16:24 +0100)
committerThomas Monjalon <thomas@monjalon.net>
Sat, 24 Nov 2018 20:43:22 +0000 (21:43 +0100)
Added a note into the coding style to
highlight the use of a bool within a struct

Signed-off-by: Marko Kovacevic <marko.kovacevic@intel.com>
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
doc/guides/contributing/coding_style.rst

index 19445c1..d96698a 100644 (file)
@@ -247,6 +247,15 @@ Structure Declarations
 * Use of the structures should be by separate variable declarations and those declarations must be extern if they are declared in a header file.
 * Externally visible structure definitions should have the structure name prefixed by ``rte_`` to avoid namespace collisions.
 
+.. note::
+
+    Uses of ``bool`` in structures are not preferred as is wastes space and
+    it's also not clear as to what type size the bool is. A preferred use of
+    ``bool`` is mainly as a return type from functions that return true/false,
+    and maybe local variable functions.
+
+    Ref: `LKML <https://lkml.org/lkml/2017/11/21/384>`_
+
 Queues
 ~~~~~~