X-Git-Url: http://git.bitcoin.ninja/index.cgi?a=blobdiff_plain;f=lightning%2Fsrc%2Fln%2Fchan_utils.rs;h=6e0e50858f8972ff90865eb3c23f5c2ce39f10fa;hb=40959b74b78dd2cf99547c10d2888ce014930b3d;hp=9f98cd03b28af31ec86b58f154bf582942ccf849;hpb=bfd9646092e3c63f07d96df35dd72488af55a176;p=rust-lightning

diff --git a/lightning/src/ln/chan_utils.rs b/lightning/src/ln/chan_utils.rs
index 9f98cd03..6e0e5085 100644
--- a/lightning/src/ln/chan_utils.rs
+++ b/lightning/src/ln/chan_utils.rs
@@ -172,7 +172,7 @@ impl Writeable for CounterpartyCommitmentSecrets {
 			writer.write_all(secret)?;
 			writer.write_all(&byte_utils::be64_to_array(*idx))?;
 		}
-		write_tlv_fields!(writer, {}, {});
+		write_tlv_fields!(writer, {});
 		Ok(())
 	}
 }
@@ -183,7 +183,7 @@ impl Readable for CounterpartyCommitmentSecrets {
 			*secret = Readable::read(reader)?;
 			*idx = Readable::read(reader)?;
 		}
-		read_tlv_fields!(reader, {}, {});
+		read_tlv_fields!(reader, {});
 		Ok(Self { old_secrets })
 	}
 }
@@ -318,12 +318,12 @@ pub struct TxCreationKeys {
 }
 
 impl_writeable_tlv_based!(TxCreationKeys, {
-	(0, per_commitment_point),
-	(2, revocation_key),
-	(4, broadcaster_htlc_key),
-	(6, countersignatory_htlc_key),
-	(8, broadcaster_delayed_payment_key),
-}, {}, {});
+	(0, per_commitment_point, required),
+	(2, revocation_key, required),
+	(4, broadcaster_htlc_key, required),
+	(6, countersignatory_htlc_key, required),
+	(8, broadcaster_delayed_payment_key, required),
+});
 
 /// One counterparty's public keys which do not change over the life of a channel.
 #[derive(Clone, PartialEq)]
@@ -350,12 +350,12 @@ pub struct ChannelPublicKeys {
 }
 
 impl_writeable_tlv_based!(ChannelPublicKeys, {
-	(0, funding_pubkey),
-	(2, revocation_basepoint),
-	(4, payment_point),
-	(6, delayed_payment_basepoint),
-	(8, htlc_basepoint),
-}, {}, {});
+	(0, funding_pubkey, required),
+	(2, revocation_basepoint, required),
+	(4, payment_point, required),
+	(6, delayed_payment_basepoint, required),
+	(8, htlc_basepoint, required),
+});
 
 impl TxCreationKeys {
 	/// Create per-state keys from channel base points and the per-commitment point.
@@ -429,13 +429,12 @@ pub struct HTLCOutputInCommitment {
 }
 
 impl_writeable_tlv_based!(HTLCOutputInCommitment, {
-	(0, offered),
-	(2, amount_msat),
-	(4, cltv_expiry),
-	(6, payment_hash),
-}, {
-	(8, transaction_output_index)
-}, {});
+	(0, offered, required),
+	(2, amount_msat, required),
+	(4, cltv_expiry, required),
+	(6, payment_hash, required),
+	(8, transaction_output_index, option),
+});
 
 #[inline]
 pub(crate) fn get_htlc_redeemscript_with_explicit_keys(htlc: &HTLCOutputInCommitment, broadcaster_htlc_key: &PublicKey, countersignatory_htlc_key: &PublicKey, revocation_key: &PublicKey) -> Script {
@@ -626,18 +625,17 @@ impl ChannelTransactionParameters {
 }
 
 impl_writeable_tlv_based!(CounterpartyChannelTransactionParameters, {
-	(0, pubkeys),
-	(2, selected_contest_delay),
-}, {}, {});
+	(0, pubkeys, required),
+	(2, selected_contest_delay, required),
+});
 
 impl_writeable_tlv_based!(ChannelTransactionParameters, {
-	(0, holder_pubkeys),
-	(2, holder_selected_contest_delay),
-	(4, is_outbound_from_holder),
-}, {
-	(6, counterparty_parameters),
-	(8, funding_outpoint),
-}, {});
+	(0, holder_pubkeys, required),
+	(2, holder_selected_contest_delay, required),
+	(4, is_outbound_from_holder, required),
+	(6, counterparty_parameters, option),
+	(8, funding_outpoint, option),
+});
 
 /// Static channel fields used to build transactions given per-commitment fields, organized by
 /// broadcaster/countersignatory.
@@ -720,11 +718,10 @@ impl PartialEq for HolderCommitmentTransaction {
 }
 
 impl_writeable_tlv_based!(HolderCommitmentTransaction, {
-	(0, inner),
-	(2, counterparty_sig),
-	(4, holder_sig_first),
-}, {}, {
-	(6, counterparty_htlc_sigs),
+	(0, inner, required),
+	(2, counterparty_sig, required),
+	(4, holder_sig_first, required),
+	(6, counterparty_htlc_sigs, vec_type),
 });
 
 impl HolderCommitmentTransaction {
@@ -809,9 +806,9 @@ pub struct BuiltCommitmentTransaction {
 }
 
 impl_writeable_tlv_based!(BuiltCommitmentTransaction, {
-	(0, transaction),
-	(2, txid)
-}, {}, {});
+	(0, transaction, required),
+	(2, txid, required),
+});
 
 impl BuiltCommitmentTransaction {
 	/// Get the SIGHASH_ALL sighash value of the transaction.
@@ -866,14 +863,13 @@ impl PartialEq for CommitmentTransaction {
 }
 
 impl_writeable_tlv_based!(CommitmentTransaction, {
-	(0, commitment_number),
-	(2, to_broadcaster_value_sat),
-	(4, to_countersignatory_value_sat),
-	(6, feerate_per_kw),
-	(8, keys),
-	(10, built),
-}, {}, {
-	(12, htlcs),
+	(0, commitment_number, required),
+	(2, to_broadcaster_value_sat, required),
+	(4, to_countersignatory_value_sat, required),
+	(6, feerate_per_kw, required),
+	(8, keys, required),
+	(10, built, required),
+	(12, htlcs, vec_type),
 });
 
 impl CommitmentTransaction {