Update .clang-format to reflect WebKit style better.
authorheycam@apple.com <heycam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 23 Mar 2021 21:23:57 +0000 (21:23 +0000)
committerheycam@apple.com <heycam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 23 Mar 2021 21:23:57 +0000 (21:23 +0000)
https://bugs.webkit.org/show_bug.cgi?id=223229

First, SpaceBeforeCpp11BracedList is set to true so that a space
is introduced after a variable name and before a braced initializer.

Second, AlwaysBreakTemplateDeclarations is changed from false (an
invalid value) to No, which should result in no line breaks being
introduced in template declarations.  Although there is a mix of
template declaration line breaking styles in the codebase, changing
this option to No will prevent clang-format from introducing one
where the patch author decides to write the declaration all on one
line.

Reviewed by Alex Christensen.

* .clang-format:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@274901 268f45cc-cd09-0410-ab3c-d52691b4dbfc

.clang-format
ChangeLog

index 5d1ba09..eb85667 100644 (file)
@@ -17,7 +17,7 @@ AllowShortLoopsOnASingleLine: false
 AlwaysBreakAfterDefinitionReturnType: None
 AlwaysBreakAfterReturnType: None
 AlwaysBreakBeforeMultilineStrings: false
-AlwaysBreakTemplateDeclarations: false
+AlwaysBreakTemplateDeclarations: No
 BinPackArguments: true
 BinPackParameters: true
 BraceWrapping:   
@@ -94,6 +94,7 @@ SortUsingDeclarations: true
 SpaceAfterCStyleCast: false
 SpaceAfterTemplateKeyword: true
 SpaceBeforeAssignmentOperators: true
+SpaceBeforeCpp11BracedList: true
 SpaceBeforeParens: ControlStatements
 SpaceInEmptyParentheses: false
 SpacesBeforeTrailingComments: 1
index 0ab1d8f..0df3900 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,23 @@
+2021-03-23  Cameron McCormack  <heycam@apple.com>
+
+        Update .clang-format to reflect WebKit style better.
+        https://bugs.webkit.org/show_bug.cgi?id=223229
+
+        First, SpaceBeforeCpp11BracedList is set to true so that a space
+        is introduced after a variable name and before a braced initializer.
+
+        Second, AlwaysBreakTemplateDeclarations is changed from false (an
+        invalid value) to No, which should result in no line breaks being
+        introduced in template declarations.  Although there is a mix of
+        template declaration line breaking styles in the codebase, changing
+        this option to No will prevent clang-format from introducing one
+        where the patch author decides to write the declaration all on one
+        line.
+
+        Reviewed by Alex Christensen.
+
+        * .clang-format:
+
 2021-03-22  Adrian Perez de Castro  <aperez@igalia.com>
 
         [CMake] Wrong variable used in FindFontconfig used to pick version from pkg-config