store ImportProject errors in a vector instead of printing directly

ludviggunne · ludviggunne · commit fd5bc2513497 · 2025-11-02T22:10:22.000+01:00
diff --git a/cli/cmdlineparser.cpp b/cli/cmdlineparser.cpp
@@ -1191,14 +1191,17 @@ CmdLineParser::Result CmdLineParser::parseFromArgs(int argc, const char* const a
                 mSettings.libraries.emplace_back("windows");
             }
             if (projectType == ImportProject::Type::MISSING) {
+                project.printErrors();
                 mLogger.printError("failed to open project '" + projectFile + "'. The file does not exist.");
                 return Result::Fail;
             }
             if (projectType == ImportProject::Type::UNKNOWN) {
+                project.printErrors();
                 mLogger.printError("failed to load project '" + projectFile + "'. The format is unknown.");
                 return Result::Fail;
             }
             if (projectType == ImportProject::Type::FAILURE) {
+                project.printErrors();
                 mLogger.printError("failed to load project '" + projectFile + "'. An error occurred.");
                 return Result::Fail;
             }
diff --git a/lib/importproject.cpp b/lib/importproject.cpp
@@ -335,7 +335,7 @@ bool ImportProject::importCompileCommands(std::istream &istr)
     picojson::value compileCommands;
     istr >> compileCommands;
     if (!compileCommands.is<picojson::array>()) {
-        printError("compilation database is not a JSON array");
+        errors.emplace_back("compilation database is not a JSON array");
         return false;
     }
 
@@ -345,12 +345,12 @@ bool ImportProject::importCompileCommands(std::istream &istr)
         picojson::object obj = fileInfo.get<picojson::object>();
 
         if (obj.count("directory") == 0) {
-            printError("'directory' field in compilation database entry missing");
+            errors.emplace_back("'directory' field in compilation database entry missing");
             return false;
         }
 
         if (!obj["directory"].is<std::string>()) {
-            printError("'directory' field in compilation database entry is not a string");
+            errors.emplace_back("'directory' field in compilation database entry is not a string");
             return false;
         }
 
@@ -377,24 +377,24 @@ bool ImportProject::importCompileCommands(std::istream &istr)
                     }
                 }
             } else {
-                printError("'arguments' field in compilation database entry is not a JSON array");
+                errors.emplace_back("'arguments' field in compilation database entry is not a JSON array");
                 return false;
             }
         } else if (obj.count("command")) {
             doUnescape = true;
             if (obj["command"].is<std::string>()) {
                 command = obj["command"].get<std::string>();
             } else {
-                printError("'command' field in compilation database entry is not a string");
+                errors.emplace_back("'command' field in compilation database entry is not a string");
                 return false;
             }
         } else {
-            printError("no 'arguments' or 'command' field found in compilation database entry");
+            errors.emplace_back("no 'arguments' or 'command' field found in compilation database entry");
             return false;
         }
 
         if (!obj.count("file") || !obj["file"].is<std::string>()) {
-            printError("skip compilation database entry because it does not have a proper 'file' field");
+            errors.emplace_back("skip compilation database entry because it does not have a proper 'file' field");
             continue;
         }
 
