Skip to content

Commit 10d65b0

Browse files
committed
TestBufferOverrun: added TODO about missing settings reset
1 parent 08e3745 commit 10d65b0

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

test/testbufferoverrun.cpp

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3327,6 +3327,7 @@ class TestBufferOverrun : public TestFixture {
33273327

33283328
void buffer_overrun_errorpath() {
33293329
setMultiline();
3330+
const Settings settingsOld = settings0;
33303331
settings0.templateLocation = "{file}:{line}:note:{info}";
33313332

33323333
check("void f() {\n"
@@ -3336,6 +3337,10 @@ class TestBufferOverrun : public TestFixture {
33363337
ASSERT_EQUALS("test.cpp:3:error:Buffer is accessed out of bounds: p\n"
33373338
"test.cpp:2:note:Assign p, buffer with size 10\n"
33383339
"test.cpp:3:note:Buffer overrun\n", errout.str());
3340+
3341+
// TODO: need to reset this but it breaks other tests
3342+
(void)settingsOld;
3343+
//settings0 = settingsOld;
33393344
}
33403345

33413346
void buffer_overrun_bailoutIfSwitch() {

0 commit comments

Comments
 (0)