[PATCH] D20917: [clang-tidy] modernize-use-auto: don't remove stars by default

Piotr Padlewski via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 3 12:43:29 PDT 2016


Prazek added a subscriber: Prazek.
Prazek added a comment.

besides it lgtm


================
Comment at: clang-tidy/modernize/UseAutoCheck.cpp:338
@@ -329,8 +337,3 @@
 
-    // Remove explicitly written '*' from declarations where there's more than
-    // one declaration in the declaration list.
-    if (Dec == *D->decl_begin())
-      continue;
-
     // All subsequent declarations should match the same non-decorated type.
     if (FirstDeclType != V->getType().getCanonicalType())
----------------
Can you change this comment to make it more clear, or tell me what it do? I don't quite understand it right now


http://reviews.llvm.org/D20917





More information about the cfe-commits mailing list