-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RSDK-3008 - add misc-unused-params #216
Conversation
@@ -24,7 +24,8 @@ class GenericComponent : public Component { | |||
/// @brief Send/receive arbitrary commands to the resource. | |||
/// @param command the command to execute. | |||
/// @return The result of the executed command. | |||
virtual AttributeMap do_command(AttributeMap command) = 0; | |||
// CR erodkin: flyby | |||
virtual AttributeMap do_command(const AttributeMap& command) = 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(flyby) make arg const&
.
this->peek_current_pose = destination; | ||
this->peek_component_name = component_name; | ||
this->peek_slam_name = slam_name; | ||
this->current_location.pose = destination; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(flyby) here and below, move
ing these args is meaningless because they're const&
, and we can't make them not const&
here without modifying the signature in the Motion
base class. So, we get rid of the move
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.