Implement Copy on NodeIdx

Cargo.toml

@@ -17,7 +17,7 @@ orx-self-or = "1.2.0"
 serde = { version = "1.0.228", optional = true, default-features = false }
 orx-pinned-vec = { version = "3.21.0", default-features = false }
 orx-split-vec = { version = "3.22.0", default-features = false }
-orx-selfref-col = { version = "2.14.0", default-features = false }
+orx-selfref-col = { version = "2.15.0", default-features = false }
 orx-concurrent-iter = { version = "3.1.0", default-features = false }
 orx-parallel = { version = "3.3.0", default-features = false, optional = true }
 

src/subtrees_within/cloned_subtree.rs

@@ -48,6 +48,6 @@ where
         });
         parent.append_subtree_as_child(subtree, child_position);
 
-        self.idx.clone()
+        self.idx
     }
 }

src/subtrees_within/copied_subtree.rs

@@ -45,6 +45,6 @@ where
         });
         parent.append_subtree_as_child(subtree, child_position);
 
-        self.idx.clone()
+        self.idx
     }
 }

src/subtrees_within/moved_subtree.rs

@@ -49,6 +49,6 @@ impl<V: TreeVariant> SubTreeWithinCore<V> for MovedSubTreeWithin<V> {
         let node_parent = unsafe { &mut *ptr_parent.ptr_mut() };
         node_parent.next_mut().insert(child_position, ptr_child);
 
-        self.idx.clone()
+        self.idx
     }
 }

src/tree_node_idx.rs

@@ -248,7 +248,7 @@ impl<V: TreeVariant> NodeIdx<V> {
     /// [`push_child_tree_within`]: crate::NodeMut::push_child_tree_within
     /// [`push_sibling_tree_within`]: crate::NodeMut::push_sibling_tree_within
     pub fn into_subtree_within(&self) -> MovedSubTreeWithin<V> {
-        MovedSubTreeWithin::new(self.clone())
+        MovedSubTreeWithin::new(*self)
     }
 
     /// Creates a subtree view including this node as the root and all of its descendants with their orientation relative
@@ -264,7 +264,7 @@ impl<V: TreeVariant> NodeIdx<V> {
     where
         V::Item: Clone,
     {
-        ClonedSubTreeWithin::new(self.clone())
+        ClonedSubTreeWithin::new(*self)
     }
 
     /// Creates a subtree view including this node as the root and all of its descendants with their orientation relative
@@ -280,13 +280,16 @@ impl<V: TreeVariant> NodeIdx<V> {
     where
         V::Item: Copy,
     {
-        CopiedSubTreeWithin::new(self.clone())
+        CopiedSubTreeWithin::new(*self)
     }
 }
 
+// Only the pointer is copied, so "V" does not need to be copy itself.
+impl<V: TreeVariant> Copy for NodeIdx<V> {}
+
 impl<V: TreeVariant> Clone for NodeIdx<V> {
     fn clone(&self) -> Self {
-        Self(self.0.clone())
+        *self
     }
 }