[PATCH] D119599: [clang-format] Add option to align compound assignments like `+=`

sstwcw via Phabricator via llvm-commits llvm-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 llvm-commits mailing list