-
Notifications
You must be signed in to change notification settings - Fork 984
Closed
Labels
cleanupenhancementhelp wantedplease consider improving this request!please consider improving this request!moveit day candidateThis issue can likely be resolved in less than a dayThis issue can likely be resolved in less than a daysimple improvementsThis issue can likely be resolved in less than a dayThis issue can likely be resolved in less than a day
Description
Similar in spirit to #2757, here is another nagging detail scattered throughout the code base:
Scott Meyer's "Effective Modern C++" Item 34 as well as TOTW 108 state:
Avoid
boost/std::bind[in favor of lambdas where applicable]
MoveIt uses bind in various places and readability suffers from it.
We should update the definitions to use lambdas with properly named arguments instead!
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
cleanupenhancementhelp wantedplease consider improving this request!please consider improving this request!moveit day candidateThis issue can likely be resolved in less than a dayThis issue can likely be resolved in less than a daysimple improvementsThis issue can likely be resolved in less than a dayThis issue can likely be resolved in less than a day