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

diff --git a/include/modules/httpd.h b/include/modules/httpd.h
index 3d94b3f42a687c6adcdc0b6305217d15a0860991..86234d53faddfcf697379681fb52712fe7a98c17 100644 (file)
--- 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 31d3141482aaef9452339d1838863daa7c45b09d..3289cf396bdff40832291d58f8cecf05ed59f3f1 100644 (file)
--- 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)
                {