Message ID | 20190301020147.18567-3-richardw.yang@linux.intel.com |
---|---|
State | New |
Headers | show |
Series | CODING_STYLE: trivial update | expand |
diff --git a/CODING_STYLE b/CODING_STYLE index e175e6ea9a..465acf8bee 100644 --- a/CODING_STYLE +++ b/CODING_STYLE @@ -147,10 +147,10 @@ block to a separate function altogether. When comparing a variable for (in)equality with a constant, list the constant on the right, as in: -if (a == 1) { - /* Reads like: "If a equals 1" */ - do_something(); -} + if (a == 1) { + /* Reads like: "If a equals 1" */ + do_something(); + } Rationale: Yoda conditions (as in 'if (1 == a)') are awkward to read. Besides, good compilers already warn users when '==' is mis-typed as '=',