Remove generation of RCL allocators from C++.

As explained in ros2#1254, there's conceptually no way to implement RCL
allocators in terms of C++ allocators. Since allocator_common can't
be fixed, delete it and inline calls to rcl_get_default_allocator at
the call sites. Since this would introduce surprising behavior for
custom allocator users (using system malloc when a custom allocator
is requested), add compile-time assertions checking for non-default
allocators. Since there's other blocking bugs for allocator use
(ros2#1061), this shouldn't break existing users.

Technically, we could roll back a lot of templating on allocator and
explicitly use std::allocator<void> in all classes. I have refrained
from doing so because you might have plans with this code, but if you
concur, I'll be more than happy to de-template all these classes to
simplify the code.

Author: stevewolter

rclcpp/include/rclcpp/allocator/allocator_common.hpp

@@ -23,7 +23,6 @@
 #include <type_traits>
 #include <utility>
 
-#include "rclcpp/allocator/allocator_common.hpp"
 #include "rclcpp/function_traits.hpp"
 #include "rclcpp/message_info.hpp"
 #include "rclcpp/visibility_control.hpp"

rclcpp/include/rclcpp/allocator/allocator_deleter.hpp

@@ -19,8 +19,6 @@
 #include <type_traits>
 #include <utility>
 
-#include "rclcpp/allocator/allocator_common.hpp"
-#include "rclcpp/allocator/allocator_deleter.hpp"
 #include "rclcpp/experimental/buffers/buffer_implementation_base.hpp"
 #include "rclcpp/macros.hpp"
 

rclcpp/include/rclcpp/any_subscription_callback.hpp

@@ -30,7 +30,6 @@
 #include <utility>
 #include <vector>
 
-#include "rclcpp/allocator/allocator_deleter.hpp"
 #include "rclcpp/experimental/subscription_intra_process.hpp"
 #include "rclcpp/experimental/subscription_intra_process_base.hpp"
 #include "rclcpp/logger.hpp"

rclcpp/include/rclcpp/experimental/buffers/intra_process_buffer.hpp

@@ -18,7 +18,6 @@
 #include <memory>
 #include <utility>
 
-#include "rclcpp/allocator/allocator_common.hpp"
 #include "rclcpp/logging.hpp"
 #include "rclcpp/publisher_base.hpp"
 

rclcpp/include/rclcpp/experimental/intra_process_manager.hpp

@@ -20,7 +20,7 @@
 
 #include "rcl/types.h"
 
-#include "rclcpp/allocator/allocator_common.hpp"
+#include "rcl/allocator.h"
 #include "rclcpp/exceptions.hpp"
 #include "rclcpp/macros.hpp"
 #include "rclcpp/serialized_message.hpp"
@@ -61,15 +61,19 @@ class MessageMemoryStrategy
     message_allocator_ = std::make_shared<MessageAlloc>();
     serialized_message_allocator_ = std::make_shared<SerializedMessageAlloc>();
     buffer_allocator_ = std::make_shared<BufferAlloc>();
-    rcutils_allocator_ = allocator::get_rcl_allocator<char, BufferAlloc>(*buffer_allocator_.get());
+    // The rcl allocator interface is incompatible with C++ allocators
+    static_assert(std::is_same<Alloc, std::allocator<void>>::value, "Non-default allocators are not supported");
+    rcutils_allocator_ = rcl_get_default_allocator();
   }
 
   explicit MessageMemoryStrategy(std::shared_ptr<Alloc> allocator)
   {
     message_allocator_ = std::make_shared<MessageAlloc>(*allocator.get());
     serialized_message_allocator_ = std::make_shared<SerializedMessageAlloc>(*allocator.get());
     buffer_allocator_ = std::make_shared<BufferAlloc>(*allocator.get());
-    rcutils_allocator_ = allocator::get_rcl_allocator<char, BufferAlloc>(*buffer_allocator_.get());
+    // The rcl allocator interface is incompatible with C++ allocators
+    static_assert(std::is_same<Alloc, std::allocator<void>>::value, "Non-default allocators are not supported");
+    rcutils_allocator_ = rcl_get_default_allocator();
   }
 
   virtual ~MessageMemoryStrategy() = default;

rclcpp/include/rclcpp/loaned_message.hpp

@@ -28,8 +28,6 @@
 #include "rcl/error_handling.h"
 #include "rcl/publisher.h"
 
-#include "rclcpp/allocator/allocator_common.hpp"
-#include "rclcpp/allocator/allocator_deleter.hpp"
 #include "rclcpp/detail/resolve_use_intra_process.hpp"
 #include "rclcpp/experimental/intra_process_manager.hpp"
 #include "rclcpp/loaned_message.hpp"

rclcpp/include/rclcpp/message_memory_strategy.hpp

@@ -19,9 +19,9 @@
 #include <string>
 #include <vector>
 
+#include "rcl/allocator.h"
 #include "rcl/publisher.h"
 
-#include "rclcpp/allocator/allocator_common.hpp"
 #include "rclcpp/detail/rmw_implementation_specific_publisher_payload.hpp"
 #include "rclcpp/intra_process_setting.hpp"
 #include "rclcpp/qos.hpp"
@@ -72,10 +72,9 @@ struct PublisherOptionsWithAllocator : public PublisherOptionsBase
   to_rcl_publisher_options(const rclcpp::QoS & qos) const
   {
     rcl_publisher_options_t result = rcl_publisher_get_default_options();
-    using AllocatorTraits = std::allocator_traits<Allocator>;
-    using MessageAllocatorT = typename AllocatorTraits::template rebind_alloc<MessageT>;
-    auto message_alloc = std::make_shared<MessageAllocatorT>(*this->get_allocator().get());
-    result.allocator = rclcpp::allocator::get_rcl_allocator<MessageT>(*message_alloc);
+    // The rcl allocator interface is incompatible with C++ allocators
+    static_assert(std::is_same<Allocator, std::allocator<void>>::value, "Non-default allocators are not supported");
+    result.allocator = rcl_get_default_allocator();
     result.qos = qos.get_rmw_qos_profile();
 
     // Apply payload to rcl_publisher_options if necessary.

rclcpp/include/rclcpp/publisher.hpp

@@ -114,9 +114,6 @@
  *
  * - Exceptions:
  *   - rclcpp/exceptions.hpp
- * - Allocator related items:
- *   - rclcpp/allocator/allocator_common.hpp
- *   - rclcpp/allocator/allocator_deleter.hpp
  * - Memory management tools:
  *   - rclcpp/memory_strategies.hpp
  *   - rclcpp/memory_strategy.hpp

rclcpp/include/rclcpp/publisher_options.hpp

@@ -20,7 +20,6 @@
 
 #include "rcl/allocator.h"
 
-#include "rclcpp/allocator/allocator_common.hpp"
 #include "rclcpp/memory_strategy.hpp"
 #include "rclcpp/node.hpp"
 #include "rclcpp/visibility_control.hpp"
@@ -437,7 +436,7 @@ class AllocatorMemoryStrategy : public memory_strategy::MemoryStrategy
 
   rcl_allocator_t get_allocator() override
   {
-    return rclcpp::allocator::get_rcl_allocator<void *, VoidAlloc>(*allocator_.get());
+    return rcl_get_default_allocator();
   }
 
   size_t number_of_ready_subscriptions() const override

rclcpp/include/rclcpp/rclcpp.hpp

@@ -20,6 +20,8 @@
 #include <string>
 #include <vector>
 
+#include "rcl/allocator.h"
+
 #include "rclcpp/callback_group.hpp"
 #include "rclcpp/detail/rmw_implementation_specific_subscription_payload.hpp"
 #include "rclcpp/intra_process_buffer_type.hpp"
@@ -99,10 +101,9 @@ struct SubscriptionOptionsWithAllocator : public SubscriptionOptionsBase
   to_rcl_subscription_options(const rclcpp::QoS & qos) const
   {
     rcl_subscription_options_t result = rcl_subscription_get_default_options();
-    using AllocatorTraits = std::allocator_traits<Allocator>;
-    using MessageAllocatorT = typename AllocatorTraits::template rebind_alloc<MessageT>;
-    auto message_alloc = std::make_shared<MessageAllocatorT>(*allocator.get());
-    result.allocator = allocator::get_rcl_allocator<MessageT>(*message_alloc);
+    // The rcl allocator interface is incompatible with C++ allocators
+    static_assert(std::is_same<Allocator, std::allocator<void>>::value, "Non-default allocators are not supported");
+    result.allocator = rcl_get_default_allocator();
     result.qos = qos.get_rmw_qos_profile();
     result.rmw_subscription_options.ignore_local_publications = this->ignore_local_publications;
 

rclcpp/include/rclcpp/strategies/allocator_memory_strategy.hpp

@@ -27,8 +27,6 @@
 #include "rcutils/logging_macros.h"
 #include "rmw/impl/cpp/demangle.hpp"
 
-#include "rclcpp/allocator/allocator_common.hpp"
-#include "rclcpp/allocator/allocator_deleter.hpp"
 #include "rclcpp/exceptions.hpp"
 #include "rclcpp/expand_topic_or_service_name.hpp"
 #include "rclcpp/experimental/intra_process_manager.hpp"

rclcpp/include/rclcpp/subscription_options.hpp

@@ -21,7 +21,6 @@
 #include <vector>
 
 #define RCLCPP_BUILDING_LIBRARY 1
-#include "rclcpp/allocator/allocator_common.hpp"
 #include "rclcpp/macros.hpp"
 #include "rclcpp/qos.hpp"
 #include "rmw/types.h"

rclcpp/src/rclcpp/publisher_base.cpp

@@ -20,7 +20,6 @@
 
 #include "rcl/types.h"
 
-#include "rclcpp/allocator/allocator_common.hpp"
 #include "rclcpp/serialized_message.hpp"
 #include "rclcpp/subscription_traits.hpp"