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

active_transactions.prioritize_chains timing fix #2004

Merged
merged 2 commits into from May 21, 2019

Conversation

2 participants
@argakiig
Copy link
Collaborator

commented May 21, 2019

fixed should_flush to properly handle dropping if active.size () >=4 inclusively
fix timing and logic in prioritize_chains.

Fix logic in prioritize_chains test
fix should_flush for tests

@argakiig argakiig added the unit test label May 21, 2019

@argakiig argakiig added this to the V19.0 milestone May 21, 2019

@argakiig argakiig requested a review from wezrule May 21, 2019

@argakiig argakiig self-assigned this May 21, 2019

@argakiig argakiig merged commit e89e960 into nanocurrency:master May 21, 2019

0 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details

@zhyatt zhyatt added this to RC 3 (TBD) in V19 May 21, 2019

argakiig added a commit to argakiig/raiblocks that referenced this pull request May 21, 2019

active_transactions.prioritize_chains timing fix (nanocurrency#2004)
* Fix logic in prioritize_chains test
fix should_flush for tests

* Update active_transactions.cpp

argakiig added a commit to argakiig/raiblocks that referenced this pull request May 22, 2019

active_transactions.prioritize_chains timing fix (nanocurrency#2004)
* Fix logic in prioritize_chains test
fix should_flush for tests

* Update active_transactions.cpp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.