[PATCH] D123479: [clang-tidy] Support parenthesized literals in modernize-macro-to-enum

[Richard via Phabricator via cfe-commits]

LegalizeAdulthood added a comment.

In D123479#3442062 <https://reviews.llvm.org/D123479#3442062>, @njames93 wrote:

> Would it not be better if these parens were stripped in the fixit as they are unnecessary in the enum value decl?

I prefer checks to do one thing only in a straightforward manner.

It's easier to place things before and after the macro expansion rather than modify the contents
of the expansion.

It doesn't feel like the responsibility of this check is to decide which parentheses are necessary or not.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123479