Skip to content

Commit 4d9deff

Browse files
committed
Switch RequiredWrapper to LengthReadable
In a subsequent commit, we will be storing `TrampolineOnionPacket`s within `PendingHTLCRouting`, requiring that they be serialized for storage. To do so, `RequiredWrapper`'s requirements must be loosened to only require `LengthReadable` instead of `Readable`.
1 parent ca40276 commit 4d9deff

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

Diff for: lightning/src/util/ser.rs

+3-3
Original file line numberDiff line numberDiff line change
@@ -403,10 +403,10 @@ impl<T: Readable> MaybeReadable for T {
403403
///
404404
/// This is not exported to bindings users as manual TLV building is not currently supported in bindings
405405
pub struct RequiredWrapper<T>(pub Option<T>);
406-
impl<T: Readable> Readable for RequiredWrapper<T> {
406+
impl<T: LengthReadable> LengthReadable for RequiredWrapper<T> {
407407
#[inline]
408-
fn read<R: Read>(reader: &mut R) -> Result<Self, DecodeError> {
409-
Ok(Self(Some(Readable::read(reader)?)))
408+
fn read_from_fixed_length_buffer<R: LengthRead>(reader: &mut R) -> Result<Self, DecodeError> {
409+
Ok(Self(Some(LengthReadable::read_from_fixed_length_buffer(reader)?)))
410410
}
411411
}
412412
impl<A, T: ReadableArgs<A>> ReadableArgs<A> for RequiredWrapper<T> {

0 commit comments

Comments
 (0)