Replies: 1 comment
-
Code is code and we should not care too much about personal (my) taste. I agree to use The imported files (now SEGGER_RTT.*) stay excluded from auto-formatting. (edited) |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
I created this issue to centralize discussions about code style (solution and problems). I took some comments from the last PR :
The problem is mixing preprocessor directives indenting with statements (if, for, switch, ...) indenting that might yield very misleading looking code when preprocessor is heavy used with code.
example:
I think we can reverse the logic, set
IndentPPDirectives: None
and add manual indentation when it is pertinent withclang-format off
.Edit: It might be worth it. Most styles do not indent directives.
I think you shouldn't code based on formatter (most of the time). We add unnecessary complexity when programming
Beta Was this translation helpful? Give feedback.
All reactions