[PATCH] D52527: [clang-format] fix Bug 38686: add AfterCaseLabel to BraceWrapping

[Krasimir Georgiev via Phabricator via cfe-commits]

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