Skip to content

Commit

Permalink
Remove individual mutable methods for Roll, Pitch, Yaw.
Browse files Browse the repository at this point in the history
Signed-off-by: pxalcantara <[email protected]>
  • Loading branch information
pxalcantara committed Jul 1, 2020
1 parent c3e9ba9 commit 8b4f417
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 26 deletions.
25 changes: 0 additions & 25 deletions include/ignition/math/Pose3.hh
Original file line number Diff line number Diff line change
Expand Up @@ -436,15 +436,6 @@ namespace ignition
return this->q.Roll();
}

/// \brief Get the mutable Roll value of the rotation.
/// \return Roll value of the orientation.
/// \note The return is made by value since
/// Quaternion<T>.Roll() is already a reference.
public: inline T &Roll()
{
return this->q.Roll();
}

/// \brief Get the Pitch value of the rotation.
/// \return Pitch value of the orientation.
/// \note The return is made by value since
Expand All @@ -454,13 +445,6 @@ namespace ignition
return this->q.Pitch();
}

/// \brief Get the mutable Pitch value of the rotation.
/// \return Pitch value of the orientation.
public: inline T &Pitch()
{
return this->q.Pitch();
}

/// \brief Get the Yaw value of the rotation.
/// \return Yaw value of the orientation.
/// \note The return is made by value since
Expand All @@ -470,15 +454,6 @@ namespace ignition
return this->q.Yaw();
}

/// \brief Get the mutable Yaw value of the rotation.
/// \return Yaw value of the orientation.
/// \note The return is made by value since
/// Quaternion<T>.Yaw() is already a reference.
public: inline T &Yaw()
{
return this->q.Yaw();
}

/// \brief Stream insertion operator
/// \param[in] _out output stream
/// \param[in] _pose pose to output
Expand Down
2 changes: 1 addition & 1 deletion src/Pose_TEST.cc
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ TEST(PoseTest, ConstPoseElements)
}

/////////////////////////////////////////////////
TEST(PoseTest, MutablePoseElements)
TEST(PoseTest, SetPoseElements)
{
math::Pose3d pose(1, 2, 3, 1.57, 1, 2);
EXPECT_DOUBLE_EQ(pose.X(), 1);
Expand Down

0 comments on commit 8b4f417

Please sign in to comment.