[PATCH] D25316: [clang-tidy] Enhance modernize-use-auto to casts
Malcolm Parsons via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 10 09:34:50 PDT 2016
malcolm.parsons marked 15 inline comments as done.
malcolm.parsons added inline comments.
Comment at: test/clang-tidy/modernize-use-auto-cast.cpp:14
+ long l = 1;
+ int i1 = static_cast<int>(l);
+ // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: use auto when initializing with a cast to avoid duplicating the type name
> How do you handle cases like
> long l = static_cast<int>(i1);
> I would expect it to produce warning, but not produce fixit (because it might change the behavior of program), but I still would like to get information about it because it seems like a bug.
The check requires the VarDecl and Initializer to have the same type, so there is no warning.
I don't think this check should warn about it.
More information about the cfe-commits