f Don't take by reference if we'd need to clone

Author: tnull

src/lib.rs

@@ -47,7 +47,7 @@
 //!
 //! 	let node_id = PublicKey::from_str("NODE_ID").unwrap();
 //! 	let node_addr = "IP_ADDR:PORT".parse().unwrap();
-//! 	node.connect_open_channel(&node_id, &node_addr, 10000, false).unwrap();
+//! 	node.connect_open_channel(node_id, node_addr, 10000, false).unwrap();
 //!
 //! 	let invoice = Invoice::from_str("INVOICE_STR").unwrap();
 //! 	node.send_payment(&invoice).unwrap();
@@ -847,7 +847,7 @@ impl Node {
 	///
 	/// Returns a temporary channel id
 	pub fn connect_open_channel(
-		&self, node_id: &PublicKey, address: &SocketAddr, channel_amount_sats: u64,
+		&self, node_id: PublicKey, address: SocketAddr, channel_amount_sats: u64,
 		announce_channel: bool,
 	) -> Result<(), Error> {
 		let runtime_lock = self.running.read().unwrap();
@@ -863,7 +863,7 @@ impl Node {
 			return Err(Error::InsufficientFunds);
 		}
 
-		let peer_info = PeerInfo { pubkey: node_id.clone(), address: address.clone() };
+		let peer_info = PeerInfo { pubkey: node_id, address };
 
 		let con_peer_pubkey = peer_info.pubkey.clone();
 		let con_peer_addr = peer_info.address.clone();

src/test/functional_tests.rs

@@ -33,7 +33,7 @@ fn channel_full_cycle() {
 
 	println!("\nA -- connect_open_channel -> B");
 	node_a
-		.connect_open_channel(&node_b.node_id(), &node_b.listening_address().unwrap(), 50000, true)
+		.connect_open_channel(node_b.node_id(), *node_b.listening_address().unwrap(), 50000, true)
 		.unwrap();
 
 	expect_event!(node_a, ChannelPending);
@@ -217,8 +217,8 @@ fn channel_open_fails_when_funds_insufficient() {
 	assert_eq!(
 		Err(Error::InsufficientFunds),
 		node_a.connect_open_channel(
-			&node_b.node_id(),
-			&node_b.listening_address().unwrap(),
+			node_b.node_id(),
+			*node_b.listening_address().unwrap(),
 			120000,
 			true
 		)