Provide a reset_on_reload TLV field serialization type

This is useful in the type serialization definition macros to avoid
writing or reading a field at all, simply using a static value on
each reload.

Author: TheBlueMatt

lightning/src/util/ser_macros.rs

@@ -11,6 +11,9 @@ macro_rules! encode_tlv {
 	($stream: expr, $type: expr, $field: expr, (default_value, $default: expr)) => {
 		encode_tlv!($stream, $type, $field, required)
 	};
+	($stream: expr, $type: expr, $field: expr, (reset_on_reload, $value: expr)) => {
+		let _ = &$field; // Ensure we "use" the $field
+	};
 	($stream: expr, $type: expr, $field: expr, required) => {
 		BigSize($type).write($stream)?;
 		BigSize($field.serialized_length() as u64).write($stream)?;
@@ -28,6 +31,17 @@ macro_rules! encode_tlv {
 	};
 }
 
+
+macro_rules! _check_tlv_ordering {
+	($last_type: expr, $type: expr, (reset_on_reload, $value: expr)) => { };
+	($last_type: expr, $type: expr, $fieldty: tt) => {
+		if let Some(t) = $last_type {
+			debug_assert!(t <= $type);
+		}
+		$last_type = Some($type);
+	};
+}
+
 macro_rules! encode_tlv_stream {
 	($stream: expr, {$(($type: expr, $field: expr, $fieldty: tt)),* $(,)*}) => { {
 		#[allow(unused_imports)]
@@ -46,10 +60,7 @@ macro_rules! encode_tlv_stream {
 		{
 			let mut last_seen: Option<u64> = None;
 			$(
-				if let Some(t) = last_seen {
-					debug_assert!(t <= $type);
-				}
-				last_seen = Some($type);
+				_check_tlv_ordering!(last_seen, $type, $fieldty);
 			)*
 		}
 	} }
@@ -59,6 +70,8 @@ macro_rules! get_varint_length_prefixed_tlv_length {
 	($len: expr, $type: expr, $field: expr, (default_value, $default: expr)) => {
 		get_varint_length_prefixed_tlv_length!($len, $type, $field, required)
 	};
+	($len: expr, $type: expr, $field: expr, (reset_on_reload, $value: expr)) => {
+	};
 	($len: expr, $type: expr, $field: expr, required) => {
 		BigSize($type).write(&mut $len).expect("No in-memory data may fail to serialize");
 		let field_len = $field.serialized_length();
@@ -102,6 +115,8 @@ macro_rules! check_tlv_order {
 			$field = $default.into();
 		}
 	}};
+	($last_seen_type: expr, $typ: expr, $type: expr, $field: ident, (reset_on_reload, $value: expr)) => {
+	};
 	($last_seen_type: expr, $typ: expr, $type: expr, $field: ident, required) => {{
 		#[allow(unused_comparisons)] // Note that $type may be 0 making the second comparison always true
 		let invalid_order = ($last_seen_type.is_none() || $last_seen_type.unwrap() < $type) && $typ.0 > $type;
@@ -131,6 +146,9 @@ macro_rules! check_missing_tlv {
 			$field = $default.into();
 		}
 	}};
+	($last_seen_type: expr, $type: expr, $field: expr, (reset_on_reload, $value: expr)) => {
+		$field = $value;
+	};
 	($last_seen_type: expr, $type: expr, $field: ident, required) => {{
 		#[allow(unused_comparisons)] // Note that $type may be 0 making the second comparison always true
 		let missing_req_type = $last_seen_type.is_none() || $last_seen_type.unwrap() < $type;
@@ -156,6 +174,8 @@ macro_rules! decode_tlv {
 	($reader: expr, $field: ident, (default_value, $default: expr)) => {{
 		decode_tlv!($reader, $field, required)
 	}};
+	($reader: expr, $field: ident, (reset_on_reload, $value: expr)) => {{
+	}};
 	($reader: expr, $field: ident, required) => {{
 		$field = $crate::util::ser::Readable::read(&mut $reader)?;
 	}};
@@ -174,6 +194,11 @@ macro_rules! decode_tlv {
 	}};
 }
 
+macro_rules! _decode_tlv_stream_match_check {
+	($val: ident, $type: expr, (reset_on_reload, $value: expr)) => { false };
+	($val: ident, $type: expr, $fieldty: tt) => { $val == $type }
+}
+
 // `$decode_custom_tlv` is a closure that may be optionally provided to handle custom message types.
 // If it is provided, it will be called with the custom type and the `FixedLengthReader` containing
 // the message contents. It should return `Ok(true)` if the custom message is successfully parsed,
@@ -224,7 +249,7 @@ macro_rules! decode_tlv_stream {
 			let length: ser::BigSize = $crate::util::ser::Readable::read(&mut stream_ref)?;
 			let mut s = ser::FixedLengthReader::new(&mut stream_ref, length.0);
 			match typ.0 {
-				$($type => {
+				$(_t if _decode_tlv_stream_match_check!(_t, $type, $fieldty) => {
 					decode_tlv!(s, $field, $fieldty);
 					if s.bytes_remain() {
 						s.eat_remaining()?; // Return ShortRead if there's actually not enough bytes
@@ -365,6 +390,9 @@ macro_rules! init_tlv_based_struct_field {
 	($field: ident, (default_value, $default: expr)) => {
 		$field.0.unwrap()
 	};
+	($field: ident, (reset_on_reload, $value: expr)) => {
+		$field
+	};
 	($field: ident, option) => {
 		$field
 	};
@@ -380,6 +408,9 @@ macro_rules! init_tlv_field_var {
 	($field: ident, (default_value, $default: expr)) => {
 		let mut $field = $crate::util::ser::OptionDeserWrapper(None);
 	};
+	($field: ident, (reset_on_reload, $value: expr)) => {
+		let $field;
+	};
 	($field: ident, required) => {
 		let mut $field = $crate::util::ser::OptionDeserWrapper(None);
 	};