[PATCH] D30990: Add more examples to clang-format configuration

Daniel Jasper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 20 04:14:49 PDT 2017


djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.

Looks good, just minor remarks.



================
Comment at: include/clang/Format/Format.h:974
+  ///    # With:
+  ///    MacroBlockBegin: "^\
+  ///    NS_MAP_BEGIN|\
----------------
I'd contract these to a single line.


================
Comment at: include/clang/Format/Format.h:1007
+  ///    int f() {                              int f() {
+  ///    int = 1;                                   int i = 1;
+  ///                                               i = foo();
----------------
nit: The indent on the right is incorrect (i.e. should not be different).


https://reviews.llvm.org/D30990





More information about the cfe-commits mailing list