From bb361c73adaf3b9285a9b19c8a9a2625afad700f Mon Sep 17 00:00:00 2001 From: Oleksandr Labetskyi Date: Wed, 13 Dec 2023 18:35:53 +0200 Subject: [PATCH] #12158: add another test and a small fix --- test/testother.cpp | 22 +++++++++++++++++++--- 1 file changed, 19 insertions(+), 3 deletions(-) diff --git a/test/testother.cpp b/test/testother.cpp index ad03ca0b7d6..6baa74cb926 100644 --- a/test/testother.cpp +++ b/test/testother.cpp @@ -102,6 +102,7 @@ class TestOther : public TestFixture { TEST_CASE(varScope34); TEST_CASE(varScope35); TEST_CASE(varScope36); // #12158 + TEST_CASE(varScope37); // #12158 TEST_CASE(oldStylePointerCast); TEST_CASE(invalidPointerCast); @@ -1675,9 +1676,24 @@ class TestOther : public TestFixture { " if ( value > 100U ) { }\n" " else if( value > 50U ) { }\n" " else{\n" - " for( i = 0U; i < 5U; i++ ) {}" - " }" - "}", nullptr, false); + " for( i = 0U; i < 5U; i++ ) {}\n" + " }\n" + "}\n", nullptr, false); + ASSERT_EQUALS("[test.cpp:2]: (style) The scope of the variable 'i' can be reduced.\n", errout.str()); + } + + void varScope37() { + // #12158 + check("void f( uint32_t value ) {\n" + " uint32_t i = 0U;\n" + " if ( value > 100U ) { }\n" + " else {\n" + " if( value > 50U ) { }\n" + " else{\n" + " for( i = 0U; i < 5U; i++ ) {}\n" + " }\n" + " }\n" + "}\n", nullptr, false); ASSERT_EQUALS("[test.cpp:2]: (style) The scope of the variable 'i' can be reduced.\n", errout.str()); }