18 Jan
2014
18 Jan
'14
11:33 p.m.
1. Please adhere to the coding style... http://www.inkscape.org/en/develop/coding-style/ Specifically, there's wrong spacing around 'if' and the variable names are incorrect. I know this is pedantic but it makes reading the code later much easier.
2. The fix itself looks OK. A more advanced solution would be to resolve an absolute path to which the reference points (this is probably already done somewhere in this code) and keep a cache of read-only SPDocument for each ref, either on the top-level Inkscape object or in the SPDesktop object, so that each referred image has its SP tree constructed only once.
3. Does this also solve the problem of svg:image rasterizing the contents of referred SVG images?
Regards, Krzysztof