PiotrZSL added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/readability/MagicNumbersCheck.cpp:40 + if (Node.get<TypeAliasDecl>() || Node.get<TypedefNameDecl>()) + return true; ---------------- This should be configurable, someone else may don't want magic numbers in template arguments. So we shouldn't change default behaviour. ================ Comment at: clang-tools-extra/docs/ReleaseNotes.rst:196 +- Improved :doc:`readability-magic-numbers + <clang-tidy/checks/readability/magic-numbers>` check. The check now allows for + magic numbers in type aliases such as ``using`` and ``typedef`` declarations. ---------------- consider: "check, now allows..." Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145778/new/ https://reviews.llvm.org/D145778 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits