-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Amcl ctest fix #114
Amcl ctest fix #114
Conversation
Will re-open this PR after PR #105 is reviewed. |
…into amcl_ctest_Fix
@@ -20,9 +20,15 @@ | |||
|
|||
/* Author: Brian Gerkey */ | |||
|
|||
#include <boost/foreach.hpp> |
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.
There shouldn't be a need for this header. std has a suitable for statement now.
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.
I will address this issue on a separate PR.
@@ -391,7 +408,7 @@ void AmclNode::reconfigureCB(AMCLConfig & config, uint32_t level) | |||
pf_ = pf_alloc(min_particles_, max_particles_, | |||
alpha_slow_, alpha_fast_, | |||
(pf_init_model_fn_t)AmclNode::uniformPoseGenerator, | |||
(void *)map_); | |||
<void *>(map_)); |
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.
What's happening here? This doesn't look like valid syntax to me. You've somehow converted a cast to void* into a template parameter of nothing, or am I reading this wrong?
Ah. It looks like this is missing the reinterpret_cast
that you used below.
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.
Yes, it's missing the reinterpret_cast
. Thanks.
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.
Fixed.
Adding reinterpret_cast to line 411 to fix casting typo.
The copyright template is not available and therefore I removed the copyright linter from the test. I've also removed the flake8 due to not checking the files correctly.
Addressing issue #109.