faster add objects

Author: chokomancarr

RadeonRays/include/radeon_rays.h

@@ -225,10 +225,14 @@ namespace RadeonRays
         // Create an instance of a shape with its own transform (set via Shape interface).
         // The call is blocking, so the returned value is ready upon return.
         virtual Shape* CreateInstance(Shape const* shape) const = 0;
+        // *EDIT* Preallocate shape container
+        virtual void AllocShapes(size_t const size) {}
         // Delete the shape (to simplify DLL boundary crossing
         virtual void DeleteShape(Shape const* shape) = 0;
         // Attach shape to participate in intersection process
         virtual void AttachShape(Shape const* shape) = 0;
+        // *EDIT* Attach shape without error checking
+        virtual void AttachShapeUnchecked(Shape const* shape) { AttachShape(shape); }
         // Detach shape, i.e. it is not going to be considered part of the scene anymore
         virtual void DetachShape(Shape const* shape) = 0;
         // Detach all objects

RadeonRays/src/api/radeon_rays_impl.cpp

@@ -87,6 +87,11 @@ namespace RadeonRays
         return instance;
     }
 
+    void IntersectionApiImpl::AllocShapes(size_t const size)
+    {
+        world_.ReserveShapes(size);
+    }
+
     void IntersectionApiImpl::DeleteShape(Shape const* shape)
     {
         delete shape;
@@ -97,6 +102,11 @@ namespace RadeonRays
         world_.AttachShape(shape);
     }
 
+    void IntersectionApiImpl::AttachShapeUnchecked(Shape const* shape)
+    {
+        world_.AttachShapeUnchecked(shape);
+    }
+
     void IntersectionApiImpl::DetachShape(Shape const* shape)
     {
         world_.DetachShape(shape);

RadeonRays/src/api/radeon_rays_impl.h

@@ -65,10 +65,14 @@ namespace RadeonRays
         // Create an instance of a shape with its own transform (set via Shape interface).
         // The call is blocking, so the returned value is ready upon return.
         Shape* CreateInstance(Shape const* shape) const override;
+        // *EDIT* Preallocate shape container
+        void AllocShapes(size_t const size) override;
         // Delete the shape (to simplify DLL boundary crossing
         void DeleteShape(Shape const* shape) override;
         // Attach shape to participate in intersection process
         void AttachShape(Shape const* shape) override;
+        // *EDIT* Attach shape without error checking
+        void AttachShapeUnchecked(Shape const* shape) override;
         // Detach shape, i.e. it is not going to be considered part of the scene anymore
         void DetachShape(Shape const* shape) override;
         // Detach all objects

RadeonRays/src/world/world.cpp

@@ -25,6 +25,11 @@ THE SOFTWARE.
 
 namespace RadeonRays
 {
+    void World::ReserveShapes(size_t const size)
+    {
+        shapes_.reserve(size);
+    }
+
     void World::AttachShape(Shape const* shape)
     {
         if (std::find(shapes_.cbegin(), shapes_.cend(), shape) == shapes_.cend())
@@ -34,6 +39,12 @@ namespace RadeonRays
         }
     }
 
+    void World::AttachShapeUnchecked(Shape const* shape)
+    {
+        shapes_.push_back(shape);
+        has_changed_ = true;
+    }
+
     void World::DetachShape(Shape const* shape)
     {
         auto iter = std::find(shapes_.begin(), shapes_.end(), shape);

RadeonRays/src/world/world.h

@@ -42,8 +42,12 @@ namespace RadeonRays
         World() = default;
         //
         virtual ~World() = default;
+        // *EDIT* Preallocate the shapes container
+        void ReserveShapes(size_t const size);
         // Attach the shape updating all the flags
         void AttachShape(Shape const* shape);
+        // *EDIT* Attach the shape updating all the flags, without checking if shape already exists.
+        void AttachShapeUnchecked(Shape const* shape);
         // Detach the shape 
         void DetachShape(Shape const* shape);
         // Detach all