RobotWebTools / rclnodejs

Node.js version of ROS 2.0 client
https://docs.ros.org/en/humble/Concepts/Basic/About-Client-Libraries.html?highlight=rclnodejs#community-maintained
Apache License 2.0
316 stars 70 forks source link

Many compiler warnings due to the new Lifecycle API #741

Closed felixdivo closed 3 years ago

felixdivo commented 3 years ago

722 introduced quite a few warnings, see this recent build from master.

They all seem to be quire easy to fix though (mainly warning: variable ‘currentContent’ set but not used).

wayneparrott commented 3 years ago

@felixdivo Thx. My BAD! I've gotten too used to seeing compiler warnings with node-gyp build. I'll submit a fix PR asap.