Skip to content

Commit 3fb83e9

Browse files
firewavefirewave
committed
made some local const Settings objects in test static 
1 parent f7d23b8 commit 3fb83e9

11 files changed

Lines changed: 27 additions & 27 deletions

‎test/testastutils.cpp‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ class TestAstUtils : public TestFixture {
4848

4949
#define findLambdaEndToken(code) findLambdaEndToken_(code, __FILE__, __LINE__)
5050
bool findLambdaEndToken_(const char code[], const char* file, int line) {
51-
const Settings settings;
51+
static const Settings settings;
5252
Tokenizer tokenizer(&settings, this);
5353
std::istringstream istr(code);
5454
ASSERT_LOC(tokenizer.tokenize(istr, "test.cpp"), file, line);
@@ -82,7 +82,7 @@ class TestAstUtils : public TestFixture {
8282

8383
#define findLambdaStartToken(code) findLambdaStartToken_(code, __FILE__, __LINE__)
8484
bool findLambdaStartToken_(const char code[], const char* file, int line) {
85-
const Settings settings;
85+
static const Settings settings;
8686
Tokenizer tokenizer(&settings, this);
8787
std::istringstream istr(code);
8888
ASSERT_LOC(tokenizer.tokenize(istr, "test.cpp"), file, line);
@@ -115,7 +115,7 @@ class TestAstUtils : public TestFixture {
115115

116116
#define isNullOperand(code) isNullOperand_(code, __FILE__, __LINE__)
117117
bool isNullOperand_(const char code[], const char* file, int line) {
118-
const Settings settings;
118+
static const Settings settings;
119119
Tokenizer tokenizer(&settings, this);
120120
std::istringstream istr(code);
121121
ASSERT_LOC(tokenizer.tokenize(istr, "test.cpp"), file, line);
@@ -137,7 +137,7 @@ class TestAstUtils : public TestFixture {
137137

138138
#define isReturnScope(code, offset) isReturnScope_(code, offset, __FILE__, __LINE__)
139139
bool isReturnScope_(const char code[], int offset, const char* file, int line) {
140-
const Settings settings;
140+
static const Settings settings;
141141
Tokenizer tokenizer(&settings, this);
142142
std::istringstream istr(code);
143143
ASSERT_LOC(tokenizer.tokenize(istr, "test.cpp"), file, line);
@@ -168,7 +168,7 @@ class TestAstUtils : public TestFixture {
168168

169169
#define isSameExpression(code, tokStr1, tokStr2) isSameExpression_(code, tokStr1, tokStr2, __FILE__, __LINE__)
170170
bool isSameExpression_(const char code[], const char tokStr1[], const char tokStr2[], const char* file, int line) {
171-
const Settings settings;
171+
static const Settings settings;
172172
Library library;
173173
Tokenizer tokenizer(&settings, this);
174174
std::istringstream istr(code);
@@ -208,7 +208,7 @@ class TestAstUtils : public TestFixture {
208208

209209
#define isVariableChanged(code, startPattern, endPattern) isVariableChanged_(code, startPattern, endPattern, __FILE__, __LINE__)
210210
bool isVariableChanged_(const char code[], const char startPattern[], const char endPattern[], const char* file, int line) {
211-
const Settings settings;
211+
static const Settings settings;
212212
Tokenizer tokenizer(&settings, this);
213213
std::istringstream istr(code);
214214
ASSERT_LOC(tokenizer.tokenize(istr, "test.cpp"), file, line);
@@ -231,7 +231,7 @@ class TestAstUtils : public TestFixture {
231231

232232
#define isVariableChangedByFunctionCall(code, pattern, inconclusive) isVariableChangedByFunctionCall_(code, pattern, inconclusive, __FILE__, __LINE__)
233233
bool isVariableChangedByFunctionCall_(const char code[], const char pattern[], bool *inconclusive, const char* file, int line) {
234-
const Settings settings;
234+
static const Settings settings;
235235
Tokenizer tokenizer(&settings, this);
236236
std::istringstream istr(code);
237237
ASSERT_LOC(tokenizer.tokenize(istr, "test.cpp"), file, line);
@@ -260,7 +260,7 @@ class TestAstUtils : public TestFixture {
260260

261261
#define nextAfterAstRightmostLeaf(code, parentPattern, rightPattern) nextAfterAstRightmostLeaf_(code, parentPattern, rightPattern, __FILE__, __LINE__)
262262
bool nextAfterAstRightmostLeaf_(const char code[], const char parentPattern[], const char rightPattern[], const char* file, int line) {
263-
const Settings settings;
263+
static const Settings settings;
264264
Tokenizer tokenizer(&settings, this);
265265
std::istringstream istr(code);
266266
ASSERT_LOC(tokenizer.tokenize(istr, "test.cpp"), file, line);
@@ -285,7 +285,7 @@ class TestAstUtils : public TestFixture {
285285
enum class Result {False, True, Fail};
286286

287287
Result isUsedAsBool(const char code[], const char pattern[]) {
288-
const Settings settings;
288+
static const Settings settings;
289289
Tokenizer tokenizer(&settings, this);
290290
std::istringstream istr(code);
291291
if (!tokenizer.tokenize(istr, "test.cpp"))

‎test/testbufferoverrun.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5479,7 +5479,7 @@ class TestBufferOverrun : public TestFixture {
54795479

54805480
void checkPipeParameterSize() { // #3521
54815481

5482-
const Settings settings = settingsBuilder().library("posix.cfg").build();
5482+
static const Settings settings = settingsBuilder().library("posix.cfg").build();
54835483

54845484
check("void f(){\n"
54855485
"int pipefd[1];\n" // <-- array of two integers is needed

‎test/testclangimport.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ class TestClangImport : public TestFixture {
137137
}
138138

139139
std::string parse(const char clang[]) {
140-
const Settings settings = settingsBuilder().clang().build();
140+
static const Settings settings = settingsBuilder().clang().build();
141141
Tokenizer tokenizer(&settings, this);
142142
std::istringstream istr(clang);
143143
clangimport::parseClangAstDump(&tokenizer, istr);

‎test/testclass.cpp‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -256,7 +256,7 @@ class TestClass : public TestFixture {
256256
void checkCopyCtorAndEqOperator_(const char code[], const char* file, int line) {
257257
// Clear the error log
258258
errout.str("");
259-
const Settings settings = settingsBuilder().severity(Severity::warning).build();
259+
static const Settings settings = settingsBuilder().severity(Severity::warning).build();
260260

261261
Preprocessor preprocessor(settings, settings.nomsg, nullptr);
262262

@@ -2883,7 +2883,7 @@ class TestClass : public TestFixture {
28832883

28842884
#define checkNoMemset(...) checkNoMemset_(__FILE__, __LINE__, __VA_ARGS__)
28852885
void checkNoMemset_(const char* file, int line, const char code[]) {
2886-
const Settings settings = settingsBuilder().severity(Severity::warning).severity(Severity::portability).build();
2886+
static const Settings settings = settingsBuilder().severity(Severity::warning).severity(Severity::portability).build();
28872887
checkNoMemset_(file, line, code, settings);
28882888
}
28892889

@@ -3147,7 +3147,7 @@ class TestClass : public TestFixture {
31473147
errout.str());
31483148

31493149
// #1655
3150-
const Settings s = settingsBuilder().library("std.cfg").build();
3150+
static const Settings s = settingsBuilder().library("std.cfg").build();
31513151
checkNoMemset("void f() {\n"
31523152
" char c[] = \"abc\";\n"
31533153
" std::string s;\n"
@@ -7231,7 +7231,7 @@ class TestClass : public TestFixture {
72317231
errout.str("");
72327232

72337233
// Check..
7234-
const Settings settings = settingsBuilder().severity(Severity::performance).build();
7234+
static const Settings settings = settingsBuilder().severity(Severity::performance).build();
72357235

72367236
Preprocessor preprocessor(settings, settings.nomsg, nullptr);
72377237

@@ -7912,7 +7912,7 @@ class TestClass : public TestFixture {
79127912
// Clear the error log
79137913
errout.str("");
79147914

7915-
const Settings settings = settingsBuilder().severity(Severity::style).build();
7915+
static const Settings settings = settingsBuilder().severity(Severity::style).build();
79167916

79177917
Preprocessor preprocessor(settings, settings.nomsg, nullptr);
79187918

@@ -8269,7 +8269,7 @@ class TestClass : public TestFixture {
82698269

82708270

82718271
void ctu(const std::vector<std::string> &code) {
8272-
const Settings settings;
8272+
static const Settings settings;
82738273
auto &check = getCheck<CheckClass>();
82748274

82758275
// getFileInfo

‎test/testconstructors.cpp‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1946,8 +1946,8 @@ class TestConstructors : public TestFixture {
19461946
}
19471947

19481948
void initvar_smartptr() { // #10237
1949-
// TODO: test shuld probably not pass without library
1950-
const Settings s = settingsBuilder() /*.library("std.cfg")*/.build();
1949+
// TODO: test should probably not pass without library
1950+
static const Settings s = settingsBuilder() /*.library("std.cfg")*/.build();
19511951
check("struct S {\n"
19521952
" explicit S(const std::shared_ptr<S>& sp) {\n"
19531953
" set(*sp);\n"
@@ -1989,7 +1989,7 @@ class TestConstructors : public TestFixture {
19891989
"{ }", true);
19901990
ASSERT_EQUALS("[test.cpp:13]: (warning, inconclusive) Member variable 'Fred::ints' is not assigned a value in 'Fred::operator='.\n", errout.str());
19911991

1992-
const Settings s = settingsBuilder().certainty(Certainty::inconclusive).severity(Severity::style).severity(Severity::warning).library("std.cfg").build();
1992+
static const Settings s = settingsBuilder().certainty(Certainty::inconclusive).severity(Severity::style).severity(Severity::warning).library("std.cfg").build();
19931993
check("struct S {\n"
19941994
" S& operator=(const S& s) { return *this; }\n"
19951995
" std::mutex m;\n"
@@ -3580,7 +3580,7 @@ class TestConstructors : public TestFixture {
35803580

35813581
void uninitVarInheritClassInit() {
35823582
// TODO: test should probably not pass without library
3583-
const Settings s = settingsBuilder() /*.library("vcl.cfg")*/.build();
3583+
static const Settings s = settingsBuilder() /*.library("vcl.cfg")*/.build();
35843584

35853585
check("class Fred: public TObject\n"
35863586
"{\n"

‎test/testnullpointer.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@ class TestNullPointer : public TestFixture {
193193
// Clear the error buffer..
194194
errout.str("");
195195

196-
const Settings settings1 = settingsBuilder(settings).certainty(Certainty::inconclusive, false).build();
196+
static const Settings settings1 = settingsBuilder(settings).certainty(Certainty::inconclusive, false).build();
197197

198198
// Raw tokens..
199199
std::vector<std::string> files(1, "test.cpp");

‎test/testsimplifytypedef.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,7 @@ class TestSimplifyTypedef : public TestFixture {
258258
errout.str("");
259259
// Tokenize..
260260
// show warnings about unhandled typedef
261-
const Settings settings = settingsBuilder(settings2).certainty(Certainty::inconclusive).debugwarnings().build();
261+
static const Settings settings = settingsBuilder(settings2).certainty(Certainty::inconclusive).debugwarnings().build();
262262
Tokenizer tokenizer(&settings, this);
263263
std::istringstream istr(code);
264264
ASSERT_LOC(tokenizer.tokenize(istr, "test.cpp"), file, line);

‎test/testsummaries.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ class TestSummaries : public TestFixture {
4444
errout.str("");
4545

4646
// tokenize..
47-
const Settings settings;
47+
static const Settings settings;
4848
Tokenizer tokenizer(&settings, this);
4949
std::istringstream istr(code);
5050
ASSERT_LOC(tokenizer.tokenize(istr, filename), file, line);

‎test/testtoken.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -396,7 +396,7 @@ class TestToken : public TestFixture {
396396

397397
void getStrSize() const {
398398
Token tok;
399-
const Settings settings;
399+
static const Settings settings;
400400

401401
tok.str("\"\"");
402402
ASSERT_EQUALS(sizeof(""), Token::getStrSize(&tok, &settings));

‎test/testtokenize.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -524,7 +524,7 @@ class TestTokenizer : public TestFixture {
524524
std::string tokenizeDebugListing_(const char* file, int line, const char code[], const char filename[] = "test.cpp") {
525525
errout.str("");
526526

527-
const Settings settings = settingsBuilder(settings2).c(Standards::C89).cpp(Standards::CPP03).build();
527+
static const Settings settings = settingsBuilder(settings2).c(Standards::C89).cpp(Standards::CPP03).build();
528528

529529
Tokenizer tokenizer(&settings, this);
530530
std::istringstream istr(code);

0 commit comments

Comments
 (0)