@@ -434,14 +434,14 @@ bool ImportProject::importSln(std::istream &istr, const std::string &path, const
     std::string line;
 
     if (!std::getline(istr,line)) {
-        printError("Visual Studio solution file is empty");
+        errors.emplace_back("Visual Studio solution file is empty");
         return false;
     }
 
     if (!startsWith(line, "Microsoft Visual Studio Solution File")) {
         // Skip BOM
         if (!std::getline(istr, line) || !startsWith(line, "Microsoft Visual Studio Solution File")) {
-            printError("Visual Studio solution file header not found");
+            errors.emplace_back("Visual Studio solution file header not found");
             return false;
         }
     }
@@ -466,14 +466,14 @@ bool ImportProject::importSln(std::istream &istr, const std::string &path, const
             vcxproj = path + vcxproj;
         vcxproj = Path::fromNativeSeparators(std::move(vcxproj));
         if (!importVcxproj(vcxproj, variables, "", fileFilters, sharedItemsProjects)) {
-            printError("failed to load '" + vcxproj + "' from Visual Studio solution");
+            errors.emplace_back("failed to load '" + vcxproj + "' from Visual Studio solution");
             return false;
         }
         found = true;
     }
 
     if (!found) {
-        printError("no projects found in Visual Studio solution file");
+        errors.emplace_back("no projects found in Visual Studio solution file");
         return false;
     }
 
@@ -730,7 +730,7 @@ bool ImportProject::importVcxproj(const std::string &filename,
     tinyxml2::XMLDocument doc;
     const tinyxml2::XMLError error = doc.LoadFile(filename.c_str());
     if (error != tinyxml2::XML_SUCCESS) {
-        printError(std::string("Visual Studio project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));
+        errors.emplace_back(std::string("Visual Studio project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));
         return false;
     }
     return importVcxproj(filename, doc, variables, additionalIncludeDirectories, fileFilters, cache);
@@ -749,7 +749,7 @@ bool ImportProject::importVcxproj(const std::string &filename, const tinyxml2::X
 
     const tinyxml2::XMLElement * const rootnode = doc.FirstChildElement();
     if (rootnode == nullptr) {
-        printError("Visual Studio project file has no XML root node");
+        errors.emplace_back("Visual Studio project file has no XML root node");
         return false;
     }
     for (const tinyxml2::XMLElement *node = rootnode->FirstChildElement(); node; node = node->NextSiblingElement()) {
@@ -810,13 +810,13 @@ bool ImportProject::importVcxproj(const std::string &filename, const tinyxml2::X
                                 pathToSharedItemsFile = variables["ProjectDir"] + projectAttribute;
                             }
                             if (!simplifyPathWithVariables(pathToSharedItemsFile, variables)) {
-                                printError("Could not simplify path to referenced shared items project");
+                                errors.emplace_back("Could not simplify path to referenced shared items project");
                                 return false;
                             }
 
                             SharedItemsProject toAdd = importVcxitems(pathToSharedItemsFile, fileFilters, cache);
                             if (!toAdd.successful) {
-                                printError("Could not load shared items project \"" + pathToSharedItemsFile + "\" from original path \"" + std::string(projectAttribute) + "\".");
+                                errors.emplace_back("Could not load shared items project \"" + pathToSharedItemsFile + "\" from original path \"" + std::string(projectAttribute) + "\".");
                                 return false;
                             }
                             sharedItemsProjects.emplace_back(toAdd);
@@ -928,12 +928,12 @@ ImportProject::SharedItemsProject ImportProject::importVcxitems(const std::strin
     tinyxml2::XMLDocument doc;
     const tinyxml2::XMLError error = doc.LoadFile(filename.c_str());
     if (error != tinyxml2::XML_SUCCESS) {
-        printError(std::string("Visual Studio project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));
+        errors.emplace_back(std::string("Visual Studio project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));
         return result;
     }
     const tinyxml2::XMLElement * const rootnode = doc.FirstChildElement();
     if (rootnode == nullptr) {
-        printError("Visual Studio project file has no XML root node");
+        errors.emplace_back("Visual Studio project file has no XML root node");
         return result;
     }
     for (const tinyxml2::XMLElement *node = rootnode->FirstChildElement(); node; node = node->NextSiblingElement()) {
@@ -951,7 +951,7 @@ ImportProject::SharedItemsProject ImportProject::importVcxitems(const std::strin
 
                         result.sourceFiles.emplace_back(file);
                     } else {
-                        printError("Could not find shared items source file");
+                        errors.emplace_back("Could not find shared items source file");
                         return result;
                     }
                 }
@@ -979,12 +979,12 @@ bool ImportProject::importBcb6Prj(const std::string &projectFilename)
     tinyxml2::XMLDocument doc;
     const tinyxml2::XMLError error = doc.LoadFile(projectFilename.c_str());
     if (error != tinyxml2::XML_SUCCESS) {
-        printError(std::string("Borland project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));
+        errors.emplace_back(std::string("Borland project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));
         return false;
     }
     const tinyxml2::XMLElement * const rootnode = doc.FirstChildElement();
     if (rootnode == nullptr) {
-        printError("Borland project file has no XML root node");
+        errors.emplace_back("Borland project file has no XML root node");
         return false;
     }
 
@@ -1296,12 +1296,12 @@ bool ImportProject::importCppcheckGuiProject(std::istream &istr, Settings &setti
     const std::string xmldata = istream_to_string(istr);
     const tinyxml2::XMLError error = doc.Parse(xmldata.data(), xmldata.size());
     if (error != tinyxml2::XML_SUCCESS) {
-        printError(std::string("Cppcheck GUI project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));
+        errors.emplace_back(std::string("Cppcheck GUI project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));
         return false;
     }
     const tinyxml2::XMLElement * const rootnode = doc.FirstChildElement();
     if (rootnode == nullptr || strcmp(rootnode->Name(), CppcheckXml::ProjectElementName) != 0) {
-        printError("Cppcheck GUI project file has no XML root node");
+        errors.emplace_back("Cppcheck GUI project file has no XML root node");
         return false;
     }
 
@@ -1420,7 +1420,7 @@ bool ImportProject::importCppcheckGuiProject(std::istream &istr, Settings &setti
                 else if (strcmp(childname, Settings::SafeChecks::XmlExternalVariables) == 0)
                     temp.safeChecks.externalVariables = true;
                 else {
-                    printError("Unknown '" + std::string(Settings::SafeChecks::XmlRootName) + "' element '" + childname + "' in Cppcheck project file");
+                    errors.emplace_back("Unknown '" + std::string(Settings::SafeChecks::XmlRootName) + "' element '" + childname + "' in Cppcheck project file");
                     return false;
                 }
             }
@@ -1443,7 +1443,7 @@ bool ImportProject::importCppcheckGuiProject(std::istream &istr, Settings &setti
         else if (strcmp(name, CppcheckXml::ProjectNameElementName) == 0)
             ; // no-op
         else {
-            printError("Unknown element '" + std::string(name) + "' in Cppcheck project file");
+            errors.emplace_back("Unknown element '" + std::string(name) + "' in Cppcheck project file");
             return false;
         }
     }
@@ -1548,6 +1548,12 @@ void ImportProject::setRelativePaths(const std::string &filename)
     }
 }
 
+void ImportProject::printErrors()
+{
+    for (const auto &error : errors)
+        printError(error);
+}
+
 void ImportProject::printError(const std::string &message)
 {
     std::cout << "cppcheck: error: " << message << std::endl;
diff --git a/lib/importproject.h b/lib/importproject.h
@@ -74,6 +74,7 @@ class CPPCHECKLIB WARN_UNUSED ImportProject {
     static void fsSetIncludePaths(FileSettings& fs, const std::string &basepath, const std::list<std::string> &in, std::map<std::string, std::string, cppcheck::stricmp> &variables);
 
     std::list<FileSettings> fileSettings;
+    std::vector<std::string> errors;
 
     ImportProject() = default;
     virtual ~ImportProject() = default;
@@ -98,6 +99,8 @@ class CPPCHECKLIB WARN_UNUSED ImportProject {
     void ignorePaths(const std::vector<std::string> &ipaths, bool debug = false);
     void ignoreOtherConfigs(const std::string &cfg);
 
+    void printErrors();
+
     Type import(const std::string &filename, Settings *settings=nullptr, Suppressions *supprs=nullptr);
 protected:
     bool importCompileCommands(std::istream &istr);
@@ -112,7 +115,7 @@ class CPPCHECKLIB WARN_UNUSED ImportProject {
     };
 
     bool importSln(std::istream &istr, const std::string &path, const std::vector<std::string> &fileFilters);
-    static SharedItemsProject importVcxitems(const std::string &filename, const std::vector<std::string> &fileFilters, std::vector<SharedItemsProject> &cache);
+    SharedItemsProject importVcxitems(const std::string &filename, const std::vector<std::string> &fileFilters, std::vector<SharedItemsProject> &cache);
     bool importVcxproj(const std::string &filename, std::map<std::string, std::string, cppcheck::stricmp> &variables, const std::string &additionalIncludeDirectories, const std::vector<std::string> &fileFilters, std::vector<SharedItemsProject> &cache);
     bool importVcxproj(const std::string &filename, const tinyxml2::XMLDocument &doc, std::map<std::string, std::string, cppcheck::stricmp> &variables, const std::string &additionalIncludeDirectories, const std::vector<std::string> &fileFilters, std::vector<SharedItemsProject> &cache);
     bool importBcb6Prj(const std::string &projectFilename);

Original file line number	Diff line number	Diff line change
`@@ -1191,14 +1191,17 @@ CmdLineParser::Result CmdLineParser::parseFromArgs(int argc, const char* const a`
`1191`	`1191`	`mSettings.libraries.emplace_back("windows");`
`1192`	`1192`	`}`
`1193`	`1193`	`if (projectType == ImportProject::Type::MISSING) {`
	`1194`	`+ project.printErrors();`
`1194`	`1195`	`mLogger.printError("failed to open project '" + projectFile + "'. The file does not exist.");`
`1195`	`1196`	`return Result::Fail;`
`1196`	`1197`	`}`
`1197`	`1198`	`if (projectType == ImportProject::Type::UNKNOWN) {`
	`1199`	`+ project.printErrors();`
`1198`	`1200`	`mLogger.printError("failed to load project '" + projectFile + "'. The format is unknown.");`
`1199`	`1201`	`return Result::Fail;`
`1200`	`1202`	`}`
`1201`	`1203`	`if (projectType == ImportProject::Type::FAILURE) {`
	`1204`	`+ project.printErrors();`
`1202`	`1205`	`mLogger.printError("failed to load project '" + projectFile + "'. An error occurred.");`
`1203`	`1206`	`return Result::Fail;`
`1204`	`1207`	`}`
Original file line number	Diff line number	Diff line change
`@@ -335,7 +335,7 @@ bool ImportProject::importCompileCommands(std::istream &istr)`
`335`	`335`	`picojson::value compileCommands;`
`336`	`336`	`istr >> compileCommands;`
`337`	`337`	`if (!compileCommands.is<picojson::array>()) {`
`338`		`- printError("compilation database is not a JSON array");`
	`338`	`+ errors.emplace_back("compilation database is not a JSON array");`
`339`	`339`	`return false;`
`340`	`340`	`}`
`341`	`341`
`@@ -345,12 +345,12 @@ bool ImportProject::importCompileCommands(std::istream &istr)`
`345`	`345`	`picojson::object obj = fileInfo.get<picojson::object>();`
`346`	`346`
`347`	`347`	`if (obj.count("directory") == 0) {`
`348`		`- printError("'directory' field in compilation database entry missing");`
	`348`	`+ errors.emplace_back("'directory' field in compilation database entry missing");`
`349`	`349`	`return false;`
`350`	`350`	`}`
`351`	`351`
`352`	`352`	`if (!obj["directory"].is<std::string>()) {`
`353`		`- printError("'directory' field in compilation database entry is not a string");`
	`353`	`+ errors.emplace_back("'directory' field in compilation database entry is not a string");`
`354`	`354`	`return false;`
`355`	`355`	`}`
`356`	`356`
`@@ -377,24 +377,24 @@ bool ImportProject::importCompileCommands(std::istream &istr)`
`377`	`377`	`}`
`378`	`378`	`}`
`379`	`379`	`} else {`
`380`		`- printError("'arguments' field in compilation database entry is not a JSON array");`
	`380`	`+ errors.emplace_back("'arguments' field in compilation database entry is not a JSON array");`
`381`	`381`	`return false;`
`382`	`382`	`}`
`383`	`383`	`} else if (obj.count("command")) {`
`384`	`384`	`doUnescape = true;`
`385`	`385`	`if (obj["command"].is<std::string>()) {`
`386`	`386`	`command = obj["command"].get<std::string>();`
`387`	`387`	`} else {`
`388`		`- printError("'command' field in compilation database entry is not a string");`
	`388`	`+ errors.emplace_back("'command' field in compilation database entry is not a string");`
`389`	`389`	`return false;`
`390`	`390`	`}`
`391`	`391`	`} else {`
`392`		`- printError("no 'arguments' or 'command' field found in compilation database entry");`
	`392`	`+ errors.emplace_back("no 'arguments' or 'command' field found in compilation database entry");`
`393`	`393`	`return false;`
`394`	`394`	`}`
`395`	`395`
`396`	`396`	`if (!obj.count("file") \|\| !obj["file"].is<std::string>()) {`
`397`		`- printError("skip compilation database entry because it does not have a proper 'file' field");`
	`397`	`+ errors.emplace_back("skip compilation database entry because it does not have a proper 'file' field");`
`398`	`398`	`continue;`
`399`	`399`	`}`
`400`	`400`
`@@ -434,14 +434,14 @@ bool ImportProject::importSln(std::istream &istr, const std::string &path, const`
`434`	`434`	`std::string line;`
`435`	`435`
`436`	`436`	`if (!std::getline(istr,line)) {`
`437`		`- printError("Visual Studio solution file is empty");`
	`437`	`+ errors.emplace_back("Visual Studio solution file is empty");`
`438`	`438`	`return false;`
`439`	`439`	`}`
`440`	`440`
`441`	`441`	`if (!startsWith(line, "Microsoft Visual Studio Solution File")) {`
`442`	`442`	`// Skip BOM`
`443`	`443`	`if (!std::getline(istr, line) \|\| !startsWith(line, "Microsoft Visual Studio Solution File")) {`
`444`		`- printError("Visual Studio solution file header not found");`
	`444`	`+ errors.emplace_back("Visual Studio solution file header not found");`
`445`	`445`	`return false;`
`446`	`446`	`}`
`447`	`447`	`}`
`@@ -466,14 +466,14 @@ bool ImportProject::importSln(std::istream &istr, const std::string &path, const`
`466`	`466`	`vcxproj = path + vcxproj;`
`467`	`467`	`vcxproj = Path::fromNativeSeparators(std::move(vcxproj));`
`468`	`468`	`if (!importVcxproj(vcxproj, variables, "", fileFilters, sharedItemsProjects)) {`
`469`		`- printError("failed to load '" + vcxproj + "' from Visual Studio solution");`
	`469`	`+ errors.emplace_back("failed to load '" + vcxproj + "' from Visual Studio solution");`
`470`	`470`	`return false;`
`471`	`471`	`}`
`472`	`472`	`found = true;`
`473`	`473`	`}`
`474`	`474`
`475`	`475`	`if (!found) {`
`476`		`- printError("no projects found in Visual Studio solution file");`
	`476`	`+ errors.emplace_back("no projects found in Visual Studio solution file");`
`477`	`477`	`return false;`
`478`	`478`	`}`
`479`	`479`
`@@ -730,7 +730,7 @@ bool ImportProject::importVcxproj(const std::string &filename,`
`730`	`730`	`tinyxml2::XMLDocument doc;`
`731`	`731`	`const tinyxml2::XMLError error = doc.LoadFile(filename.c_str());`
`732`	`732`	`if (error != tinyxml2::XML_SUCCESS) {`
`733`		`- printError(std::string("Visual Studio project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));`
	`733`	`+ errors.emplace_back(std::string("Visual Studio project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));`
`734`	`734`	`return false;`
`735`	`735`	`}`
`736`	`736`	`return importVcxproj(filename, doc, variables, additionalIncludeDirectories, fileFilters, cache);`
`@@ -749,7 +749,7 @@ bool ImportProject::importVcxproj(const std::string &filename, const tinyxml2::X`
`749`	`749`
`750`	`750`	`const tinyxml2::XMLElement * const rootnode = doc.FirstChildElement();`
`751`	`751`	`if (rootnode == nullptr) {`
`752`		`- printError("Visual Studio project file has no XML root node");`
	`752`	`+ errors.emplace_back("Visual Studio project file has no XML root node");`
`753`	`753`	`return false;`
`754`	`754`	`}`
`755`	`755`	`for (const tinyxml2::XMLElement *node = rootnode->FirstChildElement(); node; node = node->NextSiblingElement()) {`
`@@ -810,13 +810,13 @@ bool ImportProject::importVcxproj(const std::string &filename, const tinyxml2::X`
`810`	`810`	`pathToSharedItemsFile = variables["ProjectDir"] + projectAttribute;`
`811`	`811`	`}`
`812`	`812`	`if (!simplifyPathWithVariables(pathToSharedItemsFile, variables)) {`
`813`		`- printError("Could not simplify path to referenced shared items project");`
	`813`	`+ errors.emplace_back("Could not simplify path to referenced shared items project");`
`814`	`814`	`return false;`
`815`	`815`	`}`
`816`	`816`
`817`	`817`	`SharedItemsProject toAdd = importVcxitems(pathToSharedItemsFile, fileFilters, cache);`
`818`	`818`	`if (!toAdd.successful) {`
`819`		`- printError("Could not load shared items project \"" + pathToSharedItemsFile + "\" from original path \"" + std::string(projectAttribute) + "\".");`
	`819`	`+ errors.emplace_back("Could not load shared items project \"" + pathToSharedItemsFile + "\" from original path \"" + std::string(projectAttribute) + "\".");`
`820`	`820`	`return false;`
`821`	`821`	`}`
`822`	`822`	`sharedItemsProjects.emplace_back(toAdd);`
`@@ -928,12 +928,12 @@ ImportProject::SharedItemsProject ImportProject::importVcxitems(const std::strin`
`928`	`928`	`tinyxml2::XMLDocument doc;`
`929`	`929`	`const tinyxml2::XMLError error = doc.LoadFile(filename.c_str());`
`930`	`930`	`if (error != tinyxml2::XML_SUCCESS) {`
`931`		`- printError(std::string("Visual Studio project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));`
	`931`	`+ errors.emplace_back(std::string("Visual Studio project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));`
`932`	`932`	`return result;`
`933`	`933`	`}`
`934`	`934`	`const tinyxml2::XMLElement * const rootnode = doc.FirstChildElement();`
`935`	`935`	`if (rootnode == nullptr) {`
`936`		`- printError("Visual Studio project file has no XML root node");`
	`936`	`+ errors.emplace_back("Visual Studio project file has no XML root node");`
`937`	`937`	`return result;`
`938`	`938`	`}`
`939`	`939`	`for (const tinyxml2::XMLElement *node = rootnode->FirstChildElement(); node; node = node->NextSiblingElement()) {`
`@@ -951,7 +951,7 @@ ImportProject::SharedItemsProject ImportProject::importVcxitems(const std::strin`
`951`	`951`
`952`	`952`	`result.sourceFiles.emplace_back(file);`
`953`	`953`	`} else {`
`954`		`- printError("Could not find shared items source file");`
	`954`	`+ errors.emplace_back("Could not find shared items source file");`
`955`	`955`	`return result;`
`956`	`956`	`}`
`957`	`957`	`}`
`@@ -979,12 +979,12 @@ bool ImportProject::importBcb6Prj(const std::string &projectFilename)`
`979`	`979`	`tinyxml2::XMLDocument doc;`
`980`	`980`	`const tinyxml2::XMLError error = doc.LoadFile(projectFilename.c_str());`
`981`	`981`	`if (error != tinyxml2::XML_SUCCESS) {`
`982`		`- printError(std::string("Borland project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));`
	`982`	`+ errors.emplace_back(std::string("Borland project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));`
`983`	`983`	`return false;`
`984`	`984`	`}`
`985`	`985`	`const tinyxml2::XMLElement * const rootnode = doc.FirstChildElement();`
`986`	`986`	`if (rootnode == nullptr) {`
`987`		`- printError("Borland project file has no XML root node");`
	`987`	`+ errors.emplace_back("Borland project file has no XML root node");`
`988`	`988`	`return false;`
`989`	`989`	`}`
`990`	`990`
`@@ -1296,12 +1296,12 @@ bool ImportProject::importCppcheckGuiProject(std::istream &istr, Settings &setti`
`1296`	`1296`	`const std::string xmldata = istream_to_string(istr);`
`1297`	`1297`	`const tinyxml2::XMLError error = doc.Parse(xmldata.data(), xmldata.size());`
`1298`	`1298`	`if (error != tinyxml2::XML_SUCCESS) {`
`1299`		`- printError(std::string("Cppcheck GUI project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));`
	`1299`	`+ errors.emplace_back(std::string("Cppcheck GUI project file is not a valid XML - ") + tinyxml2::XMLDocument::ErrorIDToName(error));`
`1300`	`1300`	`return false;`
`1301`	`1301`	`}`
`1302`	`1302`	`const tinyxml2::XMLElement * const rootnode = doc.FirstChildElement();`
`1303`	`1303`	`if (rootnode == nullptr \|\| strcmp(rootnode->Name(), CppcheckXml::ProjectElementName) != 0) {`
`1304`		`- printError("Cppcheck GUI project file has no XML root node");`
	`1304`	`+ errors.emplace_back("Cppcheck GUI project file has no XML root node");`
`1305`	`1305`	`return false;`
`1306`	`1306`	`}`
`1307`	`1307`
`@@ -1420,7 +1420,7 @@ bool ImportProject::importCppcheckGuiProject(std::istream &istr, Settings &setti`
`1420`	`1420`	`else if (strcmp(childname, Settings::SafeChecks::XmlExternalVariables) == 0)`
`1421`	`1421`	`temp.safeChecks.externalVariables = true;`
`1422`	`1422`	`else {`
`1423`		`- printError("Unknown '" + std::string(Settings::SafeChecks::XmlRootName) + "' element '" + childname + "' in Cppcheck project file");`
	`1423`	`+ errors.emplace_back("Unknown '" + std::string(Settings::SafeChecks::XmlRootName) + "' element '" + childname + "' in Cppcheck project file");`
`1424`	`1424`	`return false;`
`1425`	`1425`	`}`
`1426`	`1426`	`}`
`@@ -1443,7 +1443,7 @@ bool ImportProject::importCppcheckGuiProject(std::istream &istr, Settings &setti`
`1443`	`1443`	`else if (strcmp(name, CppcheckXml::ProjectNameElementName) == 0)`
`1444`	`1444`	`; // no-op`
`1445`	`1445`	`else {`
`1446`		`- printError("Unknown element '" + std::string(name) + "' in Cppcheck project file");`
	`1446`	`+ errors.emplace_back("Unknown element '" + std::string(name) + "' in Cppcheck project file");`
`1447`	`1447`	`return false;`
`1448`	`1448`	`}`
`1449`	`1449`	`}`
`@@ -1548,6 +1548,12 @@ void ImportProject::setRelativePaths(const std::string &filename)`
`1548`	`1548`	`}`
`1549`	`1549`	`}`
`1550`	`1550`
	`1551`	`+void ImportProject::printErrors()`
	`1552`	`+{`
	`1553`	`+ for (const auto &error : errors)`
	`1554`	`+ printError(error);`
	`1555`	`+}`
	`1556`	`+`
`1551`	`1557`	`void ImportProject::printError(const std::string &message)`
`1552`	`1558`	`{`
`1553`	`1559`	`std::cout << "cppcheck: error: " << message << std::endl;`