[clang-tools-extra] r288034 - ClangMoveTests.cpp: Fix a bogus comparison of iterator.

NAKAMURA Takumi via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 28 06:27:38 PST 2016


Author: chapuni
Date: Mon Nov 28 08:27:37 2016
New Revision: 288034

URL: http://llvm.org/viewvc/llvm-project?rev=288034&view=rev
Log:
ClangMoveTests.cpp: Fix a bogus comparison of iterator.

msc Debug build detected it.

Modified:
    clang-tools-extra/trunk/unittests/clang-move/ClangMoveTests.cpp

Modified: clang-tools-extra/trunk/unittests/clang-move/ClangMoveTests.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/clang-move/ClangMoveTests.cpp?rev=288034&r1=288033&r2=288034&view=diff
==============================================================================
--- clang-tools-extra/trunk/unittests/clang-move/ClangMoveTests.cpp (original)
+++ clang-tools-extra/trunk/unittests/clang-move/ClangMoveTests.cpp Mon Nov 28 08:27:37 2016
@@ -521,7 +521,8 @@ TEST(ClangMove, DumpDecls) {
   const auto& Results = Reporter.getDeclarationList();
   auto ActualDeclIter = Results.begin();
   auto ExpectedDeclIter = ExpectedDeclarations.begin();
-  while (ActualDeclIter != Results.end() && ExpectedDeclIter != Results.end()) {
+  while (ActualDeclIter != Results.end() &&
+         ExpectedDeclIter != ExpectedDeclarations.end()) {
     EXPECT_EQ(*ActualDeclIter, *ExpectedDeclIter);
     ++ActualDeclIter;
     ++ExpectedDeclIter;




More information about the cfe-commits mailing list