Skip to content

Commit 6c45e4f

Browse files
committed
Minor comments added, review
1 parent 25f4103 commit 6c45e4f

File tree

2 files changed

+2
-0
lines changed

2 files changed

+2
-0
lines changed

lightning/src/ln/channel.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1757,6 +1757,7 @@ impl<SP: Deref> PendingV2Channel<SP> where SP::Target: SignerProvider {
17571757
let change_output_weight = get_output_weight(&change_output.script_pubkey).to_wu();
17581758
let change_output_fee = fee_for_weight(self.dual_funding_context.funding_feerate_sat_per_1000_weight, change_output_weight);
17591759
change_output.value = Amount::from_sat(change_value.saturating_sub(change_output_fee));
1760+
// Note: dust check not done here, should be handled before
17601761
funding_outputs.push(OutputOwned::Single(change_output));
17611762
}
17621763

lightning/src/ln/interactivetxs.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1694,6 +1694,7 @@ pub(super) fn calculate_change_output_value(
16941694
let total_input_satoshis: u64 =
16951695
funding_inputs_prev_outputs.iter().map(|out| out.value.to_sat()).sum();
16961696

1697+
// Note: in case of additional outputs, they will have to be subtracted here
16971698
let remaining_value =
16981699
total_input_satoshis.saturating_sub(our_contribution).saturating_sub(fees_sats);
16991700

0 commit comments

Comments
 (0)