Skip to content

Conversation

sdaftuar
Copy link
Member

@sdaftuar sdaftuar commented Apr 1, 2015

This adds more tests to CheckBlockIndex:

  • HAVE_DATA is true iff nTx > 0
  • BLOCK_VALID_TRANSACTIONS is true iff nTx > 0
  • BLOCK_VALID_TRANSACTIONS is true for a block and all parents iff
    nChainTx > 0

This adds more tests to CheckBlockIndex:
- HAVE_DATA is true iff nTx > 0
- BLOCK_VALID_TRANSACTIONS is true iff nTx > 0
- BLOCK_VALID_TRANSACTIONS is true for a block and all parents iff
  nChainTx > 0
@sipa
Copy link
Member

sipa commented Apr 1, 2015

utACK

@laanwj
Copy link
Member

laanwj commented Apr 8, 2015

utACK

@laanwj laanwj merged commit ede379f into bitcoin:master Apr 8, 2015
laanwj added a commit that referenced this pull request Apr 8, 2015
ede379f Add additional block index consistency checks (Suhas Daftuar)
@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.

3 participants