Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Aug 1, 2016

This fixes my nits from #8417... The largest chunk is moving most of the code into read_dump, to get rid of the duplicated code.

@laanwj
Copy link
Member

laanwj commented Aug 3, 2016

utACK fa4439d

@laanwj laanwj merged commit fa4439d into bitcoin:master Aug 3, 2016
laanwj added a commit that referenced this pull request Aug 3, 2016
fa4439d [qa] Rework hd wallet dump test (MarcoFalke)
@maflcko maflcko deleted the Mf1608-qaWalletDump branch August 3, 2016 10:54
luke-jr pushed a commit to luke-jr/bitcoin that referenced this pull request Sep 21, 2016
codablock pushed a commit to codablock/dash that referenced this pull request Feb 1, 2018
fa4439d [qa] Rework hd wallet dump test (MarcoFalke)
andvgal pushed a commit to energicryptocurrency/gen2-energi that referenced this pull request Jan 6, 2019
fa4439d [qa] Rework hd wallet dump test (MarcoFalke)
CryptoCentric pushed a commit to absolute-community/absolute that referenced this pull request Feb 28, 2019
fa4439d [qa] Rework hd wallet dump test (MarcoFalke)
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants