diff options
| author | Marc Jeanmougin <marc@jeanmougin.fr> | 2015-07-14 11:04:32 +0000 |
|---|---|---|
| committer | Marc Jeanmougin <marcjeanmougin@free.fr> | 2015-07-14 11:04:32 +0000 |
| commit | 676707120d5527ff8c8c8c89ea33d1eadf9cf1b4 (patch) | |
| tree | c2d91ac1c2c2453f8db7a5bed7b9f52b67320178 /src/extension/execution-env.cpp | |
| parent | fix "arrange" tool (typo) (diff) | |
| download | inkscape-676707120d5527ff8c8c8c89ea33d1eadf9cf1b4.tar.gz inkscape-676707120d5527ff8c8c8c89ea33d1eadf9cf1b4.zip | |
Fix for circular references detection in almost all cases, fixing https://bugs.launchpad.net/inkscape/+bug/167247 and a few of its duplicates.
This fix is aimed at preventing any sort of circular references with the URIReference::_acceptObject method, checking the absence of loops in the reference+child tree.
There can be some performance improvements done if we add a pointer from cloned sub-objects to their origin sub-object.
The remaining cases that are not fixed can involve non-trivial loops using one or more "url()" stylesheet references. Being able to take them into account would require a non-obvious style.cpp refactoring making use of URIReference for this kind of reference (and not handling manually the signals in the styling code, which would probably be a good thing to do anyway)
(bzr r14245)
Diffstat (limited to 'src/extension/execution-env.cpp')
0 files changed, 0 insertions, 0 deletions