[clang-tools-extra] r358293 - [clangd] Fix an overflow inside a test

Kadir Cetinkaya via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 12 09:40:54 PDT 2019


Author: kadircet
Date: Fri Apr 12 09:40:54 2019
New Revision: 358293

URL: http://llvm.org/viewvc/llvm-project?rev=358293&view=rev
Log:
[clangd] Fix an overflow inside a test

Modified:
    clang-tools-extra/trunk/unittests/clangd/PrintASTTests.cpp

Modified: clang-tools-extra/trunk/unittests/clangd/PrintASTTests.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/clangd/PrintASTTests.cpp?rev=358293&r1=358292&r2=358293&view=diff
==============================================================================
--- clang-tools-extra/trunk/unittests/clangd/PrintASTTests.cpp (original)
+++ clang-tools-extra/trunk/unittests/clangd/PrintASTTests.cpp Fri Apr 12 09:40:54 2019
@@ -37,6 +37,8 @@ TEST_P(ASTUtils, PrintTemplateArgs) {
   struct Visitor : RecursiveASTVisitor<Visitor> {
     Visitor(std::vector<Position> Points) : Points(std::move(Points)) {}
     bool VisitNamedDecl(const NamedDecl *ND) {
+      if (TemplateArgsAtPoints.size() == Points.size())
+        return true;
       auto Pos = sourceLocToPosition(ND->getASTContext().getSourceManager(),
                                      ND->getLocation());
       if (Pos != Points[TemplateArgsAtPoints.size()])




More information about the cfe-commits mailing list