[PATCH] D119599: [clang-format] Add option to align compound assignments like `+=`
sstwcw via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 22 15:28:05 PST 2022
sstwcw marked 18 inline comments as done.
sstwcw added a comment.
The YAML stuff are now in D120363 <https://reviews.llvm.org/D120363>.
================
Comment at: clang/docs/ClangFormatStyleOptions.rst:1408-1415
+ Precise control over the wrapping of braces.
+
+ .. code-block:: c++
+
+ # Should be declared this way:
+ BreakBeforeBraces: Custom
+ BraceWrapping:
----------------
curdeius wrote:
> That's an unrelated change. Could you please do it in another (NFC) revision?
It's in D120361.
================
Comment at: clang/docs/tools/dump_format_style.py:121
def __str__(self):
- return '\n'.join(map(str, self.values))
+ return self.comment + '\n' + '\n'.join(map(str, self.values))
----------------
MyDeveloperDay wrote:
> Can this change be separate? why is this needed? Could you add a screengrab of the html that it generates?
It's in D120361.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119599/new/
https://reviews.llvm.org/D119599
More information about the cfe-commits
mailing list