patch: crash on layer delete
Hello, regarding the recent 'crash on layer delete' I propose a patch that at least handles the simple case of that bug. As this is not my field, I'd ask you to review it.
The idea is to setCurrentLayer() after the layer is deleted, and not before. setCurrentLayer() emits the layerChanged signal which is listened to by the layer selector widget.
ralf
--- verbs.cpp~ 2005-10-31 10:03:13.000000000 +0100 +++ verbs.cpp 2005-10-31 12:18:15.000000000 +0100 @@ -1123,11 +1123,11 @@ if (!survivor) { survivor = Inkscape::previous_layer(dt->currentRoot(), old_layer); } + old_layer->deleteObject(); + sp_object_unref(old_layer, NULL); if (survivor) { dt->setCurrentLayer(survivor); } - old_layer->deleteObject(); - sp_object_unref(old_layer, NULL);
sp_document_done(SP_DT_DOCUMENT(dt));
participants (2)
-
unknown@example.com
-
Ralf Stephan