2008-12-18 22:28:57 +01:00
|
|
|
/*
|
2009-01-21 21:04:20 +01:00
|
|
|
* Cppcheck - A tool for static C/C++ code analysis
|
2014-02-15 07:45:39 +01:00
|
|
|
* Copyright (C) 2007-2014 Daniel Marjamäki and Cppcheck team.
|
2008-12-18 22:28:57 +01:00
|
|
|
*
|
|
|
|
* This program is free software: you can redistribute it and/or modify
|
|
|
|
* it under the terms of the GNU General Public License as published by
|
|
|
|
* the Free Software Foundation, either version 3 of the License, or
|
|
|
|
* (at your option) any later version.
|
|
|
|
*
|
|
|
|
* This program is distributed in the hope that it will be useful,
|
|
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
|
|
* GNU General Public License for more details.
|
|
|
|
*
|
|
|
|
* You should have received a copy of the GNU General Public License
|
2009-09-27 17:08:31 +02:00
|
|
|
* along with this program. If not, see <http://www.gnu.org/licenses/>.
|
2008-12-18 22:28:57 +01:00
|
|
|
*/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
// Check for unused variables..
|
|
|
|
|
2009-02-07 21:06:00 +01:00
|
|
|
|
2008-12-18 22:28:57 +01:00
|
|
|
#include "testsuite.h"
|
2009-10-25 12:49:06 +01:00
|
|
|
#include "tokenize.h"
|
|
|
|
#include "checkother.h"
|
2008-12-18 22:28:57 +01:00
|
|
|
|
|
|
|
#include <sstream>
|
|
|
|
|
|
|
|
extern std::ostringstream errout;
|
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
class TestIncompleteStatement : public TestFixture {
|
2008-12-18 22:28:57 +01:00
|
|
|
public:
|
2013-08-07 16:27:37 +02:00
|
|
|
TestIncompleteStatement() : TestFixture("TestIncompleteStatement") {
|
|
|
|
}
|
2008-12-18 22:28:57 +01:00
|
|
|
|
|
|
|
private:
|
2011-10-13 20:53:06 +02:00
|
|
|
void check(const char code[]) {
|
2010-12-01 18:00:55 +01:00
|
|
|
// Clear the error buffer..
|
|
|
|
errout.str("");
|
|
|
|
|
|
|
|
Settings settings;
|
2013-03-03 11:41:59 +01:00
|
|
|
settings.addEnabled("warning");
|
2010-12-01 18:00:55 +01:00
|
|
|
|
2008-12-18 22:28:57 +01:00
|
|
|
// Tokenize..
|
2010-12-01 18:00:55 +01:00
|
|
|
Tokenizer tokenizer(&settings, this);
|
2008-12-18 22:28:57 +01:00
|
|
|
std::istringstream istr(code);
|
2009-01-05 16:49:57 +01:00
|
|
|
tokenizer.tokenize(istr, "test.cpp");
|
2013-12-30 17:45:28 +01:00
|
|
|
tokenizer.simplifyTokenList2();
|
2008-12-18 22:28:57 +01:00
|
|
|
|
2010-04-17 09:23:54 +02:00
|
|
|
// Check for incomplete statements..
|
2009-03-20 18:16:21 +01:00
|
|
|
CheckOther checkOther(&tokenizer, &settings, this);
|
2009-07-05 22:16:43 +02:00
|
|
|
checkOther.checkIncompleteStatement();
|
2008-12-18 22:28:57 +01:00
|
|
|
}
|
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void run() {
|
2009-01-05 16:49:57 +01:00
|
|
|
TEST_CASE(test1);
|
|
|
|
TEST_CASE(test2);
|
2009-02-05 19:57:53 +01:00
|
|
|
TEST_CASE(test3);
|
2009-02-08 09:52:03 +01:00
|
|
|
TEST_CASE(test4);
|
2009-02-11 16:16:32 +01:00
|
|
|
TEST_CASE(test5);
|
2009-02-08 10:51:45 +01:00
|
|
|
TEST_CASE(test_numeric);
|
2009-02-11 16:17:13 +01:00
|
|
|
TEST_CASE(intarray);
|
2009-02-12 13:59:43 +01:00
|
|
|
TEST_CASE(structarraynull);
|
|
|
|
TEST_CASE(structarray);
|
2011-01-13 20:12:57 +01:00
|
|
|
TEST_CASE(conditionalcall); // ; 0==x ? X() : Y();
|
2011-01-19 18:37:33 +01:00
|
|
|
TEST_CASE(structinit); // #2462 : ABC abc{1,2,3};
|
2011-03-30 21:57:01 +02:00
|
|
|
TEST_CASE(returnstruct);
|
2011-08-16 20:16:33 +02:00
|
|
|
TEST_CASE(cast); // #3009 : (struct Foo *)123.a = 1;
|
2011-11-05 20:28:52 +01:00
|
|
|
TEST_CASE(increment); // #3251 : FP for increment
|
2008-12-18 22:28:57 +01:00
|
|
|
}
|
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void test1() {
|
2009-01-05 16:49:57 +01:00
|
|
|
check("void foo()\n"
|
|
|
|
"{\n"
|
|
|
|
" const char def[] =\n"
|
|
|
|
" \"abc\";\n"
|
2013-03-20 15:36:16 +01:00
|
|
|
"}");
|
2009-01-05 16:49:57 +01:00
|
|
|
|
2009-06-05 02:39:36 +02:00
|
|
|
ASSERT_EQUALS("", errout.str());
|
2008-12-18 22:28:57 +01:00
|
|
|
}
|
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void test2() {
|
2009-01-05 16:49:57 +01:00
|
|
|
check("void foo()\n"
|
|
|
|
"{\n"
|
2009-02-05 19:57:53 +01:00
|
|
|
" \"abc\";\n"
|
2013-03-20 15:36:16 +01:00
|
|
|
"}");
|
2008-12-18 22:28:57 +01:00
|
|
|
|
2012-09-16 19:44:02 +02:00
|
|
|
ASSERT_EQUALS("[test.cpp:3]: (warning) Redundant code: Found a statement that begins with string constant.\n", errout.str());
|
2008-12-18 22:28:57 +01:00
|
|
|
}
|
2009-02-05 19:57:53 +01:00
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void test3() {
|
2009-02-05 19:57:53 +01:00
|
|
|
check("void foo()\n"
|
|
|
|
"{\n"
|
|
|
|
" const char *str[] = { \"abc\" };\n"
|
2013-03-20 15:36:16 +01:00
|
|
|
"}");
|
2009-02-05 19:57:53 +01:00
|
|
|
|
2009-06-05 02:39:36 +02:00
|
|
|
ASSERT_EQUALS("", errout.str());
|
2009-02-05 19:57:53 +01:00
|
|
|
}
|
2009-02-08 09:52:03 +01:00
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void test4() {
|
2009-02-08 09:52:03 +01:00
|
|
|
check("void foo()\n"
|
|
|
|
"{\n"
|
|
|
|
"const char *a =\n"
|
|
|
|
"{\n"
|
|
|
|
"\"hello \"\n"
|
2009-02-08 11:25:33 +01:00
|
|
|
"\"more \"\n"
|
2009-02-08 09:52:03 +01:00
|
|
|
"\"world\"\n"
|
|
|
|
"};\n"
|
2013-03-20 15:36:16 +01:00
|
|
|
"}");
|
2009-02-08 09:52:03 +01:00
|
|
|
|
2009-06-05 02:39:36 +02:00
|
|
|
ASSERT_EQUALS("", errout.str());
|
2009-02-08 09:52:03 +01:00
|
|
|
}
|
2009-02-08 10:51:45 +01:00
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void test5() {
|
2009-02-11 16:16:32 +01:00
|
|
|
check("void foo()\n"
|
|
|
|
"{\n"
|
|
|
|
" 50;\n"
|
2013-03-20 15:36:16 +01:00
|
|
|
"}");
|
2009-02-11 16:16:32 +01:00
|
|
|
|
2012-09-16 19:44:02 +02:00
|
|
|
ASSERT_EQUALS("[test.cpp:3]: (warning) Redundant code: Found a statement that begins with numeric constant.\n", errout.str());
|
2009-02-11 16:16:32 +01:00
|
|
|
}
|
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void test_numeric() {
|
2009-02-08 10:51:45 +01:00
|
|
|
check("struct P\n"
|
|
|
|
"{\n"
|
|
|
|
"double a;\n"
|
|
|
|
"double b;\n"
|
|
|
|
"};\n"
|
|
|
|
"void f()\n"
|
|
|
|
"{\n"
|
|
|
|
"const P values[2] =\n"
|
|
|
|
"{\n"
|
|
|
|
"{ 346.1,114.1 }, { 347.1,111.1 }\n"
|
|
|
|
"};\n"
|
2013-03-20 15:36:16 +01:00
|
|
|
"}");
|
2009-02-08 10:51:45 +01:00
|
|
|
|
2009-06-05 02:39:36 +02:00
|
|
|
ASSERT_EQUALS("", errout.str());
|
2009-02-08 10:51:45 +01:00
|
|
|
}
|
2009-02-09 21:15:14 +01:00
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void intarray() {
|
2013-03-20 15:36:16 +01:00
|
|
|
check("int arr[] = { 100/2, 1*100 };");
|
2009-06-05 02:39:36 +02:00
|
|
|
ASSERT_EQUALS("", errout.str());
|
2009-02-09 21:15:14 +01:00
|
|
|
}
|
2009-02-12 13:59:43 +01:00
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void structarraynull() {
|
2009-02-12 13:59:43 +01:00
|
|
|
check("struct st arr[] = {\n"
|
|
|
|
" { 100/2, 1*100 }\n"
|
|
|
|
" { 90, 70 }\n"
|
2013-03-20 15:36:16 +01:00
|
|
|
"}");
|
2009-06-05 02:39:36 +02:00
|
|
|
ASSERT_EQUALS("", errout.str());
|
2009-02-12 13:59:43 +01:00
|
|
|
}
|
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void structarray() {
|
2009-02-12 13:59:43 +01:00
|
|
|
check("struct st arr[] = {\n"
|
|
|
|
" { 100/2, 1*100 }\n"
|
|
|
|
" { 90, 70 }\n"
|
2013-03-20 15:36:16 +01:00
|
|
|
"};");
|
2009-06-05 02:39:36 +02:00
|
|
|
ASSERT_EQUALS("", errout.str());
|
2009-02-12 13:59:43 +01:00
|
|
|
}
|
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void conditionalcall() {
|
2011-01-13 20:12:57 +01:00
|
|
|
check("void f() {\n"
|
|
|
|
" 0==x ? X() : Y();\n"
|
2013-03-20 15:36:16 +01:00
|
|
|
"}");
|
2011-01-13 20:12:57 +01:00
|
|
|
ASSERT_EQUALS("", errout.str());
|
|
|
|
}
|
2011-01-19 18:37:33 +01:00
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void structinit() {
|
2013-02-24 10:00:03 +01:00
|
|
|
// #2462 - C++11 struct initialization
|
2011-01-19 18:37:33 +01:00
|
|
|
check("void f() {\n"
|
|
|
|
" ABC abc{1,2,3};\n"
|
2013-03-20 15:36:16 +01:00
|
|
|
"}");
|
2011-01-19 18:37:33 +01:00
|
|
|
ASSERT_EQUALS("", errout.str());
|
2011-01-19 20:41:46 +01:00
|
|
|
|
|
|
|
// #2482 - false positive for empty struct
|
|
|
|
check("struct A {};");
|
|
|
|
ASSERT_EQUALS("", errout.str());
|
2013-02-23 16:26:25 +01:00
|
|
|
|
|
|
|
// #4387 - C++11 initializer list
|
|
|
|
check("A::A() : abc{0} {}");
|
|
|
|
ASSERT_EQUALS("", errout.str());
|
2013-10-05 12:26:09 +02:00
|
|
|
|
|
|
|
// #5042 - C++11 initializer list
|
|
|
|
check("A::A() : abc::def<int>{0} {}");
|
|
|
|
ASSERT_EQUALS("", errout.str());
|
2013-02-24 10:00:03 +01:00
|
|
|
|
|
|
|
// #4503 - vector init
|
|
|
|
check("void f() { vector<int> v{1}; }");
|
|
|
|
ASSERT_EQUALS("", errout.str());
|
2011-01-19 18:37:33 +01:00
|
|
|
}
|
2011-03-30 21:57:01 +02:00
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void returnstruct() {
|
2011-03-30 21:57:01 +02:00
|
|
|
check("struct s foo() {\n"
|
|
|
|
" return (struct s){0,0};\n"
|
|
|
|
"}");
|
|
|
|
ASSERT_EQUALS("", errout.str());
|
2013-04-20 07:51:44 +02:00
|
|
|
|
|
|
|
// #4754
|
|
|
|
check("unordered_map<string, string> foo() {\n"
|
|
|
|
" return {\n"
|
|
|
|
" {\"hi\", \"there\"},\n"
|
|
|
|
" {\"happy\", \"sad\"}\n"
|
|
|
|
" };\n"
|
|
|
|
"}");
|
|
|
|
ASSERT_EQUALS("", errout.str());
|
2011-03-30 21:57:01 +02:00
|
|
|
}
|
2011-08-16 20:16:33 +02:00
|
|
|
|
2011-10-13 20:53:06 +02:00
|
|
|
void cast() {
|
2011-08-16 20:16:33 +02:00
|
|
|
check("void f() {\n"
|
|
|
|
" ((struct foo *)(0x1234))->xy = 1;\n"
|
|
|
|
"}");
|
|
|
|
ASSERT_EQUALS("", errout.str());
|
|
|
|
}
|
2011-11-05 20:28:52 +01:00
|
|
|
|
|
|
|
void increment() {
|
|
|
|
check("void f() {\n"
|
|
|
|
" int x = 1;\n"
|
|
|
|
" x++, x++;\n"
|
|
|
|
"}");
|
|
|
|
ASSERT_EQUALS("", errout.str());
|
|
|
|
}
|
2008-12-18 22:28:57 +01:00
|
|
|
};
|
|
|
|
|
2009-01-05 16:49:57 +01:00
|
|
|
REGISTER_TEST(TestIncompleteStatement)
|