From db537b8f90e7b6ad5fee93e6f1f9853ed5cc13fa Mon Sep 17 00:00:00 2001 From: wmayer Date: Thu, 5 Nov 2020 15:19:39 +0100 Subject: [PATCH] Sketcher: [skip ci] use delGeometries to delete all superfluous geometry in one go after changing multiplicity --- src/Mod/Sketcher/App/SketchObject.cpp | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/src/Mod/Sketcher/App/SketchObject.cpp b/src/Mod/Sketcher/App/SketchObject.cpp index 398503a601..fdbc880252 100644 --- a/src/Mod/Sketcher/App/SketchObject.cpp +++ b/src/Mod/Sketcher/App/SketchObject.cpp @@ -5488,14 +5488,11 @@ bool SketchObject::modifyBSplineKnotMultiplicity(int GeoId, int knotIndex, int m // Trigger update now // Update geometry indices and rebuild vertexindex now via onChanged, so that ViewProvider::UpdateData is triggered. - Geometry.touch(); - - std::sort (delGeoId.begin(), delGeoId.end()); - - if (delGeoId.size()>0) { - for (std::vector::reverse_iterator it=delGeoId.rbegin(); it!=delGeoId.rend(); ++it) { - delGeometry(*it,false); - } + if (!delGeoId.empty()) { + delGeometries(delGeoId); + } + else { + Geometry.touch(); } // * DOCUMENTING OCC ISSUE OCC < 6.9.0