testrunner: removed unnecessary default constructors from test options (#7889)

firewave · web-flow · commit 59d9ba3190ba · 2025-10-15T08:26:42.000+02:00
diff --git a/test/testautovariables.cpp b/test/testautovariables.cpp
@@ -34,7 +34,6 @@ class TestAutoVariables : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool inconclusive = true;
         bool cpp = true;
     };
diff --git a/test/testbool.cpp b/test/testbool.cpp
@@ -78,7 +78,6 @@ class TestBool : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool cpp = true;
     };
 
diff --git a/test/testbufferoverrun.cpp b/test/testbufferoverrun.cpp
@@ -39,7 +39,6 @@ class TestBufferOverrun : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         const Settings* s = nullptr;
         bool cpp = true;
     };
diff --git a/test/testclass.cpp b/test/testclass.cpp
@@ -2633,7 +2633,6 @@ class TestClass : public TestFixture {
 
     struct CheckVirtualDestructorOptions
     {
-        CheckVirtualDestructorOptions() = default;
         bool inconclusive = false;
     };
 
@@ -3656,7 +3655,6 @@ class TestClass : public TestFixture {
 
     struct CheckConstOptions
     {
-        CheckConstOptions() = default;
         const Settings *s = nullptr;
         bool inconclusive = true;
     };
diff --git a/test/testcondition.cpp b/test/testcondition.cpp
@@ -130,7 +130,6 @@ class TestCondition : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         const Settings* s = nullptr;
         bool cpp = true;
         bool inconclusive = false;
diff --git a/test/testconstructors.cpp b/test/testconstructors.cpp
@@ -34,7 +34,6 @@ class TestConstructors : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool inconclusive = false;
         const Settings* s = nullptr;
     };
diff --git a/test/testexceptionsafety.cpp b/test/testexceptionsafety.cpp
@@ -61,7 +61,6 @@ class TestExceptionSafety : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool inconclusive = false;
         const Settings *s = nullptr;
     };
diff --git a/test/testfunctions.cpp b/test/testfunctions.cpp
@@ -116,7 +116,6 @@ class TestFunctions : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool cpp = true;
         const Settings* s = nullptr;
     };
diff --git a/test/testincompletestatement.cpp b/test/testincompletestatement.cpp
@@ -34,7 +34,6 @@ class TestIncompleteStatement : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool inconclusive = false;
         bool cpp = true;
     };
diff --git a/test/testio.cpp b/test/testio.cpp
@@ -86,7 +86,6 @@ class TestIO : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool inconclusive = false;
         bool portability = false;
         Platform::Type platform = Platform::Type::Unspecified;
diff --git a/test/testleakautovar.cpp b/test/testleakautovar.cpp
@@ -216,7 +216,6 @@ class TestLeakAutoVar : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool cpp = false;
         const Settings *s = nullptr;
     };
diff --git a/test/testnullpointer.cpp b/test/testnullpointer.cpp
@@ -180,7 +180,6 @@ class TestNullPointer : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool inconclusive = false;
         bool cpp = true;
         Standards::cstd_t cstd = Standards::CLatest;
diff --git a/test/testother.cpp b/test/testother.cpp
@@ -330,7 +330,6 @@ class TestOther : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool cpp = true;
         bool inconclusive = true;
         bool verbose = false;
