ros-planning / navigation_msgs

Message packages required by the navigation stack
42 stars 53 forks source link

Enable -Wall, -Wextra, and -Wpedantic in move_base_msgs #21

Closed audrow closed 4 years ago

audrow commented 4 years ago

Signed-off-by: Audrow Nash audrow.nash@gmail.com

mikeferguson commented 4 years ago

Why? These are just messages.

audrow commented 4 years ago

For Galactic, I'm adding warnings to almost all of the ROS2 packages. In discussing with the Open Robotics team, we thought it would be good to add warnings to messages since they generate code. After bringing it up to the team again, we agree it doesn't make sense to add them to interface packages.