[PATCH] D53771: [clang-tidy] Avoid C arrays check

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 26 12:17:03 PDT 2018


lebedev.ri added inline comments.


================
Comment at: test/clang-tidy/misc-avoid-c-arrays.cpp:14
+
+template <typename T, int Size>
+class array {
----------------
JonasToth wrote:
> Please add a case with templates, where `T` itself is the array type, maybe there are other fancy template tricks that could create an array implictly?
Oh i see, `array<int[4], 2>` is not diagnosed, that is where it gets interesting :S
So i need to match `type(arrayType())`, and then find the location to complain about
(the type does not seem to have it).


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53771





More information about the cfe-commits mailing list