Modern C++ best practices

[tylerjw]

Systematically go through Jason Turner's book and https://github.com/cpp-best-practices/cppbestpractices and apply modern C++ best practices to MoveIt.

Related Issues:

• https://github.com/ros-planning/moveit2/issues/841
• https://github.com/ros-planning/moveit2/issues/842
• https://github.com/ros-planning/moveit2/issues/843
• https://github.com/ros-planning/moveit2/issues/844
• https://github.com/ros-planning/moveit2/issues/845
• https://github.com/ros-planning/moveit2/issues/846
• https://github.com/ros-planning/moveit2/issues/847
• https://github.com/ros-planning/moveit2/issues/848
• https://github.com/ros-planning/moveit2/issues/849
• https://github.com/ros-planning/moveit2/issues/850
• https://github.com/ros-planning/moveit2/issues/851
• https://github.com/ros-planning/moveit2/issues/852
• https://github.com/ros-planning/moveit2/issues/853
• https://github.com/ros-planning/moveit2/issues/854
• https://github.com/ros-planning/moveit2/issues/855
• https://github.com/ros-planning/moveit2/issues/856
• https://github.com/ros-planning/moveit2/issues/857
• https://github.com/ros-planning/moveit2/issues/858
• https://github.com/ros-planning/moveit2/issues/859
• https://github.com/ros-planning/moveit2/issues/859
• https://github.com/ros-planning/moveit2/issues/860
• https://github.com/ros-planning/moveit2/issues/861
• https://github.com/ros-planning/moveit2/issues/862
• https://github.com/ros-planning/moveit2/issues/863
• https://github.com/ros-planning/moveit2/issues/864
• https://github.com/ros-planning/moveit2/issues/865
• https://github.com/ros-planning/moveit2/issues/866
• https://github.com/ros-planning/moveit2/issues/867
• https://github.com/ros-planning/moveit2/issues/868
• https://github.com/ros-planning/moveit2/issues/869
• https://github.com/ros-planning/moveit2/issues/870
• https://github.com/ros-planning/moveit2/issues/871
• https://github.com/ros-planning/moveit2/issues/872
• https://github.com/ros-planning/moveit2/issues/873
• https://github.com/ros-planning/moveit2/issues/874
• https://github.com/ros-planning/moveit2/issues/875
• https://github.com/ros-planning/moveit2/issues/876
• https://github.com/ros-planning/moveit2/issues/877
• https://github.com/ros-planning/moveit2/issues/878
• https://github.com/ros-planning/moveit2/issues/879
• https://github.com/ros-planning/moveit2/issues/880
• https://github.com/ros-planning/moveit2/issues/881
• https://github.com/ros-planning/moveit2/issues/882

[v0nser]

Hey! I would love to contribute to this project.

[tylerjw]

Hey! I would love to contribute to this project.

You are welcome to. Please just take one of the open issues and submit a PR for it.

[tarunkartik]

Interested in contributing to this effort. From my experience, a lot of these issues can be solved using certain CMake flags and running clang-tidy.