diff options
author | Andrei Karas <akaras@inbox.ru> | 2015-06-17 17:47:37 +0300 |
---|---|---|
committer | Andrei Karas <akaras@inbox.ru> | 2015-06-17 17:47:37 +0300 |
commit | baed04aada7ac8a595e1de142c01c2754debd57e (patch) | |
tree | d01e7d542e9a0a41030a2bf4cbffb660ec523eeb | |
parent | f92d8d4c27a2c8602ac5ec888fc344d01a5da5cc (diff) | |
download | paranucker-baed04aada7ac8a595e1de142c01c2754debd57e.tar.gz paranucker-baed04aada7ac8a595e1de142c01c2754debd57e.tar.bz2 paranucker-baed04aada7ac8a595e1de142c01c2754debd57e.tar.xz paranucker-baed04aada7ac8a595e1de142c01c2754debd57e.zip |
Improve null pointer warning message.
-rw-r--r-- | src/analysis/analysis.cpp | 3 | ||||
-rw-r--r-- | src/logger.cpp | 6 | ||||
-rw-r--r-- | src/logger.h | 5 |
3 files changed, 9 insertions, 5 deletions
diff --git a/src/analysis/analysis.cpp b/src/analysis/analysis.cpp index ca689e1..3d7a9de 100644 --- a/src/analysis/analysis.cpp +++ b/src/analysis/analysis.cpp @@ -110,7 +110,8 @@ void reportParmDeclNullPointer(Node *mainNode, if (wi.checkNullVars.find(node->label) != wi.checkNullVars.end()) { Log::warn(findBackLocation(mainNode), - "Using variable without check for NULL"); + "Using parameter '%s' without checking for null pointer", + node->label); } } } diff --git a/src/logger.cpp b/src/logger.cpp index ed09fc5..64a8ac2 100644 --- a/src/logger.cpp +++ b/src/logger.cpp @@ -164,9 +164,11 @@ void dump(const Node *const node) fprintf(stderr, "\n"); } -void warn(int loc, std::string message) +void warn(const int loc, + const std::string &message, + const std::string ¶m) { - warning_at(loc, 0, message.c_str()); + warning_at(loc, 0, message.c_str(), param.c_str()); } }
\ No newline at end of file diff --git a/src/logger.h b/src/logger.h index c575ef9..815ece5 100644 --- a/src/logger.h +++ b/src/logger.h @@ -51,8 +51,9 @@ namespace Log void dump(const Node *const node); - void warn(int loc, - std::string message); + void warn(const int loc, + const std::string &message, + const std::string ¶m); } #endif // LOGGER_H |