Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Process orphans on block acceptance. #345

Merged
merged 1 commit into from
Mar 10, 2015
Merged

Process orphans on block acceptance. #345

merged 1 commit into from
Mar 10, 2015

Conversation

dajohi
Copy link
Member

@dajohi dajohi commented Mar 10, 2015

For every transaction in a newly accepted block, process the orphan
pool moving now no longer orphan transactions to the mempool.

Previously, no longer orphan transactions would remain in the orphan
pool.

// transaction hash (it is possible that they are no longer orphans) and
// potentially accepts them to the memory pool. It repeats the process for the
// newly accepted transactions (to detect further orphans which may no longer be
// orphans) until there are no more.
//
// This function is safe for concurrent access.
func (mp *txMemPool) ProcessOrphans(hash *wire.ShaHash) error {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Define this after the thing is calls, please (processOrphans).

For every transaction in a newly accepted block, process the orphan
pool moving now no longer orphan transactions to the mempool.

Previously, no longer orphan transactions would remain in the orphan
pool.
@davecgh
Copy link
Member

davecgh commented Mar 10, 2015

The concept looks sound to me.

@davecgh
Copy link
Member

davecgh commented Mar 10, 2015

OK

@conformal-deploy conformal-deploy merged commit 54d7951 into btcsuite:master Mar 10, 2015
@dajohi dajohi deleted the process_orphans branch March 10, 2015 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants