[PATCH] D92257: [clang-format] Add option to control the space at the front of a line comment
Björn Schäpers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 14 02:13:58 PST 2021
HazardyKnusperkeks added a comment.
In D92257#2497535 <https://reviews.llvm.org/D92257#2497535>, @MyDeveloperDay wrote:
> My assumption is that you want to stick with the minimum and maximum is that correct?
Otherwise I have to make a breaking change, or not achieve at all what I want. So either abandon this or we need a minimum and maximum.
Although right now I have changed an existing test because in that case the behavior changed (as noted inline) and a test case still to decide how to advance.
If that little break is not acceptable I see no further base to pursue this and have to decide to drop it totally or only apply it locally.
And thanks for bringing this up again, currently I have very little time to work on clang-format and only react on the mails. Even while I have many things I want to add/change or cases where it is currently misformatted.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D92257/new/
https://reviews.llvm.org/D92257
More information about the cfe-commits
mailing list