[PATCH] Fix bug in clang-format while merging short function (PR19461)
Daniel Jasper
djasper at google.com
Mon May 5 02:46:30 PDT 2014
A few indentations aside, this looks good.
================
Comment at: unittests/Format/FormatTest.cpp:7477
@@ -7446,1 +7476,3 @@
+verifyFormat("#ifdef _DEBUG\n"
+ "int foo(int i = 0)\n"
----------------
The indentation here is off. Maybe use clang-format ;-).
================
Comment at: unittests/Format/FormatTest.cpp:7487
@@ +7486,3 @@
+
+ verifyFormat("void foo() {}\n"
+ "void bar()\n"
----------------
Same here.
================
Comment at: unittests/Format/FormatTest.cpp:7628
@@ +7627,3 @@
+
+verifyFormat("#ifdef _DEBUG\n"
+ "int foo(int i = 0)\n"
----------------
Same here.
================
Comment at: unittests/Format/FormatTest.cpp:7638
@@ +7637,3 @@
+
+ verifyFormat("void foo() {}\n"
+ "void bar()\n"
----------------
Same here.
http://reviews.llvm.org/D3439
More information about the cfe-commits
mailing list