HazardyKnusperkeks added a comment.

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

Not in my opinion, because all the tests I've seen and worked with are LLVM 
style, because that's the default. But use what ever style you want.


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