================
@@ -170,6 +170,7 @@ TEST_F(FormatTest, RemovesEmptyLines) {
CustomStyle.BreakBeforeBraces = FormatStyle::BS_Custom;
CustomStyle.BraceWrapping.AfterNamespace = true;
CustomStyle.KeepEmptyLines.AtStartOfBlock = false;
+ CustomStyle.KeepEmptyLines.AtEndOfBlock = false;
----------------
HazardyKnusperkeks wrote:
> So, setting this value to `false` makes sure that if the default will be
> changed for some reasons this test will not break.
That will **never** happen, as it would break a lot of formatting. Once a
default is set, it will not be changed. (This applies to the LLVM style, other
styles may adapt, if their owner decides to change their style guide, but that
is also controversial.)
https://github.com/llvm/llvm-project/pull/224223
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits