f tweak variable name and add comment describing behavior

Author: TheBlueMatt

lightning/src/ln/channel.rs

@@ -3682,9 +3682,11 @@ impl<Signer: Sign> Channel<Signer> {
 	/// force-close the channel, but may also indicate a harmless reorganization of a block or two.
 	pub fn funding_transaction_unconfirmed(&mut self) -> Result<(), msgs::ErrorMessage> {
 		if self.funding_tx_confirmation_height != 0 {
-			let effective_new_height = self.funding_tx_confirmation_height - 1;
+			// We handle the funding disconnection by calling update_best_block with a height one
+			// below where our funding was connected, implying a reorg back to conf_height - 1.
+			let reorg_height = self.funding_tx_confirmation_height - 1;
 			let best_time = self.update_time_counter;
-			match self.update_best_block(effective_new_height, best_time) {
+			match self.update_best_block(reorg_height, best_time) {
 				Ok((funding_locked, timed_out_htlcs)) => {
 					assert!(funding_locked.is_none(), "We can't generate a funding with 0 confirmations?");
 					assert!(timed_out_htlcs.is_empty(), "We can't have accepted HTLCs with a timeout before our funding confirmation?");