Merge pull request #61 from ros2/raise_warning_level
raise warning level
This commit is contained in:
		
						commit
						582de92d6f
					
				
					 5 changed files with 12 additions and 7 deletions
				
			
		| 
						 | 
					@ -138,7 +138,7 @@ public:
 | 
				
			||||||
  SharedFuture async_send_request(
 | 
					  SharedFuture async_send_request(
 | 
				
			||||||
    typename ServiceT::Request::SharedPtr & request)
 | 
					    typename ServiceT::Request::SharedPtr & request)
 | 
				
			||||||
  {
 | 
					  {
 | 
				
			||||||
    return async_send_request(request, [](SharedFuture f) {});
 | 
					    return async_send_request(request, [](SharedFuture) {});
 | 
				
			||||||
  }
 | 
					  }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  SharedFuture async_send_request(
 | 
					  SharedFuture async_send_request(
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -39,11 +39,13 @@ public:
 | 
				
			||||||
  RCLCPP_MAKE_SHARED_DEFINITIONS(MemoryStrategy);
 | 
					  RCLCPP_MAKE_SHARED_DEFINITIONS(MemoryStrategy);
 | 
				
			||||||
  virtual void ** borrow_handles(HandleType type, size_t number_of_handles)
 | 
					  virtual void ** borrow_handles(HandleType type, size_t number_of_handles)
 | 
				
			||||||
  {
 | 
					  {
 | 
				
			||||||
 | 
					    (void)type;
 | 
				
			||||||
    return static_cast<void **>(alloc(sizeof(void *) * number_of_handles));
 | 
					    return static_cast<void **>(alloc(sizeof(void *) * number_of_handles));
 | 
				
			||||||
  }
 | 
					  }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  virtual void return_handles(HandleType type, void ** handles)
 | 
					  virtual void return_handles(HandleType type, void ** handles)
 | 
				
			||||||
  {
 | 
					  {
 | 
				
			||||||
 | 
					    (void)type;
 | 
				
			||||||
    this->free(handles);
 | 
					    this->free(handles);
 | 
				
			||||||
  }
 | 
					  }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -48,7 +48,7 @@ public:
 | 
				
			||||||
  {
 | 
					  {
 | 
				
			||||||
    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](
 | 
					      node_->get_name() + "__get_parameters", [&node](
 | 
				
			||||||
        const std::shared_ptr<rmw_request_id_t> request_header,
 | 
					        const std::shared_ptr<rmw_request_id_t>,
 | 
				
			||||||
        const std::shared_ptr<rcl_interfaces::srv::GetParameters::Request> request,
 | 
					        const std::shared_ptr<rcl_interfaces::srv::GetParameters::Request> request,
 | 
				
			||||||
        std::shared_ptr<rcl_interfaces::srv::GetParameters::Response> response)
 | 
					        std::shared_ptr<rcl_interfaces::srv::GetParameters::Response> response)
 | 
				
			||||||
        {
 | 
					        {
 | 
				
			||||||
| 
						 | 
					@ -63,7 +63,7 @@ public:
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    get_parameter_types_service_ = node_->create_service<rcl_interfaces::srv::GetParameterTypes>(
 | 
					    get_parameter_types_service_ = node_->create_service<rcl_interfaces::srv::GetParameterTypes>(
 | 
				
			||||||
      node_->get_name() + "__get_parameter_types", [&node](
 | 
					      node_->get_name() + "__get_parameter_types", [&node](
 | 
				
			||||||
        const std::shared_ptr<rmw_request_id_t> request_header,
 | 
					        const std::shared_ptr<rmw_request_id_t>,
 | 
				
			||||||
        const std::shared_ptr<rcl_interfaces::srv::GetParameterTypes::Request> request,
 | 
					        const std::shared_ptr<rcl_interfaces::srv::GetParameterTypes::Request> request,
 | 
				
			||||||
        std::shared_ptr<rcl_interfaces::srv::GetParameterTypes::Response> response)
 | 
					        std::shared_ptr<rcl_interfaces::srv::GetParameterTypes::Response> response)
 | 
				
			||||||
        {
 | 
					        {
 | 
				
			||||||
| 
						 | 
					@ -77,7 +77,7 @@ public:
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    set_parameters_service_ = node_->create_service<rcl_interfaces::srv::SetParameters>(
 | 
					    set_parameters_service_ = node_->create_service<rcl_interfaces::srv::SetParameters>(
 | 
				
			||||||
      node_->get_name() + "__set_parameters", [&node](
 | 
					      node_->get_name() + "__set_parameters", [&node](
 | 
				
			||||||
        const std::shared_ptr<rmw_request_id_t> request_header,
 | 
					        const std::shared_ptr<rmw_request_id_t>,
 | 
				
			||||||
        const std::shared_ptr<rcl_interfaces::srv::SetParameters::Request> request,
 | 
					        const std::shared_ptr<rcl_interfaces::srv::SetParameters::Request> request,
 | 
				
			||||||
        std::shared_ptr<rcl_interfaces::srv::SetParameters::Response> response)
 | 
					        std::shared_ptr<rcl_interfaces::srv::SetParameters::Response> response)
 | 
				
			||||||
        {
 | 
					        {
 | 
				
			||||||
| 
						 | 
					@ -96,7 +96,7 @@ public:
 | 
				
			||||||
    set_parameters_atomically_service_ =
 | 
					    set_parameters_atomically_service_ =
 | 
				
			||||||
      node_->create_service<rcl_interfaces::srv::SetParametersAtomically>(
 | 
					      node_->create_service<rcl_interfaces::srv::SetParametersAtomically>(
 | 
				
			||||||
      node_->get_name() + "__set_parameters_atomically", [&node](
 | 
					      node_->get_name() + "__set_parameters_atomically", [&node](
 | 
				
			||||||
        const std::shared_ptr<rmw_request_id_t> request_header,
 | 
					        const std::shared_ptr<rmw_request_id_t>,
 | 
				
			||||||
        const std::shared_ptr<rcl_interfaces::srv::SetParametersAtomically::Request> request,
 | 
					        const std::shared_ptr<rcl_interfaces::srv::SetParametersAtomically::Request> request,
 | 
				
			||||||
        std::shared_ptr<rcl_interfaces::srv::SetParametersAtomically::Response> response)
 | 
					        std::shared_ptr<rcl_interfaces::srv::SetParametersAtomically::Response> response)
 | 
				
			||||||
        {
 | 
					        {
 | 
				
			||||||
| 
						 | 
					@ -114,7 +114,7 @@ public:
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    describe_parameters_service_ = node_->create_service<rcl_interfaces::srv::DescribeParameters>(
 | 
					    describe_parameters_service_ = node_->create_service<rcl_interfaces::srv::DescribeParameters>(
 | 
				
			||||||
      node_->get_name() + "__describe_parameters", [&node](
 | 
					      node_->get_name() + "__describe_parameters", [&node](
 | 
				
			||||||
        const std::shared_ptr<rmw_request_id_t> request_header,
 | 
					        const std::shared_ptr<rmw_request_id_t>,
 | 
				
			||||||
        const std::shared_ptr<rcl_interfaces::srv::DescribeParameters::Request> request,
 | 
					        const std::shared_ptr<rcl_interfaces::srv::DescribeParameters::Request> request,
 | 
				
			||||||
        std::shared_ptr<rcl_interfaces::srv::DescribeParameters::Response> response)
 | 
					        std::shared_ptr<rcl_interfaces::srv::DescribeParameters::Response> response)
 | 
				
			||||||
        {
 | 
					        {
 | 
				
			||||||
| 
						 | 
					@ -125,7 +125,7 @@ public:
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    list_parameters_service_ = node_->create_service<rcl_interfaces::srv::ListParameters>(
 | 
					    list_parameters_service_ = node_->create_service<rcl_interfaces::srv::ListParameters>(
 | 
				
			||||||
      node_->get_name() + "__list_parameters", [&node](
 | 
					      node_->get_name() + "__list_parameters", [&node](
 | 
				
			||||||
        const std::shared_ptr<rmw_request_id_t> request_header,
 | 
					        const std::shared_ptr<rmw_request_id_t>,
 | 
				
			||||||
        const std::shared_ptr<rcl_interfaces::srv::ListParameters::Request> request,
 | 
					        const std::shared_ptr<rcl_interfaces::srv::ListParameters::Request> request,
 | 
				
			||||||
        std::shared_ptr<rcl_interfaces::srv::ListParameters::Response> response)
 | 
					        std::shared_ptr<rcl_interfaces::srv::ListParameters::Response> response)
 | 
				
			||||||
        {
 | 
					        {
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -88,6 +88,7 @@ public:
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  void return_handles(HandleType type, void ** handles)
 | 
					  void return_handles(HandleType type, void ** handles)
 | 
				
			||||||
  {
 | 
					  {
 | 
				
			||||||
 | 
					    (void)handles;
 | 
				
			||||||
    switch (type) {
 | 
					    switch (type) {
 | 
				
			||||||
      case HandleType::subscriber_handle:
 | 
					      case HandleType::subscriber_handle:
 | 
				
			||||||
        memset(subscriber_pool_, 0, max_subscribers_);
 | 
					        memset(subscriber_pool_, 0, max_subscribers_);
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -95,6 +95,8 @@ namespace utilities
 | 
				
			||||||
void
 | 
					void
 | 
				
			||||||
init(int argc, char * argv[])
 | 
					init(int argc, char * argv[])
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
 | 
					  (void)argc;
 | 
				
			||||||
 | 
					  (void)argv;
 | 
				
			||||||
  // TODO(wjwwood): Handle rmw_ret_t's value.
 | 
					  // TODO(wjwwood): Handle rmw_ret_t's value.
 | 
				
			||||||
  rmw_init();
 | 
					  rmw_init();
 | 
				
			||||||
#ifdef HAS_SIGACTION
 | 
					#ifdef HAS_SIGACTION
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue