Skip to content

Commit 27cde7a

Browse files
committed
TestBufferOverrun: added TODO about missing settings reset
1 parent bb30d61 commit 27cde7a

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

test/testbufferoverrun.cpp

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

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

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

33473352
void buffer_overrun_bailoutIfSwitch() {

0 commit comments

Comments
 (0)