[PATCH] D145778: [clang-tidy] Change readability-magic-numbers to allow numbers in type aliases.

Piotr Zegar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 10 04:31:41 PST 2023


PiotrZSL added a comment.

"Context not available." - use arcanist (just merge all local commits into one), or generate diff with full context (diff -U 9999999).



================
Comment at: clang-tools-extra/docs/clang-tidy/checks/readability/magic-numbers.rst:58
+
+   using containerType = CustomType<int, 30>;
+   struct OtherType {
----------------
example put under "Example with magic values refactored:" should be by default as (+-):
```
using containerType = CustomType<int, NUMBER_OF_ELEMENTS>;
```

as this is required by default.


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

https://reviews.llvm.org/D145778



More information about the cfe-commits mailing list