From 0003fc3131974e95de3951654a3493e7f5f2f969 Mon Sep 17 00:00:00 2001 From: Peter Powell Date: Sun, 7 Jul 2013 19:29:50 +0100 Subject: [PATCH] Fix some warnings which are causing debug builds to fail. MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit - Clang: private field 'module' is not used - GCC: suggest a space before ‘;’ or explicit braces around empty body in ‘while’ statement --- include/modules/httpd.h | 2 +- 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 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) { -- 2.39.2