diff options
author | Peter Powell <petpow@saberuk.com> | 2013-07-07 19:29:50 +0100 |
---|---|---|
committer | Peter Powell <petpow@saberuk.com> | 2013-07-16 22:06:40 +0100 |
commit | 0003fc3131974e95de3951654a3493e7f5f2f969 (patch) | |
tree | 7c3c2b94d957c1d4f1501c303f438b03d857c867 | |
parent | 6033f1d6fa3d37e068a56c29ef47b220abb3174d (diff) |
Fix some warnings which are causing debug builds to fail.
- Clang: private field 'module' is not used
- GCC: suggest a space before ‘;’ or explicit braces around empty
body in ‘while’ statement
-rw-r--r-- | include/modules/httpd.h | 2 | ||||
-rw-r--r-- | src/configparser.cpp | 5 |
2 files changed, 5 insertions, 2 deletions
diff --git a/include/modules/httpd.h b/include/modules/httpd.h index 3d94b3f42..86234d53f 100644 --- a/include/modules/httpd.h +++ b/include/modules/httpd.h @@ -185,11 +185,11 @@ class HTTPRequest : public Event */ class HTTPDocumentResponse { + public: /** Module that generated this reply */ Module* const module; - public: std::stringstream* document; unsigned int responsecode; diff --git a/src/configparser.cpp b/src/configparser.cpp index 31d314148..3289cf396 100644 --- a/src/configparser.cpp +++ b/src/configparser.cpp @@ -183,7 +183,10 @@ struct Parser std::set<std::string> seen; tag = ConfigTag::create(name, current.filename, current.line, items); - while (kv(items, seen)); + while (kv(items, seen)) + { + // Do nothing here (silences a GCC warning). + } if (name == mandatory_tag) { |