Remove broadcaster_dust_limit_sat, feerate_per_kw from CommitmentStats

Author: tankyleo

lightning/src/ln/channel.rs

@@ -1130,10 +1130,8 @@ struct CommitmentData<'a> {
 
 /// A struct gathering stats on a commitment transaction, either local or remote.
 struct CommitmentStats {
-	feerate_per_kw: u32,             // the feerate of the commitment transaction
-	total_fee_sat: u64,              // the total fee included in the transaction
-	total_anchors_sat: u64,          // the sum of the anchors' amounts
-	broadcaster_dust_limit_sat: u64, // the broadcaster's dust limit
+	total_fee_sat: u64,     // the total fee included in the transaction
+	total_anchors_sat: u64, // the sum of the anchors' amounts
 	local_balance_before_fee_anchors_msat: u64, // local balance before fees and anchors *not* considering dust limits
 	remote_balance_before_fee_anchors_msat: u64, // remote balance before fees and anchors *not* considering dust limits
 }
@@ -4424,17 +4422,8 @@ where
 		Ok(())
 	}
 
-	/// Builds stats on a potential commitment transaction build, without actually building the
-	/// commitment transaction. See `build_commitment_transaction` for further docs.
 	#[inline]
-	#[rustfmt::skip]
-	fn build_commitment_stats(&self, funding: &FundingScope, local: bool, generated_by_local: bool) -> CommitmentStats {
-		let broadcaster_dust_limit_sat = if local { self.holder_dust_limit_satoshis } else { self.counterparty_dust_limit_satoshis };
-		let mut non_dust_htlc_count = 0;
-		let mut remote_htlc_total_msat = 0;
-		let mut local_htlc_total_msat = 0;
-		let mut value_to_self_msat_offset = 0;
-
+	fn get_commitment_feerate(&self, funding: &FundingScope, generated_by_local: bool) -> u32 {
 		let mut feerate_per_kw = self.feerate_per_kw;
 		if let Some((feerate, update_state)) = self.pending_update_fee {
 			if match update_state {
@@ -4448,6 +4437,22 @@ where
 			}
 		}
 
+		feerate_per_kw
+	}
+
+	/// Builds stats on a potential commitment transaction build, without actually building the
+	/// commitment transaction. See `build_commitment_transaction` for further docs.
+	#[inline]
+	#[rustfmt::skip]
+	fn build_commitment_stats(&self, funding: &FundingScope, local: bool, generated_by_local: bool) -> CommitmentStats {
+		let broadcaster_dust_limit_sat = if local { self.holder_dust_limit_satoshis } else { self.counterparty_dust_limit_satoshis };
+		let mut non_dust_htlc_count = 0;
+		let mut remote_htlc_total_msat = 0;
+		let mut local_htlc_total_msat = 0;
+		let mut value_to_self_msat_offset = 0;
+
+		let feerate_per_kw = self.get_commitment_feerate(funding, generated_by_local);
+
 		for htlc in self.pending_inbound_htlcs.iter() {
 			if htlc.state.included_in_commitment(generated_by_local) {
 				if !htlc.is_dust(local, feerate_per_kw, broadcaster_dust_limit_sat, funding.get_channel_type()) {
@@ -4505,10 +4510,8 @@ where
 		let total_anchors_sat = if funding.channel_transaction_parameters.channel_type_features.supports_anchors_zero_fee_htlc_tx() { ANCHOR_OUTPUT_VALUE_SATOSHI * 2 } else { 0 };
 
 		CommitmentStats {
-			feerate_per_kw,
 			total_fee_sat,
 			total_anchors_sat,
-			broadcaster_dust_limit_sat,
 			local_balance_before_fee_anchors_msat: value_to_self_msat,
 			remote_balance_before_fee_anchors_msat: value_to_remote_msat,
 		}
@@ -4532,12 +4535,13 @@ where
 	fn build_commitment_transaction<L: Deref>(&self, funding: &FundingScope, commitment_number: u64, per_commitment_point: &PublicKey, local: bool, generated_by_local: bool, logger: &L) -> CommitmentData
 		where L::Target: Logger
 	{
+		let broadcaster_dust_limit_sat = if local { self.holder_dust_limit_satoshis } else { self.counterparty_dust_limit_satoshis };
+		let feerate_per_kw = self.get_commitment_feerate(funding, generated_by_local);
+
 		let stats = self.build_commitment_stats(funding, local, generated_by_local);
 		let CommitmentStats {
-			feerate_per_kw,
 			total_fee_sat,
 			total_anchors_sat,
-			broadcaster_dust_limit_sat,
 			local_balance_before_fee_anchors_msat,
 			remote_balance_before_fee_anchors_msat
 		} = stats;