diff --git a/src/Mod/Part/App/FeatureScale.cpp b/src/Mod/Part/App/FeatureScale.cpp index 24ca43dfc4..d29e8342c1 100644 --- a/src/Mod/Part/App/FeatureScale.cpp +++ b/src/Mod/Part/App/FeatureScale.cpp @@ -127,7 +127,7 @@ TopoShape Scale::nonuniformScale(const TopoShape& source, const Scale::ScalePara mat.SetValue(3,4,matScale[2][3]); // this copy step seems to eliminate Part.OCCError: gp_GTrsf::Trsf() - non-orthogonal GTrsf - // which may to be related to the tesselation of the input shape. See Github issue #9651 + // which may to be related to the tessellation of the input shape. See Github issue #9651 // for more detail. BRepBuilderAPI_Copy copier(source.getShape()); TopoShape transTopo; diff --git a/src/Mod/Part/App/FeatureScale.h b/src/Mod/Part/App/FeatureScale.h index f0efb175e7..bf0a547fa4 100644 --- a/src/Mod/Part/App/FeatureScale.h +++ b/src/Mod/Part/App/FeatureScale.h @@ -49,11 +49,10 @@ public: /** * @brief The ScaleParameters struct is supposed to be filled with final - * scale parameters, after resolving links, applying mode logic, - * reversing, etc., and be passed to scaleShape. + * scale parameters and be passed to scaleShape. */ struct ScaleParameters { - bool uniform; + bool uniform{true}; double uniformScale{1.0}; double XScale{1.0}; double YScale{1.0}; diff --git a/src/Mod/Part/Gui/DlgScale.cpp b/src/Mod/Part/Gui/DlgScale.cpp index 6334b779eb..05fbdf1aa5 100644 --- a/src/Mod/Part/Gui/DlgScale.cpp +++ b/src/Mod/Part/Gui/DlgScale.cpp @@ -70,14 +70,6 @@ DlgScale::DlgScale(QWidget* parent, Qt::WindowFlags fl) findShapes(); } -/* - * Destroys the object and frees any allocated resources - */ -DlgScale::~DlgScale() -{ - // no need to delete child widgets, Qt does it all for us -} - void DlgScale::setupConnections() { connect(ui->rbUniform, &QRadioButton::toggled, diff --git a/src/Mod/Part/Gui/DlgScale.h b/src/Mod/Part/Gui/DlgScale.h index a8eb9132f2..3dc1962c36 100644 --- a/src/Mod/Part/Gui/DlgScale.h +++ b/src/Mod/Part/Gui/DlgScale.h @@ -41,7 +41,7 @@ class DlgScale : public QDialog public: DlgScale(QWidget* parent = nullptr, Qt::WindowFlags fl = Qt::WindowFlags()); - ~DlgScale() override; + ~DlgScale() = default; void accept() override; void apply(); void reject() override;