refactor: Use Arc::clone() instead of .clone()

in order to clearly mark that no costly deepcopy is performed.

examples/ros2/simple-publisher.rs

@@ -89,5 +89,5 @@ fn main() -> Result<(), RclrsError> {
             .publish_data(sine.next().unwrap())
             .unwrap();
     });
-    rclrs::spin(publisher.node.clone())
+    rclrs::spin(Arc::clone(&publisher.node))
 }

examples/ros2/simple-subscriber.rs

@@ -54,5 +54,5 @@ impl Subscriber {
 fn main() -> Result<(), RclrsError> {
     let subscription =
         Arc::new(Subscriber::new("joint_states_subscriber", "joint_states").unwrap());
-    rclrs::spin(subscription.node.clone())
+    rclrs::spin(Arc::clone(&subscription.node))
 }

src/main.rs

@@ -14,7 +14,7 @@ const FREQUENCY_HZ: u64 = 1000;
 fn main() -> Result<(), RclrsError> {
     let context = Context::new(env::args()).unwrap();
     let node = create_node(&context, "voraus_bridge_node")?;
-    let node_copy = node.clone();
+    let node_copy = Arc::clone(&node);
     let joint_state_publisher = Arc::new(RosPublisher::new(&node, "joint_states").unwrap());
     let publisher_thread_throttle_us =
         ((1.0 / FREQUENCY_HZ as f64) * 1000.0 * 1000.0).round() as u64;

src/simple_opc_ua_subscriber.rs

@@ -37,7 +37,7 @@ pub fn launch_subscriber() -> Result<(), ()> {
         ),
         IdentityToken::Anonymous,
     ) {
-        if let Err(result) = subscribe_to_variables(session.clone(), 2) {
+        if let Err(result) = subscribe_to_variables(Arc::clone(&session), 2) {
             println!(
                 "ERROR: Got an error while subscribing to variables - {}",
                 result