Warning cleanup

diff --git a/src/display/inkscape-cairo.cpp b/src/display/inkscape-cairo.cpp
index 2ca6cb91d0efb3a7e0e849894cf6a44ccd3294d0..9a114f48ed7e2950f3ae2d9817ac6a52055c7b8d 100644 (file)
--- 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 5ddbe8070965a2d50400e86dbf9742ba59d68d6f..7eb606db9d7b708d2f67edd248d3f8e9cd1dec87 100644 (file)
--- 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 b790f2bb54f788597596c82e5ac30424b856ce80..57d77e3a4f6e5f43bcf4902f5765193c5d666ed1 100644 (file)
--- 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 11e5b504871bfc13a7968f2c355f069ff16a2ac0..3848730187dc9cc2ff79d306fb278c0640ca9ae8 100644 (file)
--- 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 a501fbb8fa1c4e631e6ce0448130a2bdae1e7cea..60e307bdbb5d85be805b6f8127abf77ad775b61b 100644 (file)
--- 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 220b618f7569d2595276571108a4c0c93e477159..69e2520c2a5f1e21625ffcaaddfc82551d375052 100644 (file)
--- 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 a8d357415c739692077607f190580a2d4ba2753e..06e527ffd56585a48c1f5403f33ed1568ff809bb 100644 (file)
--- 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 520d7f40402ddac306e3ceca4a3341ad2754a3d9..c0aaaaa66f14152d679290bffc416d28a6548a10 100644 (file)
--- 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