-
Notifications
You must be signed in to change notification settings - Fork 938
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix some clang-tidy issues on Travis #2337
Conversation
I've seen this too and I don't know what to do with it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All the typedef to using changes look good. It is just a new style of aliases. I remember reading Scott Myers where he advises using the new style.
7b0c5e5
to
d5a7cb9
Compare
Forgot to clang-format what was fixed by clang-tidy... Should be fine now. |
It is just a new style of aliases.
Goodbye typedef! Using using is the only proper way to define aliases by now and it annoyed me for some time already that MoveIt is still full of typedefs...
|
Thanks to clang-tidy this was automatically fixed now 😉 |
8d57e85
to
3e8065c
Compare
typedef std::map<const std::pair<std::string, std::string>, std::vector<DistanceResultsData> > DistanceMap; | ||
using DistanceMap = std::map<const std::pair<std::string, std::string>, std::vector<DistanceRequest> >; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Weird. clang-tidy changed the type here, replacing std::vector<DistanceResultsData>
with std::vector<DistanceRequest>
!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very weird, we don't have it configured in some non-standard way do we?
Having switched to Noetic (and clang-tidy-10?) on Travis, we get some new clang-tidy issues reported:
https://travis-ci.com/github/ros-planning/moveit/jobs/393573194
Interestingly, locally I got even more issues!? There is also one "fix" applied that is nonsense: