search

ros2 / rmw_zenoh

RMW for ROS 2 using Zenoh as the middleware
Apache License 2.0
142 stars 29 forks source link

Move all private functions to use an anonymous namespace. #233

Closed clalancette closed 3 days ago

clalancette commented 3 days ago

This is just so we consistently use C++-style anonymous namespaces, instead of C-style "static" functions. No functional change.

This should fix #124

Timple commented 3 days ago

Just curious, what changed your mind with regards to the preference for static?

clalancette commented 3 days ago

Just curious, what changed your mind with regards to the preference for static?

We just had more anonymous functions, so it was less work to convert static -> anonymous rather than anonymous -> static.