Skip to content

Conversation

jgarzik
Copy link
Contributor

@jgarzik jgarzik commented May 14, 2012

Checking TxnCommit() return value is obviously needed. Checking TxnBegin() return value is important too; failing to start a transaction could occur for any number of reasons.

jgarzik pushed a commit that referenced this pull request May 17, 2012
[FIX] Always check return values of TxnBegin() and TxnCommit()
@jgarzik jgarzik merged commit cf2f7c3 into bitcoin:master May 17, 2012
coblee pushed a commit to litecoin-project/litecoin that referenced this pull request Jul 17, 2012
[FIX] Always check return values of TxnBegin() and TxnCommit()
@jgarzik jgarzik deleted the txn-retval branch August 24, 2014 04:18
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request Jan 22, 2019
…ers (bitcoin#1295)

Recuce the nuisance log entries every time a graphene or thinblock timer
is checked.  Only print out the log entry the very first time.
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request May 6, 2020
…Test

9575319 [Tests] Set '-staking' disabled by default on RegTest (random-zebra)

Pull request description:

  Ref: PIVX-Project#1278 (comment)
  Hardcoded default `false` value, instead of having it in the config file for the test nodes.

ACKs for top commit:
  Fuzzbawls:
    ACK 9575319
  furszy:
    ACK 9575319

Tree-SHA512: 609f4eee57ab9e32a03a88cf014c407348d89f89bbaf6f56187a1bf20e458b7c83d76220f8c047427cd7319cf8ad4b6ff71a4aca06c51b71c6a3ab56efa7cf88
@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.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant