pass shared ptrs by reference

Core/SampleBuilderEngine/SampleBuilderNode.cpp

@@ -46,7 +46,7 @@ SampleBuilderNode& SampleBuilderNode::operator=(const SampleBuilderNode& other)
 
 //! Sets sample builder and borrows its parameters.
 
-void SampleBuilderNode::setSBN(builder_t sample_builder)
+void SampleBuilderNode::setSBN(const builder_t& sample_builder)
 {
     if (!sample_builder)
         throw std::runtime_error("SampleContainer::setSampleBuilder() -> Error. "

Core/SampleBuilderEngine/SampleBuilderNode.h

@@ -25,14 +25,15 @@ class ISampleBuilder;
 
 class SampleBuilderNode : public INode
 {
-public:
     using builder_t = std::shared_ptr<ISampleBuilder>;
 
+public:
+
     SampleBuilderNode();
     SampleBuilderNode(const SampleBuilderNode& other);
     SampleBuilderNode& operator=(const SampleBuilderNode& other);
 
-    void setSBN(builder_t sample_builder);
+    void setSBN(const builder_t& sample_builder);
 
     void reset();
 

Core/SampleBuilderEngine/SampleProvider.cpp

@@ -13,7 +13,6 @@
 // ************************************************************************** //
 
 #include "Core/SampleBuilderEngine/SampleProvider.h"
-#include "Core/Basics/Assert.h"
 #include "Core/Multilayer/MultiLayer.h"
 
 SampleProvider::SampleProvider() {}
@@ -45,7 +44,7 @@ void SampleProvider::setSample(const MultiLayer& multilayer)
     m_sample_builder.reset();
 }
 
-void SampleProvider::setBuilder(const std::shared_ptr<ISampleBuilder> sample_builder)
+void SampleProvider::setBuilder(const std::shared_ptr<ISampleBuilder>& sample_builder)
 {
     m_sample_builder.setSBN(sample_builder);
     m_sample_builder.setParent(parent());

Core/SampleBuilderEngine/SampleProvider.h

@@ -34,7 +34,7 @@ public:
 
     void setSample(const MultiLayer& multilayer);
 
-    void setBuilder(const std::shared_ptr<ISampleBuilder> sample_builder);
+    void setBuilder(const std::shared_ptr<ISampleBuilder>& sample_builder);
 
     const MultiLayer* sample() const;