walkero
/
cppcheck
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

Put apostrophes consistent with other similar messages

This commit is contained in:
Dmitry-Me 2016-10-18 15:45:08 +03:00
parent d5a145f288
commit d2e667ae7b
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/checkclass.cpp
View File

@ -384,7 +384,7 @@ void CheckClass::noCopyConstructorError(const Token *tok, const std::string &cla
{ {
// The constructor might be intentionally missing. Therefore this is not a "warning" // The constructor might be intentionally missing. Therefore this is not a "warning"
reportError(tok, Severity::style, "noCopyConstructor", reportError(tok, Severity::style, "noCopyConstructor",
"'" + std::string(isStruct ? "struct" : "class") + " " + classname + std::string(isStruct ? "struct" : "class") + " '" + classname +
"' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.", CWE398, false); "' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.", CWE398, false);
} }

4
test/testclass.cpp
View File

@ -530,7 +530,7 @@ private:
" d=(char*)malloc(100);\n" " d=(char*)malloc(100);\n"
" }\n" " }\n"
"};"); "};");
ASSERT_EQUALS("[test.cpp:1]: (style) 'class F' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.\n", errout.str()); ASSERT_EQUALS("[test.cpp:1]: (style) class 'F' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.\n", errout.str());
checkCopyConstructor("class F\n" checkCopyConstructor("class F\n"
"{\n" "{\n"
@ -595,7 +595,7 @@ private:
" char *p;\n" " char *p;\n"
" F() : p(malloc(100)) {}\n" " F() : p(malloc(100)) {}\n"
"};"); "};");
ASSERT_EQUALS("[test.cpp:1]: (style) 'class F' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.\n", errout.str()); ASSERT_EQUALS("[test.cpp:1]: (style) class 'F' does not have a copy constructor which is recommended since the class contains a pointer to allocated memory.\n", errout.str());
// #7198 // #7198
checkCopyConstructor("struct F {\n" checkCopyConstructor("struct F {\n"