krasimir added inline comments.

================
Comment at: include/clang/Format/Format.h:638
+    /// \endcode
+    bool AfterCaseLabel;
     /// Wrap class definitions.
----------------
This comment seems outdated (the one in `ClangFormatStyleOptions.rst` seems 
more recent).

The `ClangFormatStyleOptions.rst` file is automatically generated from this 
file. Please update this comment and regenerate `ClangFormatStyleOptions.rst` 
by using the `clang/docs/tools/dump_format_style.py` script.


================
Comment at: lib/Format/Format.cpp:611
     Expanded.BraceWrapping = {true, true, true, true, true, true, true, true,
-                              true, true, true, true, true, true, true};
+                              true, true, true, true, true, true, true, true};
     break;
----------------
If Allman and GNU styles both follow the style suggested by this new flag, my 
mistake: this would be enough to add it to clang-format.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D52527/new/

https://reviews.llvm.org/D52527



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to