[PATCH] D101239: [clang-tidy] Enable the use of IgnoreArray flag in pro-type-member-init rule

Hana Joo via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat May 8 14:02:04 PDT 2021


h-joo added a comment.

In D101239#2746147 <https://reviews.llvm.org/D101239#2746147>, @njames93 wrote:

> Whats the expected behaviour of sugar types. Can tests be added that demonstrate the behaviour.
> I'd argue these cases shouldn't be warned on if `IgnoreArrays` is enabled and If that behaviour isn't observed currently it should be addressed.
>
>   typedef int TypedefArray[4];
>   using UsingArray = int[4];
>   
>   struct HasArrayMember {
>     HasArrayMember() {}
>     UsingArray U; // Don't warn on this?
>     TypedefArray T; // Or this?
>     int Number; // Do warn this.
>   };

Added a test case to confirm that it works fine :)


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

https://reviews.llvm.org/D101239



More information about the cfe-commits mailing list