removed testcase for #5238. it's not good enough.
This commit is contained in:
parent
84486605ec
commit
871b02ea88
|
@ -71,7 +71,6 @@ private:
|
||||||
TEST_CASE(nullpointer26); // #3589
|
TEST_CASE(nullpointer26); // #3589
|
||||||
TEST_CASE(nullpointer27); // #6568
|
TEST_CASE(nullpointer27); // #6568
|
||||||
TEST_CASE(nullpointer28); // #6491
|
TEST_CASE(nullpointer28); // #6491
|
||||||
TEST_CASE(nullpointer29); // #5238
|
|
||||||
TEST_CASE(nullpointer30); // #6392
|
TEST_CASE(nullpointer30); // #6392
|
||||||
TEST_CASE(nullpointer_addressOf); // address of
|
TEST_CASE(nullpointer_addressOf); // address of
|
||||||
TEST_CASE(nullpointerSwitch); // #2626
|
TEST_CASE(nullpointerSwitch); // #2626
|
||||||
|
@ -1307,23 +1306,6 @@ private:
|
||||||
ASSERT_EQUALS("[test.cpp:4]: (error) Possible null pointer dereference: s\n", errout.str());
|
ASSERT_EQUALS("[test.cpp:4]: (error) Possible null pointer dereference: s\n", errout.str());
|
||||||
}
|
}
|
||||||
|
|
||||||
void nullpointer29() { // #5238
|
|
||||||
check("struct SomeStruct\n"
|
|
||||||
"{\n"
|
|
||||||
" bool item;\n"
|
|
||||||
"};\n"
|
|
||||||
"bool f1(bool bFlag)\n"
|
|
||||||
"{\n"
|
|
||||||
" SomeStruct *s = 0;\n"
|
|
||||||
" if (bFlag)\n"
|
|
||||||
" {\n"
|
|
||||||
" s = new SomeStruct;\n"
|
|
||||||
" }\n"
|
|
||||||
" return s != 0 || (s != 0 && !s->item);\n"
|
|
||||||
"}\n", true);
|
|
||||||
ASSERT_EQUALS("[test.cpp:12] -> [test.cpp:12]: (warning) Either the condition 's!=0' is redundant or there is possible null pointer dereference: s.\n", errout.str());
|
|
||||||
}
|
|
||||||
|
|
||||||
void nullpointer30() { // #6392
|
void nullpointer30() { // #6392
|
||||||
check("void f(std::vector<std::string> *values)\n"
|
check("void f(std::vector<std::string> *values)\n"
|
||||||
"{\n"
|
"{\n"
|
||||||
|
|
Loading…
Reference in New Issue