[Part]code review changes for FeatureScale

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;

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};

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,

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;