diff options
| author | Johan B. C. Engelen <jbc.engelen@swissonline.ch> | 2007-10-20 15:43:07 +0000 |
|---|---|---|
| committer | johanengelen <johanengelen@users.sourceforge.net> | 2007-10-20 15:43:07 +0000 |
| commit | 82a86b15d8cdd5040d65fa27ea59237d6e7c7ed3 (patch) | |
| tree | f2607d956b1ce9c96bf57193b77913e074039ea9 /src/live_effects/effect.cpp | |
| parent | Adding config option to block use of libpoppler-cairo (for misdetected situta... (diff) | |
| download | inkscape-82a86b15d8cdd5040d65fa27ea59237d6e7c7ed3.tar.gz inkscape-82a86b15d8cdd5040d65fa27ea59237d6e7c7ed3.zip | |
LPE: add on-canvas editing of path parameters!
(bzr r3934)
Diffstat (limited to 'src/live_effects/effect.cpp')
| -rw-r--r-- | src/live_effects/effect.cpp | 8 |
1 files changed, 3 insertions, 5 deletions
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 |