[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