HazardyKnusperkeks added a comment.

Going the full way, to fix the number of empty lines after/before/between 
elements would be real nice. But even nicer would be if you can state a range.

But I think all those proposed options should not be added in one go.

In D104044#2812399 <https://reviews.llvm.org/D104044#2812399>, @darwin wrote:

> About the issue, let me explain it. It isn't bound to the google style or 
> LLVM style either, since both of them keep the first brace at the same line 
> of the namespace.

Then I would like to use the LLVM style in the tests, otherwise it suggests 
that the issue is a result of using google style.


Repository:
  rZORG LLVM Github Zorg

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104044/new/

https://reviews.llvm.org/D104044

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

Reply via email to