https://gcc.gnu.org/g:0c83096f19b075ac1b80113828188a0fd64400af
commit r15-6051-g0c83096f19b075ac1b80113828188a0fd64400af
Author: Matthew Malcomson <mmalcom...@nvidia.com>
Date: Mon Dec 9 10:51:44 2024 +0000
clang-format AlwaysBreakAfterReturnType to TopLevelDefinitions
The previous value of TopLevel meant that the function name of
declarations would also be on a new line. THis does not match the
current formatting of headers.
Manual testing done on c-common.h.
Also set BraceWrapping.BeforeWhile to true to match the formatting
specified for do/while loops in GNU coding standards.
https://www.gnu.org/prep/standards/standards.html#Formatting
Ok for trunk?
contrib/ChangeLog:
* clang-format: AlwaysBreakAfterReturnType set to
TopLevelDefinitions and BraceWrapping.BeforeWhile set to true.
Signed-off-by: Matthew Malcomson <mmalcom...@nvidia.com>
Diff:
---
contrib/clang-format | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/contrib/clang-format b/contrib/clang-format
index 95f1455c14db..4ed50ab6b268 100644
--- a/contrib/clang-format
+++ b/contrib/clang-format
@@ -27,7 +27,7 @@
---
Language: Cpp
AccessModifierOffset: -2
-AlwaysBreakAfterReturnType: TopLevel
+AlwaysBreakAfterReturnType: TopLevelDefinitions
BinPackArguments: true
BinPackParameters: true
BraceWrapping:
@@ -42,6 +42,7 @@ BraceWrapping:
AfterUnion: true
BeforeCatch: true
BeforeElse: true
+ BeforeWhile: true
IndentBraces: true
SplitEmptyFunction: false
BreakBeforeBinaryOperators: All
