[PATCH] D75791: [clang-format] Added new option IndentExternBlock
Marcus Johnson via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue May 19 06:28:36 PDT 2020
MarcusJohnson91 added a comment.
In D75791#2043758 <https://reviews.llvm.org/D75791#2043758>, @MyDeveloperDay wrote:
> Sorry to "go around the houses" but we'll get there in the end...I think we are close
I think we're close too.
Your other comment was interesting, about testing the styles to make sure they haven't changed with these new options, that didn't occur to me.
I think it might be worth looking into.
I'm just not really sure how we could do such a thing, or if after the option is submitted if it'd really be worth it to have run because I don't really think this option will be further modified, but the original AfterExternBlock guy didn't think anyone would want to expand that either and as a result we had to work around some of the issues with his design so who can really tell what the future holds?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75791/new/
https://reviews.llvm.org/D75791
More information about the cfe-commits
mailing list