From 0da7d01ffa1590174d76a7726da59541e5e3fe61 Mon Sep 17 00:00:00 2001 From: Olli Parviainen Date: Sat, 8 Dec 2018 23:11:46 +0200 Subject: Accelerate boolean path operation for larger paths by disabling redrawing the canvas during running of the break-apart algorithm. This change reduces duration of e.g. cut-path operation to a fraction of the original when working with a large path set. Signed-off-by: Olli Parviainen --- src/splivarot.cpp | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'src/splivarot.cpp') diff --git a/src/splivarot.cpp b/src/splivarot.cpp index b986af32e..1f4b4e9b5 100644 --- a/src/splivarot.cpp +++ b/src/splivarot.cpp @@ -38,6 +38,8 @@ #include "text-editing.h" #include "verbs.h" +#include "display/sp-canvas.h" + #include "helper/geom.h" #include "livarot/Path.h" @@ -321,7 +323,11 @@ BoolOpErrors Inkscape::ObjectSet::pathBoolOp(bool_op bop, const bool skip_undo, { if (nullptr != desktop() && !checked) { SPDocument *doc = desktop()->getDocument(); + // don't redraw the canvas during the operation as that can remarkably slow down the progress + desktop()->getCanvas()->_drawing_disabled = true; BoolOpErrors returnCode = ObjectSet::pathBoolOp(bop, true, true); + desktop()->getCanvas()->_drawing_disabled = false; + switch(returnCode) { case ERR_TOO_LESS_PATHS_1: boolop_display_error_message(desktop(), _("Select at least 1 path to perform a boolean union.")); -- cgit v1.2.3