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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to