[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