search

ros / console_bridge

A ROS-independent package for logging that seamlessly pipes into rosconsole/rosout for ROS-dependent packages.
BSD 3-Clause "New" or "Revised" License
22 stars 62 forks source link

Fix MSVC compiler warnings #39

Closed scpeters closed 8 years ago

scpeters commented 8 years ago

I've fixed the MSVC compiler errors and added the -WX (equivalent to -Werror) flag to the appveyor build, so we will notice if new compiler warnings are introduced.

This isn't urgent like the previous pull requests that involved build failures, so I will wait longer for review and +1's.

dirk-thomas commented 8 years ago

Looks good to me: http://ci.ros2.org/job/ci_windows/967/warnings34Result/fixed/