[clang-tools-extra] r248489 - Solve comment on rL248418.

Angel Garcia Gomez via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 24 06:26:28 PDT 2015


Author: angelgarcia
Date: Thu Sep 24 08:26:28 2015
New Revision: 248489

URL: http://llvm.org/viewvc/llvm-project?rev=248489&view=rev
Log:
Solve comment on rL248418.

Summary: Solve comment on rL248418.

Reviewers: alexfh

Subscribers: cfe-commits, klimek

Differential Revision: http://reviews.llvm.org/D13129

Modified:
    clang-tools-extra/trunk/clang-tidy/modernize/LoopConvertCheck.cpp

Modified: clang-tools-extra/trunk/clang-tidy/modernize/LoopConvertCheck.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/modernize/LoopConvertCheck.cpp?rev=248489&r1=248488&r2=248489&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-tidy/modernize/LoopConvertCheck.cpp (original)
+++ clang-tools-extra/trunk/clang-tidy/modernize/LoopConvertCheck.cpp Thu Sep 24 08:26:28 2015
@@ -394,9 +394,7 @@ static bool containerIsConst(const Expr
     // If VDec is a reference to a container, Dereference is false,
     // but we still need to check the const-ness of the underlying container
     // type.
-    if (const auto &RT = CType->getAs<ReferenceType>()) {
-      CType = RT->getPointeeType();
-    }
+    CType = CType.getNonReferenceType();
     return CType.isConstQualified();
   }
   return false;




More information about the cfe-commits mailing list