@@ -365,7 +364,6 @@ class TestOther : public TestFixture {
 
     struct CheckPOptions
     {
-        CheckPOptions() = default;
         bool cpp = true;
     };
 
@@ -2265,7 +2263,6 @@ class TestOther : public TestFixture {
 
     struct CheckInvalidPointerCastOptions
     {
-        CheckInvalidPointerCastOptions() = default;
         bool portability = true;
         bool inconclusive = false;
     };
diff --git a/test/testprocessexecutor.cpp b/test/testprocessexecutor.cpp
@@ -51,7 +51,6 @@ class TestProcessExecutorBase : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool quiet = true;
         SHOWTIME_MODES showtime = SHOWTIME_MODES::SHOWTIME_NONE;
         const char* plistOutput = nullptr;
diff --git a/test/testsimplifytemplate.cpp b/test/testsimplifytemplate.cpp
@@ -321,7 +321,6 @@ class TestSimplifyTemplate : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool debugwarnings = false;
     };
 
diff --git a/test/testsimplifytokens.cpp b/test/testsimplifytokens.cpp
@@ -166,7 +166,6 @@ class TestSimplifyTokens : public TestFixture {
 
     struct TokOptions
     {
-        TokOptions() = default;
         bool cpp = true;
         Platform::Type type = Platform::Type::Native;
     };
@@ -184,7 +183,6 @@ class TestSimplifyTokens : public TestFixture {
 
     struct TokenizeAndStringifyOptions
     {
-        TokenizeAndStringifyOptions() = default;
         Platform::Type platform = Platform::Type::Native;
         bool cpp = true;
     };
diff --git a/test/testsimplifytypedef.cpp b/test/testsimplifytypedef.cpp
@@ -259,7 +259,6 @@ class TestSimplifyTypedef : public TestFixture {
 
     struct TokOptions
     {
-        TokOptions() = default;
         bool simplify = true;
         bool debugwarnings = true;
     };
diff --git a/test/testsimplifyusing.cpp b/test/testsimplifyusing.cpp
@@ -101,7 +101,6 @@ class TestSimplifyUsing : public TestFixture {
 
     struct TokOptions
     {
-        TokOptions() = default;
         Platform::Type type = Platform::Type::Native;
         bool debugwarnings = true;
         bool preprocess = false;
diff --git a/test/testsingleexecutor.cpp b/test/testsingleexecutor.cpp
@@ -60,7 +60,6 @@ class TestSingleExecutorBase : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool quiet = true;
         SHOWTIME_MODES showtime = SHOWTIME_MODES::SHOWTIME_NONE;
         const char* plistOutput = nullptr;
diff --git a/test/teststl.cpp b/test/teststl.cpp
@@ -180,7 +180,6 @@ class TestStl : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool inconclusive = false;
         Standards::cppstd_t cppstandard = Standards::CPPLatest;
     };
diff --git a/test/teststring.cpp b/test/teststring.cpp
@@ -62,7 +62,6 @@ class TestString : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool cpp = true;
     };
 
diff --git a/test/testthreadexecutor.cpp b/test/testthreadexecutor.cpp
@@ -51,7 +51,6 @@ class TestThreadExecutorBase : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         bool quiet = true;
         SHOWTIME_MODES showtime = SHOWTIME_MODES::SHOWTIME_NONE;
         const char* plistOutput = nullptr;
diff --git a/test/testtype.cpp b/test/testtype.cpp
@@ -48,7 +48,6 @@ class TestType : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         const Settings* settings = nullptr;
         Standards::cppstd_t standard = Standards::cppstd_t::CPP11;
     };
@@ -80,7 +79,6 @@ class TestType : public TestFixture {
 
     struct CheckPOptions
     {
-        CheckPOptions() = default;
         const Settings* settings = nullptr;
         bool cpp = true;
     };
diff --git a/test/testuninitvar.cpp b/test/testuninitvar.cpp
@@ -108,7 +108,6 @@ class TestUninitVar : public TestFixture {
 
     struct CheckUninitVarOptions
     {
-        CheckUninitVarOptions() = default;
         bool cpp = true;
         bool debugwarnings = false;
         const Settings *s = nullptr;
diff --git a/test/testunusedfunctions.cpp b/test/testunusedfunctions.cpp
@@ -90,7 +90,6 @@ class TestUnusedFunctions : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         Platform::Type platform = Platform::Type::Native;
         const Settings* s = nullptr;
         bool cpp = true;
diff --git a/test/testunusedprivfunc.cpp b/test/testunusedprivfunc.cpp
@@ -88,7 +88,6 @@ class TestUnusedPrivateFunction : public TestFixture {
 
     struct CheckOptions
     {
-        CheckOptions() = default;
         Platform::Type platform = Platform::Type::Native;
     };
 
diff --git a/test/testunusedvar.cpp b/test/testunusedvar.cpp
@@ -269,7 +269,6 @@ class TestUnusedVar : public TestFixture {
 
     struct FunctionVariableUsageOptions
     {
-        FunctionVariableUsageOptions() = default;
         bool cpp = true;
     };
 
@@ -286,7 +285,6 @@ class TestUnusedVar : public TestFixture {
 
     struct CheckStructMemberUsageOptions
     {
-        CheckStructMemberUsageOptions() = default;
         const std::list<Directive>* directives = nullptr;
         bool cpp = true;
     };
diff --git a/test/testvarid.cpp b/test/testvarid.cpp
@@ -260,7 +260,6 @@ class TestVarID : public TestFixture {
 
     struct TokenizeOptions
     {
-        TokenizeOptions() = default;
         bool cpp = true;
         const Settings *s = nullptr;
     };

Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,6 @@ class TestAutoVariables : public TestFixture {`
`34`	`34`
`35`	`35`	`struct CheckOptions`
`36`	`36`	`{`
`37`		`- CheckOptions() = default;`
`38`	`37`	`bool inconclusive = true;`
`39`	`38`	`bool cpp = true;`
`40`	`39`	`};`
Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,6 @@ class TestBool : public TestFixture {`
`78`	`78`
`79`	`79`	`struct CheckOptions`
`80`	`80`	`{`
`81`		`- CheckOptions() = default;`
`82`	`81`	`bool cpp = true;`
`83`	`82`	`};`
`84`	`83`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,6 @@ class TestBufferOverrun : public TestFixture {`
`39`	`39`
`40`	`40`	`struct CheckOptions`
`41`	`41`	`{`
`42`		`- CheckOptions() = default;`
`43`	`42`	`const Settings* s = nullptr;`
`44`	`43`	`bool cpp = true;`
`45`	`44`	`};`
Original file line number	Diff line number	Diff line change
`@@ -2633,7 +2633,6 @@ class TestClass : public TestFixture {`
`2633`	`2633`
`2634`	`2634`	`struct CheckVirtualDestructorOptions`
`2635`	`2635`	`{`
`2636`		`- CheckVirtualDestructorOptions() = default;`
`2637`	`2636`	`bool inconclusive = false;`
`2638`	`2637`	`};`
`2639`	`2638`
`@@ -3656,7 +3655,6 @@ class TestClass : public TestFixture {`
`3656`	`3655`
`3657`	`3656`	`struct CheckConstOptions`
`3658`	`3657`	`{`
`3659`		`- CheckConstOptions() = default;`
`3660`	`3658`	`const Settings *s = nullptr;`
`3661`	`3659`	`bool inconclusive = true;`
`3662`	`3660`	`};`
Original file line number	Diff line number	Diff line change
`@@ -130,7 +130,6 @@ class TestCondition : public TestFixture {`
`130`	`130`
`131`	`131`	`struct CheckOptions`
`132`	`132`	`{`
`133`		`- CheckOptions() = default;`
`134`	`133`	`const Settings* s = nullptr;`
`135`	`134`	`bool cpp = true;`
`136`	`135`	`bool inconclusive = false;`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,6 @@ class TestConstructors : public TestFixture {`
`34`	`34`
`35`	`35`	`struct CheckOptions`
`36`	`36`	`{`
`37`		`- CheckOptions() = default;`
`38`	`37`	`bool inconclusive = false;`
`39`	`38`	`const Settings* s = nullptr;`
`40`	`39`	`};`
Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,6 @@ class TestExceptionSafety : public TestFixture {`
`61`	`61`
`62`	`62`	`struct CheckOptions`
`63`	`63`	`{`
`64`		`- CheckOptions() = default;`
`65`	`64`	`bool inconclusive = false;`
`66`	`65`	`const Settings *s = nullptr;`
`67`	`66`	`};`
Original file line number	Diff line number	Diff line change
`@@ -116,7 +116,6 @@ class TestFunctions : public TestFixture {`
`116`	`116`
`117`	`117`	`struct CheckOptions`
`118`	`118`	`{`
`119`		`- CheckOptions() = default;`
`120`	`119`	`bool cpp = true;`
`121`	`120`	`const Settings* s = nullptr;`
`122`	`121`	`};`
Original file line number	Diff line number	Diff line change
`@@ -34,7 +34,6 @@ class TestIncompleteStatement : public TestFixture {`
`34`	`34`
`35`	`35`	`struct CheckOptions`
`36`	`36`	`{`
`37`		`- CheckOptions() = default;`
`38`	`37`	`bool inconclusive = false;`
`39`	`38`	`bool cpp = true;`
`40`	`39`	`};`
Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,6 @@ class TestIO : public TestFixture {`
`86`	`86`
`87`	`87`	`struct CheckOptions`
`88`	`88`	`{`
`89`		`- CheckOptions() = default;`
`90`	`89`	`bool inconclusive = false;`
`91`	`90`	`bool portability = false;`
`92`	`91`	`Platform::Type platform = Platform::Type::Unspecified;`