Skip to content

Commit aa929ab

Browse files
committed
[docs] Update developer notes to discourage very long lines
1 parent 4bd5866 commit aa929ab

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

doc/developer-notes.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,11 @@ tool to clean up patches automatically before submission.
7575
on the same line as the `if`, without braces. In every other case,
7676
braces are required, and the `then` and `else` clauses must appear
7777
correctly indented on a new line.
78+
- There's no hard limit on line width, but prefer to keep lines to <100
79+
characters if doing so does not decrease readability. Break up long
80+
function declarations over multiple lines using the Clang Format
81+
[AlignAfterOpenBracket](https://clang.llvm.org/docs/ClangFormatStyleOptions.html)
82+
style option.
7883

7984
- **Symbol naming conventions**. These are preferred in new code, but are not
8085
required when doing so would need changes to significant pieces of existing

0 commit comments

Comments
 (0)