Avoid returning a reference to a u64. 2022-09-slices-not-vecs
authorMatt Corallo <git@bluematt.me>
Sun, 18 Sep 2022 13:55:08 +0000 (13:55 +0000)
committerMatt Corallo <git@bluematt.me>
Mon, 19 Sep 2022 09:23:26 +0000 (09:23 +0000)
commit85eb1fde563b3768370b8dc4e19c2ebe8d2506ee
treebb86b630586f00d981ede114f18bc1251a6edf1f
parent4910c7cffee4ce35de44c0bef361c2d822daacde
Avoid returning a reference to a u64.

In c353c3ed7c40e689a3b9fb6730c6dabbd3c92cc5 an accessor method was
added which returns an `Option<&u64>`. While this allows Rust to
return an 8-byte object, returning a reference to something
pointer-sized is a somewhat strange API.

Instead, we opt for a straight `Option<u64>`, which is sadly
somewhat larger on the stack, but is simpler and already supported
in the bindings generation.
lightning-invoice/src/payment.rs