Author: d0k Date: Sat Jul 18 09:35:41 2015 New Revision: 242607 URL: http://llvm.org/viewvc/llvm-project?rev=242607&view=rev Log: [clang-modernize] Don't rely on iterator auto-dereferencing.
Modified: clang-tools-extra/trunk/clang-modernize/UseAuto/UseAutoActions.cpp Modified: clang-tools-extra/trunk/clang-modernize/UseAuto/UseAutoActions.cpp URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-modernize/UseAuto/UseAutoActions.cpp?rev=242607&r1=242606&r2=242607&view=diff ============================================================================== --- clang-tools-extra/trunk/clang-modernize/UseAuto/UseAutoActions.cpp (original) +++ clang-tools-extra/trunk/clang-modernize/UseAuto/UseAutoActions.cpp Sat Jul 18 09:35:41 2015 @@ -46,7 +46,7 @@ void IteratorReplacer::run(const MatchFi "Expected constructor with single argument"); // Drill down to the as-written initializer. - const Expr *E = Construct->arg_begin()->IgnoreParenImpCasts(); + const Expr *E = (*Construct->arg_begin())->IgnoreParenImpCasts(); if (E != E->IgnoreConversionOperator()) // We hit a conversion operator. Early-out now as they imply an implicit // conversion from a different type. Could also mean an explicit _______________________________________________ cfe-commits mailing list cfe-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits