diff --git a/ros_ign_bridge/CMakeLists.txt b/ros_ign_bridge/CMakeLists.txt index f312164c..c71608cd 100644 --- a/ros_ign_bridge/CMakeLists.txt +++ b/ros_ign_bridge/CMakeLists.txt @@ -38,8 +38,8 @@ endif() include_directories(include) set(common_sources - src/convert_builtin_interfaces.cpp - src/builtin_interfaces_factories.cpp + src/convert.cpp + src/factories.cpp ) set(bridge_executables diff --git a/ros_ign_bridge/include/ros_ign_bridge/convert_builtin_interfaces.hpp b/ros_ign_bridge/include/ros_ign_bridge/convert.hpp similarity index 97% rename from ros_ign_bridge/include/ros_ign_bridge/convert_builtin_interfaces.hpp rename to ros_ign_bridge/include/ros_ign_bridge/convert.hpp index 9bb66375..ab11ad3b 100644 --- a/ros_ign_bridge/include/ros_ign_bridge/convert_builtin_interfaces.hpp +++ b/ros_ign_bridge/include/ros_ign_bridge/convert.hpp @@ -12,12 +12,12 @@ // See the License for the specific language governing permissions and // limitations under the License. -#ifndef ROS_IGN_BRIDGE__CONVERT_BUILTIN_INTERFACES_HPP_ -#define ROS_IGN_BRIDGE__CONVERT_BUILTIN_INTERFACES_HPP_ +#ifndef ROS_IGN_BRIDGE__CONVERT_HPP_ +#define ROS_IGN_BRIDGE__CONVERT_HPP_ #include -// include ROS 2 builtin messages +// ROS 2 messages #include #include #include @@ -42,7 +42,7 @@ #include #include -// include Ignition builtin messages +// Ignition messages #include #include @@ -372,4 +372,4 @@ convert_ign_to_ros( } // namespace ros_ign_bridge -#endif // ROS_IGN_BRIDGE__CONVERT_BUILTIN_INTERFACES_HPP_ +#endif // ROS_IGN_BRIDGE__CONVERT_HPP_ diff --git a/ros_ign_bridge/include/ros_ign_bridge/bridge.hpp b/ros_ign_bridge/src/bridge.hpp similarity index 95% rename from ros_ign_bridge/include/ros_ign_bridge/bridge.hpp rename to ros_ign_bridge/src/bridge.hpp index 9727ac41..45f86a51 100644 --- a/ros_ign_bridge/include/ros_ign_bridge/bridge.hpp +++ b/ros_ign_bridge/src/bridge.hpp @@ -12,17 +12,17 @@ // See the License for the specific language governing permissions and // limitations under the License. -#ifndef ROS_IGN_BRIDGE__BRIDGE_HPP_ -#define ROS_IGN_BRIDGE__BRIDGE_HPP_ +#ifndef BRIDGE_HPP_ +#define BRIDGE_HPP_ // include Ignition Transport #include -#include - #include #include +#include "factories.hpp" + namespace ros_ign_bridge { @@ -110,4 +110,4 @@ create_bidirectional_bridge( } // namespace ros_ign_bridge -#endif // ROS_IGN_BRIDGE__BRIDGE_HPP_ +#endif // BRIDGE_HPP_ diff --git a/ros_ign_bridge/src/convert_builtin_interfaces.cpp b/ros_ign_bridge/src/convert.cpp similarity index 99% rename from ros_ign_bridge/src/convert_builtin_interfaces.cpp rename to ros_ign_bridge/src/convert.cpp index 4d24ac58..2ab2066d 100644 --- a/ros_ign_bridge/src/convert_builtin_interfaces.cpp +++ b/ros_ign_bridge/src/convert.cpp @@ -12,7 +12,7 @@ // See the License for the specific language governing permissions and // limitations under the License. -#include +#include "ros_ign_bridge/convert.hpp" #include #include diff --git a/ros_ign_bridge/src/builtin_interfaces_factories.cpp b/ros_ign_bridge/src/factories.cpp similarity index 98% rename from ros_ign_bridge/src/builtin_interfaces_factories.cpp rename to ros_ign_bridge/src/factories.cpp index adacf87c..83cc63cb 100644 --- a/ros_ign_bridge/src/builtin_interfaces_factories.cpp +++ b/ros_ign_bridge/src/factories.cpp @@ -15,15 +15,14 @@ #include #include -// include builtin interfaces -#include "ros_ign_bridge/builtin_interfaces_factories.hpp" -#include "ros_ign_bridge/convert_builtin_interfaces.hpp" +#include "factories.hpp" +#include "ros_ign_bridge/convert.hpp" namespace ros_ign_bridge { std::shared_ptr -get_factory_builtin_interfaces( +get_factory_impl( const std::string & ros_type_name, const std::string & ign_type_name) { @@ -299,7 +298,7 @@ get_factory( const std::string & ign_type_name) { std::shared_ptr factory; - factory = get_factory_builtin_interfaces(ros_type_name, ign_type_name); + factory = get_factory_impl(ros_type_name, ign_type_name); if (factory) { return factory; } diff --git a/ros_ign_bridge/include/ros_ign_bridge/builtin_interfaces_factories.hpp b/ros_ign_bridge/src/factories.hpp similarity index 96% rename from ros_ign_bridge/include/ros_ign_bridge/builtin_interfaces_factories.hpp rename to ros_ign_bridge/src/factories.hpp index 1be3372c..045b577e 100644 --- a/ros_ign_bridge/include/ros_ign_bridge/builtin_interfaces_factories.hpp +++ b/ros_ign_bridge/src/factories.hpp @@ -12,10 +12,10 @@ // See the License for the specific language governing permissions and // limitations under the License. -#ifndef ROS_IGN_BRIDGE__BUILTIN_INTERFACES_FACTORIES_HPP_ -#define ROS_IGN_BRIDGE__BUILTIN_INTERFACES_FACTORIES_HPP_ +#ifndef FACTORIES_HPP_ +#define FACTORIES_HPP_ -// include ROS 2 messages +// ROS 2 messages #include #include #include @@ -40,22 +40,16 @@ #include #include -// include Ignition Transport messages +// Ignition messages #include -#include - #include #include +#include "factory.hpp" + namespace ros_ign_bridge { - -std::shared_ptr -get_factory_builtin_interfaces( - const std::string & ros_type_name, - const std::string & ign_type_name); - std::shared_ptr get_factory( const std::string & ros_type_name, @@ -541,4 +535,4 @@ Factory< } // namespace ros_ign_bridge -#endif // ROS_IGN_BRIDGE__BUILTIN_INTERFACES_FACTORIES_HPP_ +#endif // FACTORIES_HPP_ diff --git a/ros_ign_bridge/include/ros_ign_bridge/factory.hpp b/ros_ign_bridge/src/factory.hpp similarity index 96% rename from ros_ign_bridge/include/ros_ign_bridge/factory.hpp rename to ros_ign_bridge/src/factory.hpp index fde8eb7d..2e4bdf5a 100644 --- a/ros_ign_bridge/include/ros_ign_bridge/factory.hpp +++ b/ros_ign_bridge/src/factory.hpp @@ -12,8 +12,8 @@ // See the License for the specific language governing permissions and // limitations under the License. -#ifndef ROS_IGN_BRIDGE__FACTORY_HPP_ -#define ROS_IGN_BRIDGE__FACTORY_HPP_ +#ifndef FACTORY_HPP_ +#define FACTORY_HPP_ #include @@ -24,7 +24,7 @@ #include #include -#include "ros_ign_bridge/factory_interface.hpp" +#include "factory_interface.hpp" namespace ros_ign_bridge { @@ -150,4 +150,4 @@ class Factory : public FactoryInterface } // namespace ros_ign_bridge -#endif // ROS_IGN_BRIDGE__FACTORY_HPP_ +#endif // FACTORY_HPP_ diff --git a/ros_ign_bridge/include/ros_ign_bridge/factory_interface.hpp b/ros_ign_bridge/src/factory_interface.hpp similarity index 91% rename from ros_ign_bridge/include/ros_ign_bridge/factory_interface.hpp rename to ros_ign_bridge/src/factory_interface.hpp index 3ae95140..b3b1c555 100644 --- a/ros_ign_bridge/include/ros_ign_bridge/factory_interface.hpp +++ b/ros_ign_bridge/src/factory_interface.hpp @@ -12,8 +12,8 @@ // See the License for the specific language governing permissions and // limitations under the License. -#ifndef ROS_IGN_BRIDGE__FACTORY_INTERFACE_HPP_ -#define ROS_IGN_BRIDGE__FACTORY_INTERFACE_HPP_ +#ifndef FACTORY_INTERFACE_HPP_ +#define FACTORY_INTERFACE_HPP_ // include ROS 2 #include @@ -63,4 +63,4 @@ class FactoryInterface } // namespace ros_ign_bridge -#endif // ROS_IGN_BRIDGE__FACTORY_INTERFACE_HPP_ +#endif // FACTORY_INTERFACE_HPP_ diff --git a/ros_ign_bridge/src/parameter_bridge.cpp b/ros_ign_bridge/src/parameter_bridge.cpp index 8b9b2abf..fc28a1e5 100644 --- a/ros_ign_bridge/src/parameter_bridge.cpp +++ b/ros_ign_bridge/src/parameter_bridge.cpp @@ -18,14 +18,14 @@ // include Ignition Transport #include -#include - #include #include #include #include #include +#include "bridge.hpp" + // Direction of bridge. enum Direction { diff --git a/ros_ign_bridge/src/static_bridge.cpp b/ros_ign_bridge/src/static_bridge.cpp index 80b5ba2f..e16d078a 100644 --- a/ros_ign_bridge/src/static_bridge.cpp +++ b/ros_ign_bridge/src/static_bridge.cpp @@ -17,11 +17,11 @@ // include Ignition Transport #include -#include - #include #include +#include "bridge.hpp" + ////////////////////////////////////////////////// int main(int argc, char * argv[]) { diff --git a/ros_ign_image/src/image_bridge.cpp b/ros_ign_image/src/image_bridge.cpp index c0ef9fac..8e2c82f8 100644 --- a/ros_ign_image/src/image_bridge.cpp +++ b/ros_ign_image/src/image_bridge.cpp @@ -16,7 +16,7 @@ #include #include -#include +#include #include #include