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

Reply via email to