From 6b3fac84a01dcd3c4c2db6942193bd03465138ad Mon Sep 17 00:00:00 2001 From: mpv Date: Wed, 16 Sep 2015 08:39:25 +0300 Subject: [PATCH] Issue #948 partial fix: do not hide sketch after apply of extrusion/revolution on it --- src/Model/Model_Update.cpp | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/Model/Model_Update.cpp b/src/Model/Model_Update.cpp index fe86d7a28..5a170ae99 100644 --- a/src/Model/Model_Update.cpp +++ b/src/Model/Model_Update.cpp @@ -216,6 +216,7 @@ void Model_Update::iterateUpdateBreak(std::shared_ptr theFeatu void Model_Update::processOperation(const bool theTotalUpdate, const bool theFinish) { + /* cancel hardcode due to issue 948 if (theFinish) { // the hardcode (DBC asked): hide the sketch referenced by extrusion on apply std::set >::iterator aFIter; @@ -235,7 +236,7 @@ void Model_Update::processOperation(const bool theTotalUpdate, const bool theFin } } } - } + } */ // perform update of everything if needed if (!myIsExecuted) { #ifdef DEB_UPDATE @@ -274,6 +275,11 @@ void Model_Update::updateFeature(FeaturePtr theFeature) if (theFeature->isDisabled()) return; + // to optimize with NDS: how to update sketch state if sketch is not executed on apply of sub-element + //CompositeFeaturePtr aMain = std::dynamic_pointer_cast(theFeature); + //if (aMain.get() && aMain->isSub(theFeature->document()->currentFeature(false))) // do no update the composite that contains the current + // return; + #ifdef DEB_UPDATE std::cout<<"Update Feature "<name()<