consensus lib work: split stream classes out of serialize.h #5119

pull theuni wants to merge 1 commits into bitcoin:master from theuni:reducedeps6 changing 16 files +589 −553

theuni commented at 8:43 PM on October 22, 2014: member

serialization now has no dependencies.
boost: split stream classes out of serialize.h
serialization now has no dependencies.
fa7361907a
theuni commented at 5:08 AM on October 23, 2014: member

this is move-only, btw. With the exception of a few obvious header changes and forward-declares.
laanwj added the label Improvement on Oct 24, 2014
in src/streams.h:None in fa7361907a
```
   0 | @@ -0,0 +1,571 @@
   1 | +// Copyright (c) 2009-2010 Satoshi Nakamoto
```
laanwj commented at 2:29 PM on October 24, 2014:

Thanks for moving these to a separate include file. Indeed, they don't belong with serialize itself.
sipa commented at 5:24 PM on October 24, 2014: member

Concept ACK; I didn't verify the move.
laanwj commented at 10:42 AM on October 27, 2014: member

ACK, verified move-only apart from (necessary) #include, comment and pre-declared classes changes.
sipa commented at 11:10 AM on October 27, 2014: member

ACK, verified that the code in streams.h is move-only.
sipa merged this on Oct 27, 2014
sipa closed this on Oct 27, 2014
sipa referenced this in commit 60abd463ac on Oct 27, 2014
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels