rice 4.6.1 → 4.7.0

data/rice/Arg.ipp

@@ -70,14 +70,14 @@ namespace Rice
     return this->isOwner_;
   }
 
-  inline Arg& Arg::setArray()
+  inline Arg& Arg::setBuffer()
   {
-    this->isArray_ = true;
+    this->isBuffer_ = true;
     return *this;
   }
 
-  inline bool Arg::isArray()
+  inline bool Arg::isBuffer()
   {
-    return this->isArray_;
+    return this->isBuffer_;
   }
 } // Rice

data/rice/Buffer.hpp

@@ -7,14 +7,13 @@ namespace Rice
   class Buffer;
     
   template<typename T>
-  class Buffer<T, std::enable_if_t<!std::is_pointer_v<T>>>
+  class Buffer<T, std::enable_if_t<!std::is_pointer_v<T> && !std::is_void_v<T>>>
   {
   public:
-    using Element_T = T;
-
     Buffer(T* pointer);
     Buffer(T* pointer, size_t size);
     Buffer(VALUE value);
+    Buffer(VALUE value, size_t size);
 
     ~Buffer();
 
@@ -27,10 +26,9 @@ namespace Rice
 
     T* ptr();
     T& reference();
-    void release();
+    T* release();
 
     size_t size() const;
-    void setSize(size_t value);
 
     // Ruby API
     VALUE toString() const;
@@ -45,24 +43,24 @@ namespace Rice
     void setOwner(bool value);
 
   private:
-    void fromBuiltinType(VALUE value);
-    void fromWrappedType(VALUE value);
+    void fromBuiltinType(VALUE value, size_t size);
+    void fromWrappedType(VALUE value, size_t size);
 
     bool m_owner = false;
     size_t m_size = 0;
-    // std::unique_ptr would be greate but std::unique_ptr<void> isn't allowed
-    T* m_buffer = nullptr;
+    // std::unique_ptr would be great but std::unique_ptr<void> isn't allowed. Mutable is needed to
+    // support const T* buffers
+    mutable T* m_buffer = nullptr;
   };
 
   template<typename T>
   class Buffer<T*, std::enable_if_t<!detail::is_wrapped_v<T>>>
   {
   public:
-    using Element_T = Buffer<T>;
-
     Buffer(T** pointer);
     Buffer(T** pointer, size_t size);
     Buffer(VALUE value);
+    Buffer(VALUE value, size_t size);
 
     ~Buffer();
 
@@ -72,13 +70,12 @@ namespace Rice
     Buffer& operator=(const Buffer& other) = delete;
     Buffer& operator=(Buffer&& other);
 
-    Element_T& operator[](size_t index);
+    T* operator[](size_t index);
 
     T** ptr();
-    void release();
+    T** release();
 
     size_t size() const;
-    void setSize(size_t value);
 
     // Ruby API
     VALUE toString() const;
@@ -95,19 +92,17 @@ namespace Rice
   private:
     bool m_owner = false;
     size_t m_size = 0;
-    T** m_outer = nullptr;
-    std::vector<Buffer<T>> m_inner;
+    T** m_buffer = nullptr;
   };
 
   template<typename T>
   class Buffer<T*, std::enable_if_t<detail::is_wrapped_v<T>>>
   {
   public:
-    using Element_T = T*;
-
     Buffer(T** pointer);
     Buffer(T** pointer, size_t size);
     Buffer(VALUE value);
+    Buffer(VALUE value, size_t size);
 
     ~Buffer();
 
@@ -117,13 +112,12 @@ namespace Rice
     Buffer& operator=(const Buffer& other) = delete;
     Buffer& operator=(Buffer&& other);
 
-    Element_T& operator[](size_t index);
+    T* operator[](size_t index);
 
     T** ptr();
-    void release();
+    T** release();
 
     size_t size() const;
-    void setSize(size_t value);
 
     // Ruby API
     VALUE toString() const;
@@ -143,26 +137,36 @@ namespace Rice
     T** m_buffer = nullptr;
   };
 
-  template<>
-  class Buffer<void>
+  template<typename T>
+  class Buffer<T, std::enable_if_t<std::is_void_v<T>>>
   {
   public:
-    Buffer(void* pointer);
+    Buffer(T* pointer);
+    Buffer(VALUE value);
+    Buffer(VALUE value, size_t size);
+
     Buffer(const Buffer& other) = delete;
     Buffer(Buffer&& other);
 
     Buffer& operator=(const Buffer& other) = delete;
     Buffer& operator=(Buffer&& other);
 
-    void* ptr();
+    size_t size() const;
+      
+    VALUE bytes(size_t count) const;
+    VALUE bytes() const;
+
+    T* ptr();
+    T* release();
 
   private:
-    void* m_buffer = nullptr;
+    bool m_owner = false;
+    size_t m_size = 0;
+    T* m_buffer = nullptr;
   };
 
   template<typename T>
   Data_Type<Buffer<T>> define_buffer(std::string klassName = "");
-
-  void define_fundamental_buffer_types();
 }
+
 #endif // Rice__Buffer__hpp_