Add ReadDestTemp function to make it easier to call ReadField with less boilerplate #81

pull ryanofsky wants to merge 1 commits into bitcoin-core:master from ryanofsky:pr/tempdest changing 5 files +85 −7

ryanofsky commented at 6:21 PM on September 26, 2022: collaborator

Add RestDestTemp function to simplify ReadField calls that just want to read a field into a new object.

Instead of requiring ReadField callers to construct the object in advance and pass it with ReadDestValue, or to write a custom emplace function and pass it with ReadDestEmplace, they can use RestDestTemp to make ReadField just construct a temporary object internally that can be moved from.

Add test coverage for this. Also add missing documentation comments for ReadDestEmplace and ReadDestValue classes.

Add ReadDestTemp function to make it easier to call ReadField with less boilerplate

Add RestDestTemp function to simplify ReadField callers that just want to read a
field into a new object. Instead of requiring ReadField callers to construct the
object in advance and pass with ReadDestValue, or to write custom emplace
function with ReadDestEmplace, they call RestDestTemp to make ReadField just
return a temporary object that can be moved from.

Add some test coverage for this. Also add missing documentation for
ReadDestEmplace and ReadDestValue classes.

70d108bc60

ryanofsky merged this on Oct 4, 2022
ryanofsky closed this on Oct 4, 2022
bitcoin-core locked this on Jun 25, 2025

Contributors

ryanofsky