2023-04-08 18:06:38 +02:00
|
|
|
/*
|
|
|
|
* Cppcheck - A tool for static C/C++ code analysis
|
|
|
|
* Copyright (C) 2007-2023 Cppcheck team.
|
|
|
|
*
|
|
|
|
* This program is free software: you can redistribute it and/or modify
|
|
|
|
* it under the terms of the GNU General Public License as published by
|
|
|
|
* the Free Software Foundation, either version 3 of the License, or
|
|
|
|
* (at your option) any later version.
|
|
|
|
*
|
|
|
|
* This program is distributed in the hope that it will be useful,
|
|
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
|
|
* GNU General Public License for more details
|
|
|
|
*
|
|
|
|
* You should have received a copy of the GNU General Public License
|
|
|
|
* along with this program. If not, see <http://www.gnu.org/licenses/>.
|
|
|
|
*/
|
|
|
|
|
|
|
|
#include "cppcheck.h"
|
2023-11-02 17:42:41 +01:00
|
|
|
#include "filesettings.h"
|
2023-04-08 18:06:38 +02:00
|
|
|
#include "fixture.h"
|
|
|
|
#include "helpers.h"
|
|
|
|
#include "redirect.h"
|
|
|
|
#include "settings.h"
|
|
|
|
#include "singleexecutor.h"
|
|
|
|
#include "timer.h"
|
|
|
|
|
|
|
|
#include <algorithm>
|
2023-10-09 10:07:20 +02:00
|
|
|
#include <cstdlib>
|
2023-08-02 10:36:17 +02:00
|
|
|
#include <list>
|
2023-04-08 18:06:38 +02:00
|
|
|
#include <memory>
|
|
|
|
#include <string>
|
|
|
|
#include <utility>
|
|
|
|
#include <vector>
|
|
|
|
|
2023-05-03 17:32:28 +02:00
|
|
|
class TestSingleExecutorBase : public TestFixture {
|
|
|
|
protected:
|
|
|
|
TestSingleExecutorBase(const char * const name, bool useFS) : TestFixture(name), useFS(useFS) {}
|
2023-04-08 18:06:38 +02:00
|
|
|
|
|
|
|
private:
|
2023-05-02 15:54:19 +02:00
|
|
|
Settings settings = settingsBuilder().library("std.cfg").build();
|
2023-05-03 17:32:28 +02:00
|
|
|
bool useFS;
|
|
|
|
|
|
|
|
std::string fprefix() const
|
|
|
|
{
|
|
|
|
if (useFS)
|
|
|
|
return "singlefs";
|
|
|
|
return "single";
|
|
|
|
}
|
2023-04-08 18:06:38 +02:00
|
|
|
|
|
|
|
static std::string zpad3(int i)
|
|
|
|
{
|
|
|
|
if (i < 10)
|
|
|
|
return "00" + std::to_string(i);
|
|
|
|
if (i < 100)
|
|
|
|
return "0" + std::to_string(i);
|
|
|
|
return std::to_string(i);
|
|
|
|
}
|
|
|
|
|
2023-08-04 13:56:18 +02:00
|
|
|
struct CheckOptions
|
|
|
|
{
|
2023-08-08 23:55:07 +02:00
|
|
|
CheckOptions() = default;
|
2023-10-05 19:04:06 +02:00
|
|
|
bool quiet = true;
|
2023-08-04 13:56:18 +02:00
|
|
|
SHOWTIME_MODES showtime = SHOWTIME_MODES::SHOWTIME_NONE;
|
|
|
|
const char* plistOutput = nullptr;
|
|
|
|
std::vector<std::string> filesList;
|
2023-10-08 09:06:02 +02:00
|
|
|
bool clangTidy = false;
|
2023-08-07 20:47:24 +02:00
|
|
|
bool executeCommandCalled = false;
|
|
|
|
std::string exe;
|
|
|
|
std::vector<std::string> args;
|
2023-08-04 13:56:18 +02:00
|
|
|
};
|
|
|
|
|
2023-08-08 23:55:07 +02:00
|
|
|
void check(int files, int result, const std::string &data, const CheckOptions& opt = make_default_obj{}) {
|
2023-04-08 18:06:38 +02:00
|
|
|
errout.str("");
|
2023-10-08 09:06:02 +02:00
|
|
|
|
2023-11-03 23:24:04 +01:00
|
|
|
std::list<FileSettings> fileSettings;
|
2023-04-08 18:06:38 +02:00
|
|
|
|
2023-11-07 21:21:24 +01:00
|
|
|
std::list<std::pair<std::string, std::size_t>> filelist;
|
2023-08-04 13:56:18 +02:00
|
|
|
if (opt.filesList.empty()) {
|
2023-04-08 18:06:38 +02:00
|
|
|
for (int i = 1; i <= files; ++i) {
|
2023-10-08 09:06:02 +02:00
|
|
|
std::string f_s = fprefix() + "_" + zpad3(i) + ".cpp";
|
2023-11-07 21:21:24 +01:00
|
|
|
filelist.emplace_back(f_s, data.size());
|
2023-05-03 17:32:28 +02:00
|
|
|
if (useFS) {
|
2023-11-02 17:42:41 +01:00
|
|
|
FileSettings fs;
|
2023-10-08 09:06:02 +02:00
|
|
|
fs.filename = std::move(f_s);
|
2023-11-03 23:24:04 +01:00
|
|
|
fileSettings.emplace_back(std::move(fs));
|
2023-05-03 17:32:28 +02:00
|
|
|
}
|
2023-04-08 18:06:38 +02:00
|
|
|
}
|
|
|
|
}
|
|
|
|
else {
|
2023-08-04 13:56:18 +02:00
|
|
|
for (const auto& f : opt.filesList)
|
2023-04-08 18:06:38 +02:00
|
|
|
{
|
2023-11-07 21:21:24 +01:00
|
|
|
filelist.emplace_back(f, data.size());
|
2023-05-03 17:32:28 +02:00
|
|
|
if (useFS) {
|
2023-11-02 17:42:41 +01:00
|
|
|
FileSettings fs;
|
2023-05-03 17:32:28 +02:00
|
|
|
fs.filename = f;
|
2023-11-03 23:24:04 +01:00
|
|
|
fileSettings.emplace_back(std::move(fs));
|
2023-05-03 17:32:28 +02:00
|
|
|
}
|
2023-04-08 18:06:38 +02:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2023-11-03 23:24:04 +01:00
|
|
|
Settings s = settings;
|
2023-10-08 09:06:02 +02:00
|
|
|
s.showtime = opt.showtime;
|
|
|
|
s.quiet = opt.quiet;
|
2023-08-04 13:56:18 +02:00
|
|
|
if (opt.plistOutput)
|
2023-10-08 09:06:02 +02:00
|
|
|
s.plistOutput = opt.plistOutput;
|
|
|
|
s.clangTidy = opt.clangTidy;
|
2023-08-07 20:47:24 +02:00
|
|
|
|
|
|
|
bool executeCommandCalled = false;
|
|
|
|
std::string exe;
|
|
|
|
std::vector<std::string> args;
|
2023-04-08 18:06:38 +02:00
|
|
|
// NOLINTNEXTLINE(performance-unnecessary-value-param)
|
2023-08-07 20:47:24 +02:00
|
|
|
CppCheck cppcheck(*this, true, [&executeCommandCalled, &exe, &args](std::string e,std::vector<std::string> a,std::string,std::string&){
|
|
|
|
executeCommandCalled = true;
|
|
|
|
exe = std::move(e);
|
|
|
|
args = std::move(a);
|
2023-09-20 10:40:57 +02:00
|
|
|
return EXIT_SUCCESS;
|
2023-04-08 18:06:38 +02:00
|
|
|
});
|
2023-10-08 09:06:02 +02:00
|
|
|
cppcheck.settings() = s;
|
2023-05-03 17:32:28 +02:00
|
|
|
|
2023-04-08 18:06:38 +02:00
|
|
|
std::vector<std::unique_ptr<ScopedFile>> scopedfiles;
|
2023-11-07 21:21:24 +01:00
|
|
|
scopedfiles.reserve(filelist.size());
|
|
|
|
for (std::list<std::pair<std::string, std::size_t>>::const_iterator i = filelist.cbegin(); i != filelist.cend(); ++i)
|
2023-04-08 18:06:38 +02:00
|
|
|
scopedfiles.emplace_back(new ScopedFile(i->first, data));
|
|
|
|
|
2023-05-03 17:32:28 +02:00
|
|
|
// clear files list so only fileSettings are used
|
|
|
|
if (useFS)
|
2023-11-07 21:21:24 +01:00
|
|
|
filelist.clear();
|
2023-05-03 17:32:28 +02:00
|
|
|
|
2023-11-07 21:21:24 +01:00
|
|
|
SingleExecutor executor(cppcheck, filelist, fileSettings, s, s.nomsg, *this);
|
2023-04-08 18:06:38 +02:00
|
|
|
ASSERT_EQUALS(result, executor.check());
|
2023-08-07 20:47:24 +02:00
|
|
|
ASSERT_EQUALS(opt.executeCommandCalled, executeCommandCalled);
|
|
|
|
ASSERT_EQUALS(opt.exe, exe);
|
|
|
|
ASSERT_EQUALS(opt.args.size(), args.size());
|
|
|
|
for (int i = 0; i < args.size(); ++i)
|
|
|
|
{
|
|
|
|
ASSERT_EQUALS(opt.args[i], args[i]);
|
|
|
|
}
|
2023-04-08 18:06:38 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
void run() override {
|
|
|
|
TEST_CASE(many_files);
|
|
|
|
TEST_CASE(many_files_showtime);
|
|
|
|
TEST_CASE(many_files_plist);
|
|
|
|
TEST_CASE(no_errors_more_files);
|
|
|
|
TEST_CASE(no_errors_less_files);
|
|
|
|
TEST_CASE(no_errors_equal_amount_files);
|
|
|
|
TEST_CASE(one_error_less_files);
|
|
|
|
TEST_CASE(one_error_several_files);
|
2023-08-07 20:47:24 +02:00
|
|
|
TEST_CASE(clangTidy);
|
2023-10-05 19:04:06 +02:00
|
|
|
TEST_CASE(showtime_top5_file);
|
|
|
|
TEST_CASE(showtime_top5_summary);
|
|
|
|
TEST_CASE(showtime_file);
|
|
|
|
TEST_CASE(showtime_summary);
|
|
|
|
TEST_CASE(showtime_file_total);
|
aligned and optimized unique error handling (#5280)
The handling in `CppCheck::reportErr()` and `Executor::hasToLog()` was
slightly different. I hope this can somehow be shared after the executor
reworking.
We were also using a very inappropriate container for the error list
which caused a lot of overhead.
`-D__GNUC__ --debug-warnings --template=daca2 --check-library -j2
../test/testsymboldatabase.cpp`
Clang 15
main process `284,218,587` -> `175,691,241`
worker process `9,123,697,183` -> `8,951,903,360`
2023-12-17 21:59:06 +01:00
|
|
|
TEST_CASE(suppress_error_library);
|
|
|
|
TEST_CASE(unique_errors);
|
2023-04-08 18:06:38 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
void many_files() {
|
|
|
|
check(100, 100,
|
|
|
|
"int main()\n"
|
|
|
|
"{\n"
|
2023-10-17 20:24:37 +02:00
|
|
|
" int i = *((int*)0);\n"
|
2023-04-08 18:06:38 +02:00
|
|
|
" return 0;\n"
|
2023-10-05 19:04:06 +02:00
|
|
|
"}", dinit(CheckOptions,
|
|
|
|
$.quiet = false));
|
2023-04-08 18:06:38 +02:00
|
|
|
std::string expected;
|
|
|
|
for (int i = 1; i <= 100; ++i) {
|
2023-05-03 17:32:28 +02:00
|
|
|
expected += "Checking " + fprefix() + "_" + zpad3(i) + ".cpp ...\n";
|
2023-04-08 18:06:38 +02:00
|
|
|
expected += std::to_string(i) + "/100 files checked " + std::to_string(i) + "% done\n";
|
|
|
|
}
|
2023-12-06 22:01:00 +01:00
|
|
|
ASSERT_EQUALS(expected, output_str());
|
2023-04-08 18:06:38 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
void many_files_showtime() {
|
|
|
|
SUPPRESS;
|
|
|
|
check(100, 100,
|
|
|
|
"int main()\n"
|
|
|
|
"{\n"
|
2023-10-17 20:24:37 +02:00
|
|
|
" int i = *((int*)0);\n"
|
2023-04-08 18:06:38 +02:00
|
|
|
" return 0;\n"
|
2023-08-04 13:56:18 +02:00
|
|
|
"}", dinit(CheckOptions, $.showtime = SHOWTIME_MODES::SHOWTIME_SUMMARY));
|
2023-04-08 18:06:38 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
void many_files_plist() {
|
2023-08-22 15:25:28 +02:00
|
|
|
const std::string plistOutput = "plist_" + fprefix() + "/";
|
|
|
|
ScopedFile plistFile("dummy", "", plistOutput);
|
2023-04-08 18:06:38 +02:00
|
|
|
|
|
|
|
check(100, 100,
|
|
|
|
"int main()\n"
|
|
|
|
"{\n"
|
2023-10-17 20:24:37 +02:00
|
|
|
" int i = *((int*)0);\n"
|
2023-04-08 18:06:38 +02:00
|
|
|
" return 0;\n"
|
2023-08-22 15:25:28 +02:00
|
|
|
"}", dinit(CheckOptions, $.plistOutput = plistOutput.c_str()));
|
2023-04-08 18:06:38 +02:00
|
|
|
}
|
|
|
|
|
|
|
|
void no_errors_more_files() {
|
|
|
|
check(3, 0,
|
|
|
|
"int main()\n"
|
|
|
|
"{\n"
|
|
|
|
" return 0;\n"
|
|
|
|
"}");
|
|
|
|
}
|
|
|
|
|
|
|
|
void no_errors_less_files() {
|
|
|
|
check(1, 0,
|
|
|
|
"int main()\n"
|
|
|
|
"{\n"
|
|
|
|
" return 0;\n"
|
|
|
|
"}");
|
|
|
|
}
|
|
|
|
|
|
|
|
void no_errors_equal_amount_files() {
|
|
|
|
check(2, 0,
|
|
|
|
"int main()\n"
|
|
|
|
"{\n"
|
|
|
|
" return 0;\n"
|
|
|
|
"}");
|
|
|
|
}
|
|
|
|
|
|
|
|
void one_error_less_files() {
|
|
|
|
check(1, 1,
|
|
|
|
"int main()\n"
|
|
|
|
"{\n"
|
2023-10-17 20:24:37 +02:00
|
|
|
" {int i = *((int*)0);}\n"
|
2023-04-08 18:06:38 +02:00
|
|
|
" return 0;\n"
|
|
|
|
"}");
|
|
|
|
}
|
|
|
|
|
|
|
|
void one_error_several_files() {
|
|
|
|
check(20, 20,
|
|
|
|
"int main()\n"
|
|
|
|
"{\n"
|
2023-10-17 20:24:37 +02:00
|
|
|
" {int i = *((int*)0);}\n"
|
2023-04-08 18:06:38 +02:00
|
|
|
" return 0;\n"
|
|
|
|
"}");
|
|
|
|
}
|
|
|
|
|
2023-08-07 20:47:24 +02:00
|
|
|
void clangTidy() {
|
|
|
|
// TODO: we currently only invoke it with ImportProject::FileSettings
|
|
|
|
if (!useFS)
|
|
|
|
return;
|
|
|
|
|
|
|
|
#ifdef _WIN32
|
2023-11-03 09:55:44 +01:00
|
|
|
constexpr char exe[] = "clang-tidy.exe";
|
2023-08-07 20:47:24 +02:00
|
|
|
#else
|
2023-11-03 09:55:44 +01:00
|
|
|
constexpr char exe[] = "clang-tidy";
|
2023-08-07 20:47:24 +02:00
|
|
|
#endif
|
|
|
|
|
|
|
|
const std::string file = fprefix() + "_001.cpp";
|
|
|
|
check(1, 0,
|
|
|
|
"int main()\n"
|
|
|
|
"{\n"
|
|
|
|
" return 0;\n"
|
|
|
|
"}",
|
|
|
|
dinit(CheckOptions,
|
2023-10-05 19:04:06 +02:00
|
|
|
$.quiet = false,
|
2023-10-08 09:06:02 +02:00
|
|
|
$.clangTidy = true,
|
2023-10-05 19:04:06 +02:00
|
|
|
$.executeCommandCalled = true,
|
2023-08-07 20:47:24 +02:00
|
|
|
$.exe = exe,
|
|
|
|
$.args = {"-quiet", "-checks=*,-clang-analyzer-*,-llvm*", file, "--"}));
|
2023-12-06 22:01:00 +01:00
|
|
|
ASSERT_EQUALS("Checking " + file + " ...\n", output_str());
|
2023-08-07 20:47:24 +02:00
|
|
|
}
|
|
|
|
|
2023-10-05 19:04:06 +02:00
|
|
|
// TODO: provide data which actually shows values above 0
|
|
|
|
|
|
|
|
void showtime_top5_file() {
|
|
|
|
REDIRECT;
|
|
|
|
check(2, 0,
|
|
|
|
"int main() {}",
|
|
|
|
dinit(CheckOptions,
|
|
|
|
$.showtime = SHOWTIME_MODES::SHOWTIME_TOP5_FILE));
|
|
|
|
const std::string output_s = GET_REDIRECT_OUTPUT;
|
|
|
|
// for each file: top5 results + overall + empty line
|
|
|
|
ASSERT_EQUALS((5 + 1 + 1) * 2, cppcheck::count_all_of(output_s, '\n'));
|
|
|
|
}
|
|
|
|
|
|
|
|
void showtime_top5_summary() {
|
|
|
|
REDIRECT;
|
|
|
|
check(2, 0,
|
|
|
|
"int main() {}",
|
|
|
|
dinit(CheckOptions,
|
|
|
|
$.showtime = SHOWTIME_MODES::SHOWTIME_TOP5_SUMMARY));
|
|
|
|
const std::string output_s = GET_REDIRECT_OUTPUT;
|
|
|
|
// once: top5 results + overall + empty line
|
|
|
|
ASSERT_EQUALS(5 + 1 + 1, cppcheck::count_all_of(output_s, '\n'));
|
|
|
|
// should only report the top5 once
|
|
|
|
ASSERT(output_s.find("1 result(s)") == std::string::npos);
|
|
|
|
ASSERT(output_s.find("2 result(s)") != std::string::npos);
|
|
|
|
}
|
|
|
|
|
|
|
|
void showtime_file() {
|
|
|
|
REDIRECT;
|
|
|
|
check(2, 0,
|
|
|
|
"int main() {}",
|
|
|
|
dinit(CheckOptions,
|
|
|
|
$.showtime = SHOWTIME_MODES::SHOWTIME_FILE));
|
|
|
|
const std::string output_s = GET_REDIRECT_OUTPUT;
|
|
|
|
ASSERT_EQUALS(2, cppcheck::count_all_of(output_s, "Overall time:"));
|
|
|
|
}
|
|
|
|
|
|
|
|
void showtime_summary() {
|
|
|
|
REDIRECT;
|
|
|
|
check(2, 0,
|
|
|
|
"int main() {}",
|
|
|
|
dinit(CheckOptions,
|
|
|
|
$.showtime = SHOWTIME_MODES::SHOWTIME_SUMMARY));
|
|
|
|
const std::string output_s = GET_REDIRECT_OUTPUT;
|
|
|
|
// should only report the actual summary once
|
|
|
|
ASSERT(output_s.find("1 result(s)") == std::string::npos);
|
|
|
|
ASSERT(output_s.find("2 result(s)") != std::string::npos);
|
|
|
|
}
|
|
|
|
|
|
|
|
void showtime_file_total() {
|
|
|
|
REDIRECT;
|
|
|
|
check(2, 0,
|
|
|
|
"int main() {}",
|
|
|
|
dinit(CheckOptions,
|
|
|
|
$.showtime = SHOWTIME_MODES::SHOWTIME_FILE_TOTAL));
|
|
|
|
const std::string output_s = GET_REDIRECT_OUTPUT;
|
|
|
|
ASSERT(output_s.find("Check time: " + fprefix() + "_" + zpad3(1) + ".cpp: ") != std::string::npos);
|
|
|
|
ASSERT(output_s.find("Check time: " + fprefix() + "_" + zpad3(2) + ".cpp: ") != std::string::npos);
|
|
|
|
}
|
|
|
|
|
aligned and optimized unique error handling (#5280)
The handling in `CppCheck::reportErr()` and `Executor::hasToLog()` was
slightly different. I hope this can somehow be shared after the executor
reworking.
We were also using a very inappropriate container for the error list
which caused a lot of overhead.
`-D__GNUC__ --debug-warnings --template=daca2 --check-library -j2
../test/testsymboldatabase.cpp`
Clang 15
main process `284,218,587` -> `175,691,241`
worker process `9,123,697,183` -> `8,951,903,360`
2023-12-17 21:59:06 +01:00
|
|
|
void suppress_error_library() {
|
|
|
|
SUPPRESS;
|
|
|
|
const Settings settingsOld = settings;
|
|
|
|
const char xmldata[] = R"(<def format="2"><markup ext=".cpp" reporterrors="false"/></def>)";
|
|
|
|
settings = settingsBuilder().libraryxml(xmldata, sizeof(xmldata)).build();
|
|
|
|
check(1, 0,
|
|
|
|
"int main()\n"
|
|
|
|
"{\n"
|
|
|
|
" int i = *((int*)0);\n"
|
|
|
|
" return 0;\n"
|
|
|
|
"}");
|
|
|
|
ASSERT_EQUALS("", errout.str());
|
|
|
|
settings = settingsOld;
|
|
|
|
}
|
|
|
|
|
|
|
|
void unique_errors() {
|
|
|
|
SUPPRESS;
|
|
|
|
ScopedFile inc_h(fprefix() + ".h",
|
|
|
|
"inline void f()\n"
|
|
|
|
"{\n"
|
|
|
|
" (void)*((int*)0);\n"
|
|
|
|
"}");
|
|
|
|
check(2, 2,
|
|
|
|
"#include \"" + inc_h.name() + "\"");
|
|
|
|
// these are not actually made unique by the implementation. That needs to be done by the given ErrorLogger
|
|
|
|
ASSERT_EQUALS("[" + inc_h.name() + ":3]: (error) Null pointer dereference: (int*)0\n", errout.str());
|
|
|
|
}
|
|
|
|
|
2023-04-08 18:06:38 +02:00
|
|
|
// TODO: test whole program analysis
|
aligned and optimized unique error handling (#5280)
The handling in `CppCheck::reportErr()` and `Executor::hasToLog()` was
slightly different. I hope this can somehow be shared after the executor
reworking.
We were also using a very inappropriate container for the error list
which caused a lot of overhead.
`-D__GNUC__ --debug-warnings --template=daca2 --check-library -j2
../test/testsymboldatabase.cpp`
Clang 15
main process `284,218,587` -> `175,691,241`
worker process `9,123,697,183` -> `8,951,903,360`
2023-12-17 21:59:06 +01:00
|
|
|
// TODO: test unique errors
|
2023-04-08 18:06:38 +02:00
|
|
|
};
|
|
|
|
|
2023-05-03 17:32:28 +02:00
|
|
|
class TestSingleExecutorFiles : public TestSingleExecutorBase {
|
|
|
|
public:
|
|
|
|
TestSingleExecutorFiles() : TestSingleExecutorBase("TestSingleExecutorFiles", false) {}
|
|
|
|
};
|
|
|
|
|
|
|
|
class TestSingleExecutorFS : public TestSingleExecutorBase {
|
|
|
|
public:
|
|
|
|
TestSingleExecutorFS() : TestSingleExecutorBase("TestSingleExecutorFS", true) {}
|
|
|
|
};
|
|
|
|
|
|
|
|
REGISTER_TEST(TestSingleExecutorFiles)
|
|
|
|
REGISTER_TEST(TestSingleExecutorFS)
|