krasimir added a comment.
Generally I'm against introducing new style flags or flag options unless a
supported style requires it.
IMO this increases maintenance burden and can quickly lead to a big space of
style flags that contain tricky incompatibilities.
CHANGES SINCE LAST ACTION
https://
krasimir added inline comments.
Comment at: clang/docs/ClangFormatStyleOptions.rst:411
* ``SIS_Always`` (in configuration: ``Always``)
-Allow short if statements even if the else is a compound statement.
+Allow short if/else if/else statements when they are not compoun
klimek added inline comments.
Comment at: clang/docs/ClangFormatStyleOptions.rst:401
+ * ``SIS_AlwaysNoElse`` (in configuration: ``AlwaysNoElse``)
+Allow short if/else if statements even if the else is a compound statement.
+
MyDeveloperDay wrote:
> klimek w
MyDeveloperDay created this revision.
MyDeveloperDay added reviewers: djasper, klimek, reuk, JonasToth, alexfh,
krasimir, russellmcc.
MyDeveloperDay added a project: clang-tools-extra.
Herald added a subscriber: jdoerfert.
An addendum to D59087: [clang-format] [PR25010]
AllowShortIfStatementsOn
klimek added inline comments.
Comment at: clang/docs/ClangFormatStyleOptions.rst:401
+ * ``SIS_AlwaysNoElse`` (in configuration: ``AlwaysNoElse``)
+Allow short if/else if statements even if the else is a compound statement.
+
I'd try to make this either unco
MyDeveloperDay marked 2 inline comments as done.
MyDeveloperDay added inline comments.
Comment at: clang/docs/ClangFormatStyleOptions.rst:401
+ * ``SIS_AlwaysNoElse`` (in configuration: ``AlwaysNoElse``)
+Allow short if/else if statements even if the else is a compound state
rdwampler added inline comments.
Comment at: clang/include/clang/Format/Format.h:264
+/// If Else statements have no braces don't put them
+/// on the same line.
+/// \code
These comments can be used to auto generate the corresponding sections in
cla