Skip to content

Commit fd474bb

Browse files
committed
TestBufferOverrun: added TODO about missing settings reset
1 parent b675612 commit fd474bb

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
@@ -3334,6 +3334,7 @@ class TestBufferOverrun : public TestFixture {
33343334

33353335
void buffer_overrun_errorpath() {
33363336
setMultiline();
3337+
const Settings settingsOld = settings0;
33373338
settings0.templateLocation = "{file}:{line}:note:{info}";
33383339

33393340
check("void f() {\n"
@@ -3343,6 +3344,10 @@ class TestBufferOverrun : public TestFixture {
33433344
ASSERT_EQUALS("test.cpp:3:error:Buffer is accessed out of bounds: p\n"
33443345
"test.cpp:2:note:Assign p, buffer with size 10\n"
33453346
"test.cpp:3:note:Buffer overrun\n", errout.str());
3347+
3348+
// TODO: need to reset this but it breaks other tests
3349+
(void)settingsOld;
3350+
//settings0 = settingsOld;
33463351
}
33473352

33483353
void buffer_overrun_bailoutIfSwitch() {

0 commit comments

Comments
 (0)