Remove LOCK(cs_main) from decodescript #7013

pull petertodd wants to merge 1 commits into bitcoin:master from petertodd:2015-11-remove-cs-main-from-decodescript changing 1 files +0 −1

petertodd commented at 10:20 PM on November 13, 2015: contributor

Completely static RPC call that doesn't change or even look at mutable state anywhere.

Fixes issue #7012

Remove LOCK(cs_main) from decodescript

Completely static RPC call that doesn't change or even look at mutable
state anywhere.

b3ae384a8d

petertodd cross-referenced this on Nov 13, 2015 from issue RPC: decodescript RPC call holds cs_main by pstratem
gmaxwell commented at 5:23 AM on November 14, 2015: contributor

utACK.
dcousens commented at 5:28 AM on November 14, 2015: contributor

utACK
laanwj commented at 7:15 AM on November 14, 2015: member

Yes, makes sense, woohoo, a RPC call that can support parallelism. utACK (this, incidentally, was the longer-term goal of pushing down the RPC locks in #5711)
laanwj added the label RPC on Nov 14, 2015
laanwj merged this on Nov 14, 2015
laanwj closed this on Nov 14, 2015
laanwj referenced this in commit 36baa9f475 on Nov 14, 2015
luke-jr referenced this in commit e411c416d3 on Nov 18, 2015
laanwj cross-referenced this on Dec 2, 2015 from issue rpc: remove cs_main lock from `createrawtransaction` by laanwj
luke-jr referenced this in commit 28262da0ea on Dec 8, 2015
dagurval cross-referenced this on Jan 15, 2018 from issue Misc createrawtransaction changes by dagurval
LarryRuane referenced this in commit bc3da48a11 on Feb 18, 2021
LarryRuane cross-referenced this on Feb 18, 2021 from issue Bitcoin 0.12 locking PRs by LarryRuane
zkbot referenced this in commit b08c673ddd on Feb 19, 2021
bitcoin locked this on Sep 8, 2021

Contributors

Labels