[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