[PATCH] D104044: [clang-format] Fix the issue of no empty line after namespace

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jun 13 07:34:16 PDT 2021


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



More information about the cfe-commits mailing list