From 06f58c82c17d398e0f919324482f048f30888882 Mon Sep 17 00:00:00 2001
From: grumbel <grumbel@837edb03-e0f3-0310-88ca-d4d4e8b29345>
Date: Sat, 21 Nov 2009 14:39:51 +0000
Subject: [PATCH] -Weffc++ cleanup for miniswig

git-svn-id: http://supertux.lethargik.org/svn/supertux/trunk/supertux@6079 837edb03-e0f3-0310-88ca-d4d4e8b29345
---
 tools/miniswig/create_wrapper.hpp | 15 +++++---
 tools/miniswig/parser.yy          |  3 +-
 tools/miniswig/tree.hpp           | 78 +++++++++++++++++++++++++++++++--------
 tools/miniswig/xmlwriter.cpp      |  8 +++-
 4 files changed, 81 insertions(+), 23 deletions(-)

diff --git a/tools/miniswig/create_wrapper.hpp b/tools/miniswig/create_wrapper.hpp
index a2e5395d6..2fbe54d5c 100644
--- a/tools/miniswig/create_wrapper.hpp
+++ b/tools/miniswig/create_wrapper.hpp
@@ -12,11 +12,12 @@ public:
     std::ostream& out;
     std::ostream& hppout;
 
-    WrapperCreator(std::ostream& _out = std::cout, std::ostream& _hppout = std::cout)
-        : out(_out), hppout(_hppout)
-    {
-        ind = "  ";
-    }
+    WrapperCreator(std::ostream& _out = std::cout, std::ostream& _hppout = std::cout) :
+        ind("  "),
+        out(_out),
+        hppout(_hppout),
+        ns_prefix()
+    { }
 
     void create_wrapper(Namespace* ns);
 
@@ -41,6 +42,10 @@ private:
     void create_function_wrapper(Class* _class, Function* function);
     void prepare_argument(const Type& type, size_t idx, const std::string& var);
     void push_to_stack(const Type& type, const std::string& var);
+
+private:
+    WrapperCreator(const WrapperCreator&);
+    WrapperCreator& operator=(const WrapperCreator&);
 };
 
 #endif
diff --git a/tools/miniswig/parser.yy b/tools/miniswig/parser.yy
index 03fb00c99..a47ffedbf 100644
--- a/tools/miniswig/parser.yy
+++ b/tools/miniswig/parser.yy
@@ -40,7 +40,8 @@ static ClassMember::Visibility current_visibility;
 class ParseError : public std::exception
 {
 public:
-    ParseError(const std::string& message) throw()
+    ParseError(const std::string& message) throw() :
+         message()
     {
         std::ostringstream msg;
         msg << "Parse error in '" << current_file
diff --git a/tools/miniswig/tree.hpp b/tools/miniswig/tree.hpp
index 5c7927fd9..47741de51 100644
--- a/tools/miniswig/tree.hpp
+++ b/tools/miniswig/tree.hpp
@@ -12,8 +12,9 @@ class Namespace;
 
 class AtomicType {
 public:
-    AtomicType()
-        : parent(0)
+    AtomicType() :
+      name(),
+      parent(0)
     { }
     virtual ~AtomicType()
     { }
@@ -25,6 +26,10 @@ public:
 
     std::string name;
     Namespace* parent;
+
+private:
+    AtomicType(const AtomicType&);
+    AtomicType& operator=(const AtomicType&);
 };
 
 class BasicType : public AtomicType {
@@ -156,14 +161,23 @@ private:
     static StringType* _instance;
 };
 
-class Parameter {
+class Parameter
+{
 public:
+    Parameter() :
+        name(),
+        type()
+    { }
+
     std::string name;
     Type type;
 };
 
 class ClassMember {
 public:
+    ClassMember() :
+        visibility()
+    { }
     virtual ~ClassMember()
     { }
 
@@ -177,10 +191,19 @@ public:
 
 class Function : public ClassMember {
 public:
-    Function() {
-      type = FUNCTION;
-      suspend = false;
-      custom = false;
+    Function() :
+        type(),
+        suspend(),
+        custom(),
+        parameter_spec(),
+        docu_comment(),
+        name(),
+        return_type(),
+        parameters()
+    {
+        type = FUNCTION;
+        suspend = false;
+        custom = false;
     }
 
     enum FuncType {
@@ -202,7 +225,14 @@ public:
 
 class Field : public ClassMember {
 public:
-    Field()
+    Field() :
+        type(),
+        docu_comment(),
+        name(),
+        has_const_value(),
+        const_float_value(),
+        const_int_value(),
+        const_string_value()
     {
         has_const_value = false;
     }
@@ -212,18 +242,25 @@ public:
     std::string name;
     bool has_const_value;
 
-    union {
-        float const_float_value;
-        int const_int_value;
-    };
+    float const_float_value;
+    int const_int_value;
     std::string const_string_value;
+
+private:
+    Field(const Field&);
+    Field& operator=(const Field&);
 };
 
 class Class : public AtomicType {
 public:
+    Class() :
+        members(),
+        super_classes(),
+        sub_classes(),
+        docu_comment()
+    { }
     ~Class() {
-        for(std::vector<ClassMember*>::iterator i = members.begin();
-                i != members.end(); ++i)
+        for(std::vector<ClassMember*>::iterator i = members.begin(); i != members.end(); ++i)
             delete *i;
     }
 
@@ -235,7 +272,14 @@ public:
 
 class Namespace {
 public:
-    Namespace() {
+    Namespace() :
+        functions(),
+        fields(),
+        types(),
+        namespaces(),
+        parent(),
+        name()
+    {
         parent = 0;
     }
     virtual ~Namespace() {
@@ -303,6 +347,10 @@ public:
 
     Namespace* parent;
     std::string name;
+
+private:
+    Namespace(const Namespace&);
+    Namespace& operator=(const Namespace&);
 };
 
 class CompilationUnit : public Namespace {
diff --git a/tools/miniswig/xmlwriter.cpp b/tools/miniswig/xmlwriter.cpp
index 8a1f228ff..5bc0b5389 100644
--- a/tools/miniswig/xmlwriter.cpp
+++ b/tools/miniswig/xmlwriter.cpp
@@ -4,8 +4,12 @@
 #include <sstream>
 #include "xmlwriter.hpp"
 
-XmlWriter::XmlWriter(std::ostream& outstream)
-    : out(outstream), indent(0)
+XmlWriter::XmlWriter(std::ostream& outstream) :
+    out(outstream), 
+    indent(0),
+    closetag(),
+    lasttag(),
+    sections()
 {
 }
 
-- 
2.11.0