5 files changed, 15 insertions, 4 deletions
diff --git a/src/extension/internal/pov-out.cpp b/src/extension/internal/pov-out.cpp
index d501777b5..3d149928f 100644
--- a/src/extension/internal/pov-out.cpp
+++ b/src/extension/internal/pov-out.cpp
@@ -95,6 +95,18 @@ static double effective_opacity(SPItem const *item)
 //# OUTPUT FORMATTING
 //########################################################################
 
+PovOutput::PovOutput() :
+    outbuf (),
+    nrNodes (0),
+    nrSegments (0),
+    nrShapes (0),
+    idIndex (0),
+    minx (0),
+    miny (0),
+    maxx (0),
+    maxy (0)
+{
+}
 
 /**
  * We want to control floating output format
diff --git a/src/extension/internal/pov-out.h b/src/extension/internal/pov-out.h
index 0c3b73a7f..2381a974c 100644
--- a/src/extension/internal/pov-out.h
+++ b/src/extension/internal/pov-out.h
@@ -42,6 +42,8 @@ class PovOutput : public Inkscape::Extension::Implementation::Implementation
 
 public:
 
+    PovOutput();
+    
     /**
      * Our internal String definition
      */
diff --git a/src/extension/param/notebook.cpp b/src/extension/param/notebook.cpp
index 6bb3cda80..b1ca359cd 100644
--- a/src/extension/param/notebook.cpp
+++ b/src/extension/param/notebook.cpp
@@ -141,7 +141,6 @@ ParamNotebookPage::makepage (Inkscape::XML::Node * in_repr, Inkscape::Extension:
     const char * desc;
     const char * scope_str;
     Parameter::_scope_t scope = Parameter::SCOPE_USER;
-	bool gui_hidden = false;
 	const char * gui_hide;
 	const char * gui_tip;
 
@@ -160,7 +159,6 @@ ParamNotebookPage::makepage (Inkscape::XML::Node * in_repr, Inkscape::Extension:
 	if (gui_hide != NULL) {
 		if (strcmp(gui_hide, "1") == 0 ||
 			strcmp(gui_hide, "true") == 0) {
-			gui_hidden = true;
 		}
 		/* else stays false */
 	}
diff --git a/src/extension/patheffect.cpp b/src/extension/patheffect.cpp
index a3094d536..bedab7fd8 100644
--- a/src/extension/patheffect.cpp
+++ b/src/extension/patheffect.cpp
@@ -44,7 +44,7 @@ PathEffect::processPathEffects (SPDocument * doc, Inkscape::XML::Node * path)
     gchar ** patheffects = g_strsplit(patheffectlist, ";", 128);
     Inkscape::XML::Node * defs = doc->getDefs()->getRepr();
 
-    for (int i = 0; patheffects[i] != NULL && i < 128; i++) {
+    for (int i = 0; (i < 128) && (patheffects[i] != NULL); i++) {
         gchar * patheffect = patheffects[i];
 
         // This is weird, they should all be references... but anyway
diff --git a/src/extension/system.cpp b/src/extension/system.cpp
index 82d0a726f..a959913f6 100644
--- a/src/extension/system.cpp
+++ b/src/extension/system.cpp
@@ -219,7 +219,6 @@ save(Extension *key, SPDocument *doc, gchar const *filename, bool setextension,
     if (!dynamic_cast<Output *>(omod)) {
         g_warning("Unable to find output module to handle file: %s\n", filename);
         throw Output::no_extension_found();
-        return;
     }
 
     omod->set_state(Extension::STATE_LOADED);