Skip to content

fix rpcmining/getblocktemplate univalue transition logic error - #6234

Merged
laanwj merged 1 commit into
bitcoin:masterfrom
jonasschnelli:2015/06/univalue_fix
Jun 5, 2015
Merged

fix rpcmining/getblocktemplate univalue transition logic error#6234
laanwj merged 1 commit into
bitcoin:masterfrom
jonasschnelli:2015/06/univalue_fix

Conversation

@jonasschnelli

Copy link
Copy Markdown
Contributor

Reported by @sdaftuar (Thanks!).

This fixes a bug that was included in #6121 during the transition from Json Spirit to UniValue.

@sdaftuar

sdaftuar commented Jun 4, 2015

Copy link
Copy Markdown
Member

Verified this fixes getblocktemplate_proposals.py, looks good to me.

@jonasschnelli

Copy link
Copy Markdown
Contributor Author

Here is the code line before the transition (before #6121) to verify the change: https://github.com/bitcoin/bitcoin/blob/0.11/src/rpcmining.cpp#L403

@jgarzik

jgarzik commented Jun 4, 2015

Copy link
Copy Markdown
Contributor

ACK

@fanquake

fanquake commented Jun 5, 2015

Copy link
Copy Markdown
Member

utACK

@laanwj
laanwj merged commit ef5fca8 into bitcoin:master Jun 5, 2015
laanwj added a commit that referenced this pull request Jun 5, 2015
ef5fca8 fix rpcmining/getblocktemplate univalue transition logic error (Jonas Schnelli)
@laanwj

laanwj commented Jun 5, 2015

Copy link
Copy Markdown
Member

Thanks

zkbot added a commit to zcash/zcash that referenced this pull request Feb 5, 2017
…=<try>

Convert entire source tree from json_spirit to UniValue

This PR cherry-picks bitcoin/bitcoin#6121 and then migrates the Zcash-specific code to UniValue.

Also cherry-picks:
- bitcoin/bitcoin#6241
- bitcoin/bitcoin#6234

Closes #1985.
zkbot added a commit to zcash/zcash that referenced this pull request Feb 10, 2017
…=str4d

Convert entire source tree from json_spirit to UniValue

This PR cherry-picks bitcoin/bitcoin#6121 and then migrates the Zcash-specific code to UniValue.

Also cherry-picks:
- bitcoin/bitcoin#6241
- bitcoin/bitcoin#6234

Closes #1985.
@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.

5 participants