rusefi
/
atbetaflight
mirror of https://github.com/rusefi/atbetaflight.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Added exception for ternary conditional operator.

This commit is contained in:
mikeller 2018-12-30 10:15:12 +13:00
parent 82e999e589
commit af0a3ad58b
1 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
docs/development/CodingStyle.md
View File

@ -272,6 +272,14 @@ In rare cases such constructs can be justified but only when you have considered
In expressions, parentheses should only be used where they are required, i.e. where operator precedence will not evaluate in the right order, or where a compiler warning is triggered without parentheses. This brings all expressions into a canonical form, and avoids the problem of different developers having different ideas of what 'easy to read' expressions are. In expressions, parentheses should only be used where they are required, i.e. where operator precedence will not evaluate in the right order, or where a compiler warning is triggered without parentheses. This brings all expressions into a canonical form, and avoids the problem of different developers having different ideas of what 'easy to read' expressions are.
One exception to this rule is the ternary conditional operator
```
pidStabilisationEnabled = (pidControllerState == PID_STABILISATION_ON) ? true : false
```
Here, the condition shall be enclosed in braces, to make the ternary operator easier to spot when reading left to right.
# Includes # Includes
All files must include their own dependencies and not rely on includes from the included files or that some other file was included first. All files must include their own dependencies and not rely on includes from the included files or that some other file was included first.