[PATCH] D69764: [clang-format] Add East/West Const fixer capability

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 14 06:46:55 PDT 2021


MyDeveloperDay added a comment.

> So yes, I'm in favour of landing this patch (though not exactly in the current form, I'd prefer more future-proof options for instance, not only handling const)

I am in agreement, but I don't want to not putting more effort into improving the current design of this patch to handle more cases UNTIL we round out on the go/no go decision.

>From my perspective the use of violate is lower priority as its used like less than 0.01% as often as const. but I definitely think that we can add additional options on the lines of ReSharper to give even greater flexibility

F17930223: image.png <https://reviews.llvm.org/F17930223>


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69764/new/

https://reviews.llvm.org/D69764



More information about the cfe-commits mailing list