Do NOT clone the implementation

include/aidge/backend/OperatorImpl.hpp

@@ -20,10 +20,6 @@
 namespace Aidge {
 class OperatorImpl {
 public:
-    /**
-     * @brief Clone the implementation using its copy-constructor.
-     */
-    virtual std::unique_ptr<OperatorImpl> clone() const = 0;
 
     virtual void forward(){};
     virtual void backward(){};

include/aidge/operator/Operator.hpp

@@ -42,7 +42,9 @@ public:
     std::enable_shared_from_this<Operator>()
   {
     mType = op.mType;
-    mImpl = op.mImpl->clone();
+    // Implementation is never cloned. It is up to the non-abstract Operator copy-constructor to create a new implementation matching the copied Operator implementation.
+    // See https://gitlab.eclipse.org/eclipse/aidge/aidge_core/-/merge_requests/8#note_1214050 for the discussion.
+    // Hooks are not copied.
   }
 
 public:

src/graph/Node.cpp

@@ -332,13 +332,13 @@ Aidge::NodePtr Aidge::Node::cloneSharedOperators() const {
 Aidge::NodePtr Aidge::Node::cloneSharedProducers() const {
     std::shared_ptr<Operator> op = (mOperator->type() == Producer_Op::Type)
         ? mOperator
-        : std::shared_ptr<Operator>(mOperator->clone());
+        : mOperator->clone();
 
     return std::make_shared<Node>(op, mName);
 }
 
 Aidge::NodePtr Aidge::Node::clone() const {
-    return std::make_shared<Node>(std::shared_ptr<Operator>(mOperator->clone()), mName);
+    return std::make_shared<Node>(mOperator->clone(), mName);
 }
 
 /////////////////////////////////////////////////////////////////////////////////////////////