Skip to content

Commit 08fe4d6

Browse files
committed
TestBufferOverrun: added TODO about missing settings reset
1 parent adab5de commit 08fe4d6

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

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

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

33343339
void buffer_overrun_bailoutIfSwitch() {

0 commit comments

Comments
 (0)