From 416115e38fe9767e76cf51a175c8f862f7d4fe1c Mon Sep 17 00:00:00 2001 From: Gianluca Scacco Date: Sun, 22 Mar 2009 12:00:21 +0100 Subject: [PATCH 1/2] Fixed error compiling testautovariables --- Makefile | 4 ++ test/testautovariables.cpp | 91 ++++++++++++++++++++++++++++++++++++++ 2 files changed, 95 insertions(+) create mode 100644 test/testautovariables.cpp diff --git a/Makefile b/Makefile index aaba07b49..57fefa967 100644 --- a/Makefile +++ b/Makefile @@ -33,6 +33,7 @@ TESTOBJ = test/testautovariables.o \ test/testconstructors.o \ test/testcppcheck.o \ test/testdangerousfunctions.o \ + test/testautovariables.o \ test/testdivision.o \ test/testfilelister.o \ test/testfunctionusage.o \ @@ -184,6 +185,9 @@ test/testcppcheck.o: test/testcppcheck.cpp test/testsuite.h src/errorlogger.h sr test/testdangerousfunctions.o: test/testdangerousfunctions.cpp src/tokenize.h src/settings.h src/errorlogger.h src/token.h src/checkdangerousfunctions.h src/check.h test/testsuite.h $(CXX) $(CXXFLAGS) -c -o test/testdangerousfunctions.o test/testdangerousfunctions.cpp +test/testautovariables.o: test/testautovariables.cpp src/tokenize.h src/settings.h src/errorlogger.h src/token.h src/checkdangerousfunctions.h src/check.h test/testsuite.h + $(CXX) $(CXXFLAGS) -c -o test/testautovariables.o test/testautovariables.cpp + test/testdivision.o: test/testdivision.cpp src/tokenize.h src/settings.h src/errorlogger.h src/token.h src/checkother.h src/check.h test/testsuite.h $(CXX) $(CXXFLAGS) -c -o test/testdivision.o test/testdivision.cpp diff --git a/test/testautovariables.cpp b/test/testautovariables.cpp new file mode 100644 index 000000000..61964b9e0 --- /dev/null +++ b/test/testautovariables.cpp @@ -0,0 +1,91 @@ +/* + * Cppcheck - A tool for static C/C++ code analysis + * Copyright (C) 2007-2009 Daniel Marjamäki, Reijo Tomperi, Nicolas Le Cam, + * Leandro Penz, Kimmo Varis, Vesa Pikki, Gianluca Scacco + * + * 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 + * along with this program. If not, see + +extern std::ostringstream errout; + +class TestAutoVariables : public TestFixture +{ +public: + TestAutoVariables() : TestFixture("TestAutoVariables") + { } + +private: + + + + void check(const char code[]) + { + // Tokenize.. + Tokenizer tokenizer; + std::istringstream istr(code); + tokenizer.tokenize(istr, "test.cpp"); + tokenizer.simplifyTokenList(); + + // Assign variable ids + tokenizer.setVarId(); + + // Fill function list + tokenizer.fillFunctionList(); + + // Clear the error buffer.. + errout.str(""); + + // Check for buffer overruns.. + Settings settings; + settings._showAll = true; + CheckAutoVariables checkAutoVariables(&tokenizer, &settings, this); + checkAutoVariables.autoVariables(); + } + + void run() + { + TEST_CASE(testautovar); + TEST_CASE(testautovararray); + } + + + + void testautovar() + { + check("void func1(int *res)\n" + "{\n" + " int num=2;" + "res=#"); + ASSERT_EQUALS(std::string("[test.cpp:3]: (error) Wrong assignement of an auto-variable to an effective parameter of a function\n"), errout.str()); + } + void testautovararray() + { + check("void func1(int* arr[2])\n" + "{\n" + " int num=2;" + "arr[0]=#"); + ASSERT_EQUALS(std::string("[test.cpp:3]: (error) Wrong assignement of an auto-variable to an effective parameter of a function\n"), errout.str()); + } +}; + +REGISTER_TEST(TestAutoVariables) + + From a065ed9d0548c40b26c3c971a59e578a25d1197c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20Marjam=C3=A4ki?= Date: Sun, 22 Mar 2009 12:46:20 +0100 Subject: [PATCH 2/2] updated the Makefile --- Makefile | 4 ---- 1 file changed, 4 deletions(-) diff --git a/Makefile b/Makefile index 57fefa967..aaba07b49 100644 --- a/Makefile +++ b/Makefile @@ -33,7 +33,6 @@ TESTOBJ = test/testautovariables.o \ test/testconstructors.o \ test/testcppcheck.o \ test/testdangerousfunctions.o \ - test/testautovariables.o \ test/testdivision.o \ test/testfilelister.o \ test/testfunctionusage.o \ @@ -185,9 +184,6 @@ test/testcppcheck.o: test/testcppcheck.cpp test/testsuite.h src/errorlogger.h sr test/testdangerousfunctions.o: test/testdangerousfunctions.cpp src/tokenize.h src/settings.h src/errorlogger.h src/token.h src/checkdangerousfunctions.h src/check.h test/testsuite.h $(CXX) $(CXXFLAGS) -c -o test/testdangerousfunctions.o test/testdangerousfunctions.cpp -test/testautovariables.o: test/testautovariables.cpp src/tokenize.h src/settings.h src/errorlogger.h src/token.h src/checkdangerousfunctions.h src/check.h test/testsuite.h - $(CXX) $(CXXFLAGS) -c -o test/testautovariables.o test/testautovariables.cpp - test/testdivision.o: test/testdivision.cpp src/tokenize.h src/settings.h src/errorlogger.h src/token.h src/checkother.h src/check.h test/testsuite.h $(CXX) $(CXXFLAGS) -c -o test/testdivision.o test/testdivision.cpp