search

tesseract-robotics / trajopt

Trajectory Optimization Motion Planner for ROS
381 stars 103 forks source link

Trajopt clang-tidy clean-up v3 #414

Closed rjoomen closed 1 month ago

rjoomen commented 1 month ago

Who'd have thought retrofitting const is such a lot of work.

Levi-Armstrong commented 1 month ago

Who'd have thought retrofitting const is such a lot of work.

Yea, this is one thing I had intentionally avoided doing because it can have performance impact that was not checked by the older version of clang-tidy, but now it will produce an error if you return a variable which was previous defined const.