[PATCH] D75791: [clang-format] Added new option IndentExternBlock
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun May 17 07:58:57 PDT 2020
MyDeveloperDay added a comment.
In D75791#2040558 <https://reviews.llvm.org/D75791#2040558>, @MarcusJohnson91 wrote:
> In D75791#2040532 <https://reviews.llvm.org/D75791#2040532>, @MyDeveloperDay wrote:
>
> > LGTM
>
>
> So what's the next step? I've never committed to LLVM before.
In my mind you have two choices
1. require commit access
2. get someone else to land it
If you think you'd like to continue to be involved and maybe help out from time to time (especially with reviews which is where we REALLY lack people) then I would recommend 1)
but I'm happy to do 2) for you if not, but you have to be prepared to support the patch a little afterwards. (no fire and forget please ;-) )
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D75791/new/
https://reviews.llvm.org/D75791
More information about the cfe-commits
mailing list