From 82a86b15d8cdd5040d65fa27ea59237d6e7c7ed3 Mon Sep 17 00:00:00 2001 From: "Johan B. C. Engelen" Date: Sat, 20 Oct 2007 15:43:07 +0000 Subject: LPE: add on-canvas editing of path parameters! (bzr r3934) --- src/live_effects/effect.cpp | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'src/live_effects/effect.cpp') diff --git a/src/live_effects/effect.cpp b/src/live_effects/effect.cpp index 1c0f5318f..cfabc88b4 100644 --- a/src/live_effects/effect.cpp +++ b/src/live_effects/effect.cpp @@ -186,14 +186,15 @@ Effect::readallParameters(Inkscape::XML::Node * repr) const gchar * key = (*it).first.c_str(); const gchar * value = repr->attribute(key); if(value) { - setParameter(repr, key, NULL, value); + setParameter(key, value); } it++; } } +/* This function does not and SHOULD NOT write to XML */ void -Effect::setParameter(Inkscape::XML::Node * repr, const gchar * key, const gchar * old_value, const gchar * new_value) +Effect::setParameter(const gchar * key, const gchar * new_value) { Glib::ustring stringkey(key); @@ -203,9 +204,6 @@ Effect::setParameter(Inkscape::XML::Node * repr, const gchar * key, const gchar bool accepted = it->second->param_readSVGValue(new_value); if (!accepted) { g_warning("Effect::setParameter - '%s' not accepted for %s", new_value, key); - // change was not accepted, so change it back. - // think: can this backfire and create infinite loop when started with unacceptable old_value? - // repr->setAttribute(key, old_value); } } else { // set default value -- cgit v1.2.3