-
Notifications
You must be signed in to change notification settings - Fork 92
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
Replaced all BOOST_FOREACH calls with new for() syntax (c++11) #525
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed changes, pending travis
should we add a linter to check there's no regression on this? |
Make sense, but I think we need to finish the |
Resolved merge conflict after merging PR #524, waiting on travis. builds and runs on ubuntu 18.04 |
Rebasing now, will push when done. |
Rebase pushed |
Merged changes from master |
Rebased PR on updated master branch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tested sending, receiving, creating proposals, creating payment requests, staking, syncing on mainnet, on ubuntu 18.10
Fixed merge conflict, will merge when builds for the daemon and QT on windows/osx pass |
No description provided.