Failed to get response from ROS service runtime_error should be informative

[tomasruizt]

Feature request

runtime_error should be informative.

Feature description

Expected behavior

"what(): Failed to get response from ROS service: myservice/"

Actual behavior

"what(): Failed to get response from ROS service"

Implementation considerations

Change the code in:

ros1_bridge/include/ros1_bridge/factory.hpp

Line 252 in 961ddee

throw std::runtime_error("Failed to get response from ROS service");

to

throw std::runtime_error("Failed to get response from ROS service: " + client.getService());

Optional

Maybe it should include the fact that the error was thrown from the ros1_bridge

[dirk-thomas]

Please consider to create a pull request with your proposed patch.