Skip to content

Commit 35b24cd

Browse files
committed
TestBufferOverrun: added TODO about missing settings reset
1 parent 3fb83e9 commit 35b24cd

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

33203320
void buffer_overrun_errorpath() {
33213321
setMultiline();
3322+
const Settings settingsOld = settings0;
33223323
settings0.templateLocation = "{file}:{line}:note:{info}";
33233324

33243325
check("void f() {\n"
@@ -3328,6 +3329,10 @@ class TestBufferOverrun : public TestFixture {
33283329
ASSERT_EQUALS("test.cpp:3:error:Buffer is accessed out of bounds: p\n"
33293330
"test.cpp:2:note:Assign p, buffer with size 10\n"
33303331
"test.cpp:3:note:Buffer overrun\n", errout.str());
3332+
3333+
// TODO: need to reset this but it breaks other tests
3334+
(void)settingsOld;
3335+
//settings0 = settingsOld;
33313336
}
33323337

33333338
void buffer_overrun_bailoutIfSwitch() {

0 commit comments

Comments
 (0)