fix spelling in comment
This commit is contained in:
parent
a5fa8277f3
commit
0f58c5305c
13 changed files with 23 additions and 23 deletions
|
@ -108,7 +108,7 @@ public:
|
||||||
if (rcl_client_init(&client_handle_, this->node_handle_.get(),
|
if (rcl_client_init(&client_handle_, this->node_handle_.get(),
|
||||||
service_type_support_handle, service_name.c_str(), &client_options) != RCL_RET_OK)
|
service_type_support_handle, service_name.c_str(), &client_options) != RCL_RET_OK)
|
||||||
{
|
{
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
throw std::runtime_error(
|
throw std::runtime_error(
|
||||||
std::string("could not create client: ") +
|
std::string("could not create client: ") +
|
||||||
rcl_get_error_string_safe());
|
rcl_get_error_string_safe());
|
||||||
|
@ -175,7 +175,7 @@ public:
|
||||||
std::lock_guard<std::mutex> lock(pending_requests_mutex_);
|
std::lock_guard<std::mutex> lock(pending_requests_mutex_);
|
||||||
int64_t sequence_number;
|
int64_t sequence_number;
|
||||||
if (RCL_RET_OK != rcl_send_request(get_client_handle(), request.get(), &sequence_number)) {
|
if (RCL_RET_OK != rcl_send_request(get_client_handle(), request.get(), &sequence_number)) {
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
throw std::runtime_error(
|
throw std::runtime_error(
|
||||||
std::string("failed to send request: ") + rcl_get_error_string_safe());
|
std::string("failed to send request: ") + rcl_get_error_string_safe());
|
||||||
// *INDENT-ON*
|
// *INDENT-ON*
|
||||||
|
|
|
@ -51,7 +51,7 @@ public:
|
||||||
auto it = sub_contexts_.find(type_i);
|
auto it = sub_contexts_.find(type_i);
|
||||||
if (it == sub_contexts_.end()) {
|
if (it == sub_contexts_.end()) {
|
||||||
// It doesn't exist yet, make it
|
// It doesn't exist yet, make it
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
sub_context = std::shared_ptr<SubContext>(
|
sub_context = std::shared_ptr<SubContext>(
|
||||||
new SubContext(std::forward<Args>(args) ...),
|
new SubContext(std::forward<Args>(args) ...),
|
||||||
[] (SubContext * sub_context_ptr) {
|
[] (SubContext * sub_context_ptr) {
|
||||||
|
|
|
@ -220,7 +220,7 @@ private:
|
||||||
typename std::vector<element, VectorAlloc>::iterator
|
typename std::vector<element, VectorAlloc>::iterator
|
||||||
get_iterator_of_key(uint64_t key)
|
get_iterator_of_key(uint64_t key)
|
||||||
{
|
{
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
auto it = std::find_if(elements_.begin(), elements_.end(), [key](element & e) -> bool {
|
auto it = std::find_if(elements_.begin(), elements_.end(), [key](element & e) -> bool {
|
||||||
return e.key == key && e.in_use;
|
return e.key == key && e.in_use;
|
||||||
});
|
});
|
||||||
|
|
|
@ -180,7 +180,7 @@ public:
|
||||||
std::string("failed to get rmw handle: ") + rcl_get_error_string_safe());
|
std::string("failed to get rmw handle: ") + rcl_get_error_string_safe());
|
||||||
}
|
}
|
||||||
if (rmw_get_gid_for_publisher(publisher_rmw_handle, &rmw_gid_) != RMW_RET_OK) {
|
if (rmw_get_gid_for_publisher(publisher_rmw_handle, &rmw_gid_) != RMW_RET_OK) {
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
throw std::runtime_error(
|
throw std::runtime_error(
|
||||||
std::string("failed to get publisher gid: ") + rmw_get_error_string_safe());
|
std::string("failed to get publisher gid: ") + rmw_get_error_string_safe());
|
||||||
// *INDENT-ON*
|
// *INDENT-ON*
|
||||||
|
@ -231,7 +231,7 @@ public:
|
||||||
ipm.message_sequence = message_seq;
|
ipm.message_sequence = message_seq;
|
||||||
auto status = rcl_publish(&intra_process_publisher_handle_, &ipm);
|
auto status = rcl_publish(&intra_process_publisher_handle_, &ipm);
|
||||||
if (status != RCL_RET_OK) {
|
if (status != RCL_RET_OK) {
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
throw std::runtime_error(
|
throw std::runtime_error(
|
||||||
std::string("failed to publish intra process message: ") + rcl_get_error_string_safe());
|
std::string("failed to publish intra process message: ") + rcl_get_error_string_safe());
|
||||||
// *INDENT-ON*
|
// *INDENT-ON*
|
||||||
|
@ -306,7 +306,7 @@ protected:
|
||||||
{
|
{
|
||||||
auto status = rcl_publish(&publisher_handle_, msg);
|
auto status = rcl_publish(&publisher_handle_, msg);
|
||||||
if (status != RCL_RET_OK) {
|
if (status != RCL_RET_OK) {
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
throw std::runtime_error(
|
throw std::runtime_error(
|
||||||
std::string("failed to publish message: ") + rcl_get_error_string_safe());
|
std::string("failed to publish message: ") + rcl_get_error_string_safe());
|
||||||
// *INDENT-ON*
|
// *INDENT-ON*
|
||||||
|
|
|
@ -150,7 +150,7 @@ public:
|
||||||
rcl_ret_t status = rcl_send_response(get_service_handle(), req_id.get(), response.get());
|
rcl_ret_t status = rcl_send_response(get_service_handle(), req_id.get(), response.get());
|
||||||
|
|
||||||
if (status != RCL_RET_OK) {
|
if (status != RCL_RET_OK) {
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
throw std::runtime_error(
|
throw std::runtime_error(
|
||||||
std::string("failed to send response: ") + rcl_get_error_string_safe());
|
std::string("failed to send response: ") + rcl_get_error_string_safe());
|
||||||
// *INDENT-ON*
|
// *INDENT-ON*
|
||||||
|
|
|
@ -251,7 +251,7 @@ private:
|
||||||
(get_topic_name() + "__intra").c_str(),
|
(get_topic_name() + "__intra").c_str(),
|
||||||
&intra_process_options) != RCL_RET_OK)
|
&intra_process_options) != RCL_RET_OK)
|
||||||
{
|
{
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
throw std::runtime_error(
|
throw std::runtime_error(
|
||||||
std::string("could not create intra process subscription: ") + rcl_get_error_string_safe());
|
std::string("could not create intra process subscription: ") + rcl_get_error_string_safe());
|
||||||
// *INDENT-ON*
|
// *INDENT-ON*
|
||||||
|
|
|
@ -112,7 +112,7 @@ Executor::remove_node(rclcpp::node::Node::SharedPtr node_ptr, bool notify)
|
||||||
weak_nodes_.erase(
|
weak_nodes_.erase(
|
||||||
std::remove_if(
|
std::remove_if(
|
||||||
weak_nodes_.begin(), weak_nodes_.end(),
|
weak_nodes_.begin(), weak_nodes_.end(),
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
[&](std::weak_ptr<rclcpp::node::Node> & i)
|
[&](std::weak_ptr<rclcpp::node::Node> & i)
|
||||||
{
|
{
|
||||||
bool matched = (i.lock() == node_ptr);
|
bool matched = (i.lock() == node_ptr);
|
||||||
|
@ -329,7 +329,7 @@ Executor::wait_for_work(std::chrono::nanoseconds timeout)
|
||||||
weak_nodes_.erase(
|
weak_nodes_.erase(
|
||||||
remove_if(
|
remove_if(
|
||||||
weak_nodes_.begin(), weak_nodes_.end(),
|
weak_nodes_.begin(), weak_nodes_.end(),
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
[](std::weak_ptr<rclcpp::node::Node> i)
|
[](std::weak_ptr<rclcpp::node::Node> i)
|
||||||
{
|
{
|
||||||
return i.expired();
|
return i.expired();
|
||||||
|
|
|
@ -69,7 +69,7 @@ IntraProcessManager::get_next_unique_id()
|
||||||
// So around 585 million years. Even at 1 GHz, it would take 585 years.
|
// So around 585 million years. Even at 1 GHz, it would take 585 years.
|
||||||
// I think it's safe to avoid trying to handle overflow.
|
// I think it's safe to avoid trying to handle overflow.
|
||||||
// If we roll over then it's most likely a bug.
|
// If we roll over then it's most likely a bug.
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
throw std::overflow_error(
|
throw std::overflow_error(
|
||||||
"exhausted the unique id's for publishers and subscribers in this process "
|
"exhausted the unique id's for publishers and subscribers in this process "
|
||||||
"(congratulations your computer is either extremely fast or extremely old)");
|
"(congratulations your computer is either extremely fast or extremely old)");
|
||||||
|
|
|
@ -105,7 +105,7 @@ ParameterVariant::get_type_name() const
|
||||||
case rclcpp::parameter::ParameterType::PARAMETER_NOT_SET:
|
case rclcpp::parameter::ParameterType::PARAMETER_NOT_SET:
|
||||||
return "not set";
|
return "not set";
|
||||||
default:
|
default:
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
throw std::runtime_error(
|
throw std::runtime_error(
|
||||||
"Unexpected type from ParameterVariant: " + std::to_string(get_type()));
|
"Unexpected type from ParameterVariant: " + std::to_string(get_type()));
|
||||||
// *INDENT-ON*
|
// *INDENT-ON*
|
||||||
|
@ -172,7 +172,7 @@ ParameterVariant::from_parameter(const rcl_interfaces::msg::Parameter & paramete
|
||||||
throw std::runtime_error("Type from ParameterValue is not set");
|
throw std::runtime_error("Type from ParameterValue is not set");
|
||||||
default:
|
default:
|
||||||
// TODO(wjwwood): use custom exception
|
// TODO(wjwwood): use custom exception
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
throw std::runtime_error(
|
throw std::runtime_error(
|
||||||
"Unexpected type from ParameterVariant: " + std::to_string(parameter.value.type));
|
"Unexpected type from ParameterVariant: " + std::to_string(parameter.value.type));
|
||||||
// *INDENT-ON*
|
// *INDENT-ON*
|
||||||
|
@ -218,7 +218,7 @@ ParameterVariant::value_to_string() const
|
||||||
case rclcpp::parameter::ParameterType::PARAMETER_NOT_SET:
|
case rclcpp::parameter::ParameterType::PARAMETER_NOT_SET:
|
||||||
return "not set";
|
return "not set";
|
||||||
default:
|
default:
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
throw std::runtime_error(
|
throw std::runtime_error(
|
||||||
"Unexpected type from ParameterVariant: " + std::to_string(get_type()));
|
"Unexpected type from ParameterVariant: " + std::to_string(get_type()));
|
||||||
// *INDENT-ON*
|
// *INDENT-ON*
|
||||||
|
|
|
@ -57,7 +57,7 @@ AsyncParametersClient::get_parameters(
|
||||||
auto request = std::make_shared<rcl_interfaces::srv::GetParameters::Request>();
|
auto request = std::make_shared<rcl_interfaces::srv::GetParameters::Request>();
|
||||||
request->names = names;
|
request->names = names;
|
||||||
|
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
get_parameters_client_->async_send_request(
|
get_parameters_client_->async_send_request(
|
||||||
request,
|
request,
|
||||||
[request, promise_result, future_result, &callback](
|
[request, promise_result, future_result, &callback](
|
||||||
|
@ -100,7 +100,7 @@ AsyncParametersClient::get_parameter_types(
|
||||||
auto request = std::make_shared<rcl_interfaces::srv::GetParameterTypes::Request>();
|
auto request = std::make_shared<rcl_interfaces::srv::GetParameterTypes::Request>();
|
||||||
request->names = names;
|
request->names = names;
|
||||||
|
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
get_parameter_types_client_->async_send_request(
|
get_parameter_types_client_->async_send_request(
|
||||||
request,
|
request,
|
||||||
[promise_result, future_result, &callback](
|
[promise_result, future_result, &callback](
|
||||||
|
@ -135,7 +135,7 @@ AsyncParametersClient::set_parameters(
|
||||||
|
|
||||||
auto request = std::make_shared<rcl_interfaces::srv::SetParameters::Request>();
|
auto request = std::make_shared<rcl_interfaces::srv::SetParameters::Request>();
|
||||||
|
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
std::transform(parameters.begin(), parameters.end(), std::back_inserter(request->parameters),
|
std::transform(parameters.begin(), parameters.end(), std::back_inserter(request->parameters),
|
||||||
[](rclcpp::parameter::ParameterVariant p) {
|
[](rclcpp::parameter::ParameterVariant p) {
|
||||||
return p.to_parameter();
|
return p.to_parameter();
|
||||||
|
@ -171,7 +171,7 @@ AsyncParametersClient::set_parameters_atomically(
|
||||||
|
|
||||||
auto request = std::make_shared<rcl_interfaces::srv::SetParametersAtomically::Request>();
|
auto request = std::make_shared<rcl_interfaces::srv::SetParametersAtomically::Request>();
|
||||||
|
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
std::transform(parameters.begin(), parameters.end(), std::back_inserter(request->parameters),
|
std::transform(parameters.begin(), parameters.end(), std::back_inserter(request->parameters),
|
||||||
[](rclcpp::parameter::ParameterVariant p) {
|
[](rclcpp::parameter::ParameterVariant p) {
|
||||||
return p.to_parameter();
|
return p.to_parameter();
|
||||||
|
@ -210,7 +210,7 @@ AsyncParametersClient::list_parameters(
|
||||||
request->prefixes = prefixes;
|
request->prefixes = prefixes;
|
||||||
request->depth = depth;
|
request->depth = depth;
|
||||||
|
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
list_parameters_client_->async_send_request(
|
list_parameters_client_->async_send_request(
|
||||||
request,
|
request,
|
||||||
[promise_result, future_result, &callback](
|
[promise_result, future_result, &callback](
|
||||||
|
|
|
@ -23,7 +23,7 @@ ParameterService::ParameterService(const rclcpp::node::Node::SharedPtr node)
|
||||||
: node_(node)
|
: node_(node)
|
||||||
{
|
{
|
||||||
std::weak_ptr<rclcpp::node::Node> captured_node = node_;
|
std::weak_ptr<rclcpp::node::Node> captured_node = node_;
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
get_parameters_service_ = node_->create_service<rcl_interfaces::srv::GetParameters>(
|
get_parameters_service_ = node_->create_service<rcl_interfaces::srv::GetParameters>(
|
||||||
node_->get_name() + "__get_parameters",
|
node_->get_name() + "__get_parameters",
|
||||||
[captured_node](
|
[captured_node](
|
||||||
|
|
|
@ -123,7 +123,7 @@ PublisherBase::setup_intra_process(
|
||||||
auto ret = rmw_get_gid_for_publisher(
|
auto ret = rmw_get_gid_for_publisher(
|
||||||
publisher_rmw_handle, &intra_process_rmw_gid_);
|
publisher_rmw_handle, &intra_process_rmw_gid_);
|
||||||
if (ret != RMW_RET_OK) {
|
if (ret != RMW_RET_OK) {
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
throw std::runtime_error(
|
throw std::runtime_error(
|
||||||
std::string("failed to create intra process publisher gid: ") +
|
std::string("failed to create intra process publisher gid: ") +
|
||||||
rmw_get_error_string_safe());
|
rmw_get_error_string_safe());
|
||||||
|
|
|
@ -95,7 +95,7 @@ rclcpp::utilities::init(int argc, char * argv[])
|
||||||
{
|
{
|
||||||
g_is_interrupted.store(false);
|
g_is_interrupted.store(false);
|
||||||
if (rcl_init(argc, argv, rcl_get_default_allocator()) != RCL_RET_OK) {
|
if (rcl_init(argc, argv, rcl_get_default_allocator()) != RCL_RET_OK) {
|
||||||
// *INDENT-OFF* (prevent uncrustify from making unecessary indents here)
|
// *INDENT-OFF* (prevent uncrustify from making unnecessary indents here)
|
||||||
throw std::runtime_error(
|
throw std::runtime_error(
|
||||||
std::string("failed to initialize rmw implementation: ") + rcl_get_error_string_safe());
|
std::string("failed to initialize rmw implementation: ") + rcl_get_error_string_safe());
|
||||||
// *INDENT-ON*
|
// *INDENT-ON*
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue