f - improve sign_justice_tx docs

Author: alecchendev

lightning/src/chain/channelmonitor.rs

@@ -1457,11 +1457,16 @@ impl<Signer: WriteableEcdsaChannelSigner> ChannelMonitor<Signer> {
 
 	/// Wrapper around [`EcdsaChannelSigner::sign_justice_revoked_output`] to make
 	/// signing the justice transaction easier for implementors of
-	/// [`chain::chainmonitor::Persist`]. On success this method returns a fully signed
-	/// transaction that is ready to be broadcasted.
+	/// [`chain::chainmonitor::Persist`]. On success this method returns the provided transaction
+	/// signing the input at `input_idx`.
+	///
+	/// `Value` is the value of the output being spent by the input at `input_idx`, committed
+	/// in the BIP 143 signature.
 	///
 	/// This method will only succeed if this monitor has received the revocation secret for the
-	/// provided commitment number.
+	/// provided `commitment_number`. If a commitment number is provided that does not correspond
+	/// to the commitment transaction being revoked, this will return a signed transaction, but
+	/// the signature will not be valid.
 	///
 	/// [`EcdsaChannelSigner::sign_justice_revoked_output`]: crate::sign::EcdsaChannelSigner::sign_justice_revoked_output
 	/// [`Persist`]: crate::chain::chainmonitor::Persist