Add else and catch Coding style rule (#1868)
authorEfimov Alexander/AI Tools Lab/./Samsung Electronics <a.efimov@samsung.com>
Fri, 19 Oct 2018 16:34:08 +0000 (19:34 +0300)
committerРоман Михайлович Русяев/AI Tools Lab /SRR/Staff Engineer/삼성전자 <r.rusyaev@samsung.com>
Fri, 19 Oct 2018 16:34:08 +0000 (19:34 +0300)
Specify position of else and catch words in code

Signed-off-by: Efimov Alexander <a.efimov@samsung.com>
contrib/nnc/doc/codestyle.rst

index 5624893..426bba8 100644 (file)
@@ -106,7 +106,29 @@ Control flow contructions:
 --------------------------
 
 - Function call should have no spaces around parentheses and have spaces after commas: **someFunc(arg1, arg2);**
-- **if**, **while**, **for** should have space after reserved word: **if (cond)** **while (cond)** **for (cond)**
+- **if**, **while**, **for** should have space after reserved word: **if (cond)**, **while (cond)**, **for (cond)**
+- **else**, **catch** key words should be placed on the same line as closing code block brace:
+
+.. code-block:: c++
+
+   if (cond1) {
+     doThis();
+   } else if (cond2) {
+     doThat();
+   } else {
+     getConfused();
+   }
+
+.. code-block:: c++
+
+   try {
+     doThis();
+   } catch (PassException& e) {
+     doThat();
+   } catch (OtherException& e) {
+     doABarrelRoll();
+   }
+
 - **switch** should always have **default** case, if all cases are checked and **default** is redundant, place **assert(false)** in it
 - Avoid redundant braces inside switch **case**
 - **case** labels should be indented by 2 spaces relative to **switch**
@@ -122,7 +144,6 @@ Control flow contructions:
    else
      b = that;
 
-
 .. code-block:: c++
 
    switch(cond) {