[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