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

Darwin Xu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jun 13 07:26:29 PDT 2021


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



More information about the cfe-commits mailing list