mxbOctasic added inline comments.

================
Comment at: unittests/Format/FormatTest.cpp:285
@@ +284,3 @@
+
+  EXPECT_EQ("class Foo\n"
+            "{\n"
----------------
djasper wrote:
> How does this break? I generally add an Before and After of one of the test 
> cases into my patch descriptions.
This is a fun one.

The newline between `Foo() {}` and `void Funk() {}` is removed.
This is a combination of the line merge problem and 
`KeepEmptyLinesAtTheStartOfBlocks`.

`Last` on the merged line for `Foo() {}` points to `{` instead of `}`, so the 
newline before `void` is seen as a newline at the start of a block. (Checks 
`PreviousLine->Last->is(tok::l_brace)`)


http://reviews.llvm.org/D19063



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

Reply via email to