================
@@ -397,6 +398,7 @@ TEST_F(FormatTest, RemovesEmptyLines) {
   Style.BraceWrapping.AfterClass = true;
   Style.BraceWrapping.AfterFunction = true;
   Style.KeepEmptyLines.AtStartOfBlock = false;
+  Style.KeepEmptyLines.AtEndOfBlock = false;
----------------
arkq wrote:

> The presence suggests the executed test depends on that value. If it is 
> relevant, that the option is set to false then ASSERT_EQ on that.

OK I'll remove that one. However, the `AtStartOfBlock` is also not tested here 
:D The test passes regardless of the value (setting it to `true` does not fail 
the test).

https://github.com/llvm/llvm-project/pull/224223
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to