Part: [skip ci] make arguments const references

FreeCAD · Nov 3, 2020 · f4aceec · f4aceec
1 parent 0324d2d
commit f4aceec
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 12 deletions.
diff --git a/src/Mod/Part/App/Geometry.cpp b/src/Mod/Part/App/Geometry.cpp
@@ -383,19 +383,19 @@ Geometry *Geometry::clone(void) const
     return cpy;
 }
 
-void Geometry::mirror(Base::Vector3d point)
+void Geometry::mirror(const Base::Vector3d& point)
 {
     gp_Pnt pnt(point.x, point.y, point.z);
     handle()->Mirror(pnt);
 }
 
-void Geometry::mirror(Base::Vector3d point, Base::Vector3d dir)
+void Geometry::mirror(const Base::Vector3d& point, const Base::Vector3d& dir)
 {
     gp_Ax1 ax1(gp_Pnt(point.x,point.y,point.z), gp_Dir(dir.x,dir.y,dir.z));
     handle()->Mirror(ax1);
 }
 
-void Geometry::rotate(Base::Placement plm)
+void Geometry::rotate(const Base::Placement& plm)
 {
     Base::Rotation rot(plm.getRotation());
     Base::Vector3d pnt, dir;
@@ -409,13 +409,13 @@ void Geometry::rotate(Base::Placement plm)
     handle()->Rotate(ax1, angle);
 }
 
-void Geometry::scale(Base::Vector3d vec, double scale)
+void Geometry::scale(const Base::Vector3d& vec, double scale)
 {
     gp_Pnt pnt(vec.x, vec.y, vec.z);
     handle()->Scale(pnt, scale);
 }
 
-void Geometry::transform(Base::Matrix4D mat)
+void Geometry::transform(const Base::Matrix4D& mat)
 {
     gp_Trsf trf;
     trf.SetValues(mat[0][0],mat[0][1],mat[0][2],mat[0][3],
@@ -428,7 +428,7 @@ void Geometry::transform(Base::Matrix4D mat)
     handle()->Transform(trf);
 }
 
-void Geometry::translate(Base::Vector3d vec)
+void Geometry::translate(const Base::Vector3d& vec)
 {
     gp_Vec trl(vec.x, vec.y, vec.z);
     handle()->Translate(trl);

diff --git a/src/Mod/Part/App/Geometry.h b/src/Mod/Part/App/Geometry.h
@@ -111,12 +111,12 @@ class PartExport Geometry: public Base::Persistence
     void deleteExtension(Base::Type type);
     void deleteExtension(std::string name);
 
-    void mirror(Base::Vector3d point);
-    void mirror(Base::Vector3d point, Base::Vector3d dir);
-    void rotate(Base::Placement plm);
-    void scale(Base::Vector3d vec, double scale);
-    void transform(Base::Matrix4D mat);
-    void translate(Base::Vector3d vec);
+    void mirror(const Base::Vector3d& point);
+    void mirror(const Base::Vector3d& point, const Base::Vector3d& dir);
+    void rotate(const Base::Placement& plm);
+    void scale(const Base::Vector3d& vec, double scale);
+    void transform(const Base::Matrix4D& mat);
+    void translate(const Base::Vector3d& vec);
 
 protected:
     /// create a new tag for the geometry object