darwin added a comment.

In D104044#2814016 <https://reviews.llvm.org/D104044#2814016>, 
@HazardyKnusperkeks wrote:

> 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.

Yes, adding a new option is a new feature requirement. What I am trying to do 
here is to keep the clang-format behaving reasonably.

> 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.

Wouldn't this suggest that this issue is a result of using LLVM 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