generate_parameter_library icon indicating copy to clipboard operation
generate_parameter_library copied to clipboard

`std::move()` improvements suggested by `clang-tidy`

Open AndyZe opened this issue 1 year ago • 0 comments

/home/andy/ws_moveit/build/moveit_servo/moveit_servo_lib_parameters/include/moveit_servo_lib_parameters.hpp:167:17: warning: parameter 'logger' is passed by value and only copied once; consider moving it to avoid unnecessary copies [performance-unnecessary-value-param]
   17 |       logger_ = logger;
      |                 ^     
      |                 std::move( )
/home/andy/ws_moveit/build/moveit_servo/moveit_servo_lib_parameters/include/moveit_servo_lib_parameters.hpp:988:19: warning: parameter 'updated_params' is passed by value and only copied once; consider moving it to avoid unnecessary copies [performance-unnecessary-value-param]
  988 |         params_ = updated_params;
      |                   ^             
      |                   std::move(    )
      ```

AndyZe avatar Aug 11 '24 13:08 AndyZe