Skip to content

Commit

Permalink
(fix) unused variable
Browse files Browse the repository at this point in the history
  • Loading branch information
Karsten1987 committed Dec 9, 2016
1 parent e26913c commit 456baa0
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
6 changes: 3 additions & 3 deletions rclcpp/test/test_externally_defined_services.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ TEST_F(TestExternallyDefinedServices, default_behavior) {
auto srv = node_handle->create_service<rclcpp::srv::Mock>("test",
callback);
EXPECT_STREQ(srv->get_service_name().c_str(), "test");
} catch (const std::exception & e) {
} catch (const std::exception &) {
FAIL();
return;
}
Expand All @@ -69,7 +69,7 @@ TEST_F(TestExternallyDefinedServices, extern_defined_uninitialized) {
try {
rclcpp::service::Service<rclcpp::srv::Mock>(node_handle->get_shared_node_handle(),
&service_handle, cb);
} catch (const std::runtime_error & e) {
} catch (const std::runtime_error &) {
SUCCEED();
return;
}
Expand Down Expand Up @@ -97,7 +97,7 @@ TEST_F(TestExternallyDefinedServices, extern_defined_initialized) {
try {
rclcpp::service::Service<rclcpp::srv::Mock>(node_handle->get_shared_node_handle(),
&service_handle, cb);
} catch (const std::runtime_error & e) {
} catch (const std::runtime_error &) {
FAIL();
return;
}
Expand Down
1 change: 1 addition & 0 deletions rclcpp_lifecycle/src/lifecycle_node_impl.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -204,6 +204,7 @@ class LifecycleNode::LifecycleNodeImpl
} catch (const std::exception & e) {
fprintf(stderr, "Caught exception in callback for transition %d\n",
it->first);
fprintf(stderr, "Original error msg: %s\n", e.what());
cb_success = false;
}
} else {
Expand Down

0 comments on commit 456baa0

Please sign in to comment.