summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPeter Powell <petpow@saberuk.com>2013-07-07 19:29:50 +0100
committerPeter Powell <petpow@saberuk.com>2013-07-16 22:06:40 +0100
commit0003fc3131974e95de3951654a3493e7f5f2f969 (patch)
tree7c3c2b94d957c1d4f1501c303f438b03d857c867
parent6033f1d6fa3d37e068a56c29ef47b220abb3174d (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.h2
-rw-r--r--src/configparser.cpp5
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)
{