diff --git a/src/Mod/PartDesign/Gui/Command.cpp b/src/Mod/PartDesign/Gui/Command.cpp index ce9b46c482..6ee21fdd16 100644 --- a/src/Mod/PartDesign/Gui/Command.cpp +++ b/src/Mod/PartDesign/Gui/Command.cpp @@ -1103,7 +1103,6 @@ CmdPartDesignPad::CmdPartDesignPad() void CmdPartDesignPad::activated(int iMsg) { Q_UNUSED(iMsg); - App::Document *doc = getDocument(); prepareProfileBased(this, "Pad", 10.0); } @@ -1133,7 +1132,6 @@ CmdPartDesignPocket::CmdPartDesignPocket() void CmdPartDesignPocket::activated(int iMsg) { Q_UNUSED(iMsg); - App::Document *doc = getDocument(); prepareProfileBased(this, "Pocket", 5.0); } @@ -1163,7 +1161,6 @@ CmdPartDesignHole::CmdPartDesignHole() void CmdPartDesignHole::activated(int iMsg) { Q_UNUSED(iMsg); - App::Document *doc = getDocument(); PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); @@ -1208,7 +1205,6 @@ CmdPartDesignRevolution::CmdPartDesignRevolution() void CmdPartDesignRevolution::activated(int iMsg) { Q_UNUSED(iMsg); - App::Document *doc = getDocument(); PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); @@ -1265,7 +1261,6 @@ CmdPartDesignGroove::CmdPartDesignGroove() void CmdPartDesignGroove::activated(int iMsg) { Q_UNUSED(iMsg); - App::Document *doc = getDocument(); PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); @@ -1330,7 +1325,6 @@ CmdPartDesignAdditivePipe::CmdPartDesignAdditivePipe() void CmdPartDesignAdditivePipe::activated(int iMsg) { Q_UNUSED(iMsg); - App::Document *doc = getDocument(); PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); @@ -1379,7 +1373,6 @@ CmdPartDesignSubtractivePipe::CmdPartDesignSubtractivePipe() void CmdPartDesignSubtractivePipe::activated(int iMsg) { Q_UNUSED(iMsg); - App::Document *doc = getDocument(); PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); @@ -1428,7 +1421,6 @@ CmdPartDesignAdditiveLoft::CmdPartDesignAdditiveLoft() void CmdPartDesignAdditiveLoft::activated(int iMsg) { Q_UNUSED(iMsg); - App::Document *doc = getDocument(); PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); @@ -1477,7 +1469,6 @@ CmdPartDesignSubtractiveLoft::CmdPartDesignSubtractiveLoft() void CmdPartDesignSubtractiveLoft::activated(int iMsg) { Q_UNUSED(iMsg); - App::Document *doc = getDocument(); PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); @@ -1525,7 +1516,6 @@ CmdPartDesignAdditiveHelix::CmdPartDesignAdditiveHelix() void CmdPartDesignAdditiveHelix::activated(int iMsg) { Q_UNUSED(iMsg); - App::Document *doc = getDocument(); PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); @@ -1600,7 +1590,6 @@ CmdPartDesignSubtractiveHelix::CmdPartDesignSubtractiveHelix() void CmdPartDesignSubtractiveHelix::activated(int iMsg) { Q_UNUSED(iMsg); - App::Document *doc = getDocument(); PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); @@ -1642,9 +1631,6 @@ bool CmdPartDesignSubtractiveHelix::isActive() bool dressupGetSelected(Gui::Command* cmd, const std::string& which, Gui::SelectionObject &selected, bool &useAllEdges, bool& noSelection) { - // No PartDesign feature without Body past FreeCAD 0.16 - App::Document *doc = cmd->getDocument(); - PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); if (!pcActiveBody) @@ -2027,8 +2013,6 @@ CmdPartDesignMirrored::CmdPartDesignMirrored() void CmdPartDesignMirrored::activated(int iMsg) { Q_UNUSED(iMsg); - // No PartDesign feature without Body past FreeCAD 0.16 - App::Document *doc = getDocument(); PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); @@ -2080,8 +2064,6 @@ CmdPartDesignLinearPattern::CmdPartDesignLinearPattern() void CmdPartDesignLinearPattern::activated(int iMsg) { Q_UNUSED(iMsg); - // No PartDesign feature without Body past FreeCAD 0.16 - App::Document *doc = getDocument(); PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); @@ -2135,8 +2117,6 @@ CmdPartDesignPolarPattern::CmdPartDesignPolarPattern() void CmdPartDesignPolarPattern::activated(int iMsg) { Q_UNUSED(iMsg); - // No PartDesign feature without Body past FreeCAD 0.16 - App::Document *doc = getDocument(); PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); @@ -2192,7 +2172,6 @@ CmdPartDesignScaled::CmdPartDesignScaled() void CmdPartDesignScaled::activated(int iMsg) { Q_UNUSED(iMsg); - App::Document *doc = getDocument(); PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); @@ -2235,9 +2214,6 @@ CmdPartDesignMultiTransform::CmdPartDesignMultiTransform() void CmdPartDesignMultiTransform::activated(int iMsg) { Q_UNUSED(iMsg); - // No PartDesign feature without Body past FreeCAD 0.16 - App::Document *doc = getDocument(); - PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); if (!pcActiveBody) diff --git a/src/Mod/PartDesign/Gui/CommandPrimitive.cpp b/src/Mod/PartDesign/Gui/CommandPrimitive.cpp index d7434b0fb3..4a78fcdbe8 100644 --- a/src/Mod/PartDesign/Gui/CommandPrimitive.cpp +++ b/src/Mod/PartDesign/Gui/CommandPrimitive.cpp @@ -248,8 +248,6 @@ CmdPrimtiveCompSubtractive::CmdPrimtiveCompSubtractive() void CmdPrimtiveCompSubtractive::activated(int iMsg) { - App::Document *doc = getDocument(); - PartDesign::Body *pcActiveBody = PartDesignGui::getBody(true); if (!pcActiveBody) diff --git a/src/Mod/Sketcher/App/SketchObject.cpp b/src/Mod/Sketcher/App/SketchObject.cpp index 66151adab1..2518eb21f0 100644 --- a/src/Mod/Sketcher/App/SketchObject.cpp +++ b/src/Mod/Sketcher/App/SketchObject.cpp @@ -4756,8 +4756,6 @@ int SketchObject::addSymmetric(const std::vector& geoIdList, int refGeoId, // no need to check input data validity as this is an sketchobject managed operation. Base::StateLocker lock(managedoperation, true); - const std::vector& geovals = getInternalGeometry(); - const std::vector& constrvals = this->Constraints.getValues(); std::vector newconstrVals(constrvals);