[PATCH] D117522: [clang-tidy] Add modernize-macro-to-enum check
Nathan James via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 17 18:01:03 PST 2022
njames93 added inline comments.
================
Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize-macro-to-enum.rst:45
+ enum {
+ RED = 0xFF0000,
+ GREEN = 0x00FF00,
----------------
Eugene.Zelenko wrote:
> It'll be reasonable to support indentation. Option could be used to specify string (tab or several spaces).
That's not actually necessary as clang-tidy runs clang-format on the fixes it generates
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117522/new/
https://reviews.llvm.org/D117522
More information about the cfe-commits
mailing list