niklas/rclcpp
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix warning about comparison between signed and unsigned integers

Browse source
This commit is contained in:
Esteve Fernandez 2015-06-01 15:27:55 -07:00
parent b7c3b7cf0f
commit 286084b05e
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
rclcpp/include/rclcpp/node_impl.hpp
View file

@ -309,13 +309,14 @@ Node::list_parameters(
std::lock_guard<std::mutex> lock(mutex_);
rcl_interfaces::ListParametersResult result;
// TODO: define parameter separator, use "." for now
// TODO(esteve): define parameter separator, use "." for now
for (auto & kv : parameters_) {
if (std::any_of(prefixes.cbegin(), prefixes.cend(), [&kv, &depth](const std::string & prefix) {
if (kv.first.find(prefix + ".") == 0) {
size_t length = prefix.length();
std::string substr = kv.first.substr(length);
return std::count(substr.begin(), substr.end(), '.') < depth;
// Cast as unsigned integer to avoid warning
return static_cast<uint64_t>(std::count(substr.begin(), substr.end(), '.')) < depth;
}
return false;
}))