On Mon, 28 Feb 2005 19:00:03 -0400, bulia byak <buliabyak@...400...> wrote:
On Mon, 28 Feb 2005 17:55:48 -0500, mental@...3... <mental@...3...> wrote:
[ I'm a bit surprised that having the redundant root nodes isn't causing a parse failure, though (it should, IMO)... ]
it's not another root - it's nested _under_ the original <inkscape> as its last child
By the way, have you filed a bug so we can track this?
i will if it's not fixed soon enough
Ok, it was an improper conversion on my part and I've fixed it in current CVS. Thanks Buila for finding this.
diff -u -3 -r1.85 inkscape.cpp --- src/inkscape.cpp 27 Feb 2005 02:26:26 -0000 1.85 +++ src/inkscape.cpp 28 Feb 2005 23:47:16 -0000 @@ -629,7 +629,7 @@ return FALSE; }
- config->root()->mergeFrom(doc, "id"); + config->root()->mergeFrom(doc->root(), "id");
Luckily, there was only one use of sp_repr_document_merge that I converted.
There may be another issue that Ill look at later. After a default preferences is written, the next time inkscape exits, it will expand the preferences settings.
< <group id="printing"> < <settings id="ps"/> < <group id="debug" add-label-comments="0"/> < </group>
will become:
<group id="printing"> <settings id="ps" /> <group add-label-comments="0" id="debug" />
</group>
It seems to stay stable after that.
Cheers,