JonasToth added inline comments.
================
Comment at: test/clang-tidy/misc-avoid-c-arrays.cpp:14
+
+template <typename T, int Size>
+class array {
----------------
lebedev.ri wrote:
> 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).
you could leave it as known limitation as well.
If the type `T` is then diagnosed as array creation through
template-instantiation the warning is still valid.
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
