Warning cleanup

(bzr r6156)

8 files changed, 13 insertions, 11 deletions

diff --git a/src/display/inkscape-cairo.cpp b/src/display/inkscape-cairo.cpp
index 2ca6cb91d..9a114f48e 100644
--- a/src/display/inkscape-cairo.cpp
+++ b/src/display/inkscape-cairo.cpp
@@ -55,7 +55,7 @@ nr_create_cairo_context_for_data (NRRectL *area, NRRectL *buf_area, unsigned cha
 
 /** Creates a cairo context to render to the given SPCanvasBuf on the given area */
 cairo_t *
-nr_create_cairo_context_canvasbuf (NRRectL *area, SPCanvasBuf *b)
+nr_create_cairo_context_canvasbuf (NRRectL */*area*/, SPCanvasBuf *b)
 {
     return nr_create_cairo_context_for_data (&(b->rect), &(b->rect), b->buf, b->buf_rowstride);
 }
diff --git a/src/livarot/AVL.cpp b/src/livarot/AVL.cpp
index 5ddbe8070..7eb606db9 100644
--- a/src/livarot/AVL.cpp
+++ b/src/livarot/AVL.cpp
@@ -69,7 +69,7 @@ AVLTree *AVLTree::leaf(AVLTree *from, Side s)
     return NULL;
 }
 
-AVLTree *AVLTree::leafFromDad(AVLTree *from, Side s)
+AVLTree *AVLTree::leafFromDad(AVLTree */*from*/, Side s)
 {
     if (son[s]) {
 	return son[s]->leafFromDad(this, s);
diff --git a/src/livarot/float-line.cpp b/src/livarot/float-line.cpp
index b790f2bb5..57d77e3a4 100644
--- a/src/livarot/float-line.cpp
+++ b/src/livarot/float-line.cpp
@@ -306,8 +306,9 @@ int FloatLigne::AppendBord(float spos, float sval, float epos, float eval, float
 
 
 // insertion in a boubly-linked list. nothing interesting here
-void FloatLigne::InsertBord(int no, float p, int guess)
+void FloatLigne::InsertBord(int no, float /*p*/, int guess)
 {
+// TODO check if ignoring p is bad
     if ( no < 0 || no >= int(bords.size()) ) {
         return;
     }
diff --git a/src/live_effects/effect.cpp b/src/live_effects/effect.cpp
index 11e5b5048..384873018 100644
--- a/src/live_effects/effect.cpp
+++ b/src/live_effects/effect.cpp
@@ -295,7 +295,7 @@ Effect::writeParamsToSVG() {
  * your LPE. But don't forget to call the parent method so that done_pathparam_set is set to true!
  */
 void
-Effect::acceptParamPath (SPPath *param_path) {
+Effect::acceptParamPath (SPPath */*param_path*/) {
     done_pathparam_set = true;
 }
 
@@ -460,7 +460,7 @@ Effect::addHelperPaths(SPLPEItem *lpeitem, SPDesktop *desktop)
 }
 
 void
-Effect::addHelperPathsImpl(SPLPEItem *lpeitem, SPDesktop *desktop)
+Effect::addHelperPathsImpl(SPLPEItem */*lpeitem*/, SPDesktop */*desktop*/)
 {
     // if this method is overloaded in derived classes, provides_own_flash_paths will be true
     provides_own_flash_paths = false;
diff --git a/src/live_effects/lpe-mirror_reflect.cpp b/src/live_effects/lpe-mirror_reflect.cpp
index a501fbb8f..60e307bdb 100644
--- a/src/live_effects/lpe-mirror_reflect.cpp
+++ b/src/live_effects/lpe-mirror_reflect.cpp
@@ -83,7 +83,7 @@ LPEMirrorReflect::doEffect_path (std::vector<Geom::Path> const & path_in)
     m = m * m2.inverse();
     m = m * m1;
 
-    for (int i = 0; i < path_in.size(); ++i) {
+    for (int i = 0; i < static_cast<int>(path_in.size()); ++i) {
         path_out.push_back(path_in[i] * m);
     }
 
diff --git a/src/live_effects/parameter/point.cpp b/src/live_effects/parameter/point.cpp
index 220b618f7..69e2520c2 100644
--- a/src/live_effects/parameter/point.cpp
+++ b/src/live_effects/parameter/point.cpp
@@ -78,7 +78,7 @@ PointParam::param_getSVGValue() const
 }
 
 Gtk::Widget *
-PointParam::param_newWidget(Gtk::Tooltips * tooltips)
+PointParam::param_newWidget(Gtk::Tooltips * /*tooltips*/)
 {
     Inkscape::UI::Widget::RegisteredTransformedPoint * pointwdg = Gtk::manage(
         new Inkscape::UI::Widget::RegisteredTransformedPoint( param_label,
@@ -153,7 +153,7 @@ PointParam::set_oncanvas_looks(SPKnotShapeType shape, SPKnotModeType mode, guint
 }
 
 void
-PointParam::knot_set(NR::Point const &p, NR::Point const &origin, guint state)
+PointParam::knot_set(NR::Point const &p, NR::Point const &/*origin*/, guint /*state*/)
 {
     param_setValue(p.to_2geom());
     sp_lpe_item_update_patheffect(SP_LPE_ITEM(item), false, false);
@@ -166,7 +166,7 @@ PointParam::knot_get()
 }
 
 void
-PointParam::knot_click(guint state)
+PointParam::knot_click(guint /*state*/)
 {
     g_print ("This is the handle associated to the parameter '%s'\n", param_key.c_str());
 }
diff --git a/src/live_effects/parameter/pointparam-knotholder.cpp b/src/live_effects/parameter/pointparam-knotholder.cpp
index a8d357415..06e527ffd 100644
--- a/src/live_effects/parameter/pointparam-knotholder.cpp
+++ b/src/live_effects/parameter/pointparam-knotholder.cpp
@@ -72,12 +72,13 @@ KnotHolderEntityPointParam::knot_get() {
 }
 
 void
-KnotHolderEntityPointParam::knot_set(NR::Point const &p, NR::Point const &origin, guint state) {
+KnotHolderEntityPointParam::knot_set(NR::Point const &/*p*/, NR::Point const &/*origin*/, guint /*state*/) {
 }
 
 void
 PointParamKnotHolder::add_knot (
     Geom::Point         & p,
+// TODO: check if knot_click being ignored is bad:
     PointParamKnotHolderClickedFunc knot_click,
     SPKnotShapeType     shape,
     SPKnotModeType      mode,
diff --git a/src/ui/widget/selected-style.cpp b/src/ui/widget/selected-style.cpp
index 520d7f404..c0aaaaa66 100644
--- a/src/ui/widget/selected-style.cpp
+++ b/src/ui/widget/selected-style.cpp
@@ -1343,7 +1343,7 @@ RotateableStrokeWidth::~RotateableStrokeWidth() {
 }
 
 double
-RotateableStrokeWidth::value_adjust(double current, double by, guint modifier, bool final)
+RotateableStrokeWidth::value_adjust(double current, double by, guint /*modifier*/, bool final)
 {
     double newval;
     // by is -1..1