From 2d2706a9d621c0da51b7c4178fd5c7c5e4765122 Mon Sep 17 00:00:00 2001 From: joncruz Date: Fri, 4 Jul 2008 21:43:41 +0000 Subject: [PATCH] Warning cleanup --- src/display/inkscape-cairo.cpp | 2 +- src/livarot/AVL.cpp | 2 +- src/livarot/float-line.cpp | 3 ++- src/live_effects/effect.cpp | 4 ++-- src/live_effects/lpe-mirror_reflect.cpp | 2 +- src/live_effects/parameter/point.cpp | 6 +++--- src/live_effects/parameter/pointparam-knotholder.cpp | 3 ++- src/ui/widget/selected-style.cpp | 2 +- 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 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(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 -- 2.30.2