[PATCH] Fixes bug 20587 - Add K&R break before braces style

Roman Kashitsyn romankashicin at gmail.com
Mon Aug 11 01:59:48 PDT 2014


Well, just to clarify:
Are we decided to follow AStyle way and break after classes and namespaces, but not before structs?

================
Comment at: unittests/Format/FormatTest.cpp:8072
@@ +8071,3 @@
+  verifyFormat("namespace a {\n"
+               "class A {\n"
+               "  void f()\n"
----------------
Daniel Jasper wrote:
> Can you add small test for structs just so that we realize when we are changing the behavior there?
Ok, I'll add one.

http://reviews.llvm.org/D4837






More information about the cfe-commits mailing list