Comment 7 for bug 1171109

Revision history for this message
insaner (insaner) wrote :

hi john-smithi,

i tested the patch and it almost works.. i had been trying something similar, ie simply commenting out the line that reads

"if (gr->isSolid() || gr->getVector()->isSolid()) {"
(and the corresponding closing brace)

 in document.cpp, and while that prevented the crashes caused by pasting, when i do the alt+scroll part after pasting it would still crash.

with your patch, the pasting doesnt cause a crash, but it also doesnt properly paste the object (specifically, the objects that use the swatch named "light" do not show up, though in outline mode you can see that they are there), and doing the alt+scroll where those swatched objects/paths should be, triggers the crash.

using the trick above (commenting out the line and its corresponding brace) makes the alt+scroll problem worse, but it prevents the duplication of the swatches..