diff --git a/Base/Collision/CollisionDetection/DataStructures/imstkSpatialHashTableSeparateChaining.cpp b/Base/Collision/CollisionDetection/DataStructures/imstkSpatialHashTableSeparateChaining.cpp index be688ff604e2f8997dc9e928407c7c36d0470f2c..a8f6d6db4748cf89e0c63d8575b81b0cbf1ff9bb 100644 --- a/Base/Collision/CollisionDetection/DataStructures/imstkSpatialHashTableSeparateChaining.cpp +++ b/Base/Collision/CollisionDetection/DataStructures/imstkSpatialHashTableSeparateChaining.cpp @@ -32,7 +32,7 @@ SpatialHashTableSeparateChaining::SpatialHashTableSeparateChaining() } void -SpatialHashTableSeparateChaining::insertPoints(StdVectorOfVec3d points) +SpatialHashTableSeparateChaining::insertPoints(const StdVectorOfVec3d& points) { // TODO: make more efficient for (auto i = 0; i < points.size(); i++) @@ -42,7 +42,7 @@ SpatialHashTableSeparateChaining::insertPoints(StdVectorOfVec3d points) } void -SpatialHashTableSeparateChaining::insertPoint(Vec3d point) +SpatialHashTableSeparateChaining::insertPoint(const Vec3d& point) { PointEntry entry; entry.point = point; @@ -55,14 +55,14 @@ SpatialHashTableSeparateChaining::insertPoint(Vec3d point) } void -SpatialHashTableSeparateChaining::clearTable() +SpatialHashTableSeparateChaining::clear() { m_table->clear(); m_currentID = 0; } std::vector<Vec3d> -SpatialHashTableSeparateChaining::getPointsInAABB(Vec3d corner1, Vec3d corner2) +SpatialHashTableSeparateChaining::getPointsInAABB(const Vec3d& corner1, const Vec3d& corner2) { auto min_x = std::fmin(corner1.x(), corner2.x()); auto max_x = std::fmax(corner1.x(), corner2.x()); diff --git a/Base/Collision/CollisionDetection/DataStructures/imstkSpatialHashTableSeparateChaining.h b/Base/Collision/CollisionDetection/DataStructures/imstkSpatialHashTableSeparateChaining.h index 2a338c12d575a7b882dfdada7947071bb57d008d..8eb2b636f884c2cde6f408fbd569dfaf2b4069b6 100644 --- a/Base/Collision/CollisionDetection/DataStructures/imstkSpatialHashTableSeparateChaining.h +++ b/Base/Collision/CollisionDetection/DataStructures/imstkSpatialHashTableSeparateChaining.h @@ -95,13 +95,13 @@ public: /// \brief Insert an array of points /// \param points An array of point /// - void insertPoints(StdVectorOfVec3d points); + void insertPoints(const StdVectorOfVec3d& points); /// /// \brief Insert an array of points /// \param point A point /// - void insertPoint(Vec3d point); + void insertPoint(const Vec3d& point); /// /// \brief Sets the max load factor @@ -114,12 +114,12 @@ public: /// \param corner1 One corner to the box /// \param corner2 The other corner to the box /// - std::vector<Vec3d> getPointsInAABB(Vec3d corner1, Vec3d corner2); + std::vector<Vec3d> getPointsInAABB(const Vec3d& corner1, const Vec3d& corner2); /// /// \brief Clears the table /// - void clearTable(); + void clear(); /// /// \brief Protected constructor