This resulted in just passing all transactions to filtered wallets which worked surprisingly well, except where it didn't.
Update the bloom state on the real object, not the temporary one. #2919
pull gmaxwell wants to merge 1 commits into bitcoin:master from gmaxwell:bloom_faster changing 1 files +1 −1-
gmaxwell commented at 12:43 AM on August 21, 2013: contributor
-
a7f533a972
Update the bloom state on the real object, not the temporary one.
This resulted in just passing all transactions to filtered wallets which worked surprisingly well, except where it didn't.
-
gavinandresen commented at 1:14 AM on August 21, 2013: contributor
Looks good to me-- compiles and runs on my OSX machine.
-
petertodd commented at 1:39 AM on August 21, 2013: contributor
ACK
-
gavinandresen commented at 1:56 AM on August 21, 2013: contributor
ACK: tested by importing a private key into a MultiBit connected to bitcoind running this patch, it sync'ed correctly and ended up with the correct balance.
-
BitcoinPullTester commented at 3:07 AM on August 21, 2013: none
Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/a7f533a9723c7eafea4aea7d00fa9129a4ffe1d5 for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
- gavinandresen referenced this in commit b986663ccd on Aug 21, 2013
- gavinandresen merged this on Aug 21, 2013
- gavinandresen closed this on Aug 21, 2013
- Bushstar referenced this in commit 5652ea0237 on Apr 8, 2020
- Bushstar referenced this in commit a198a04e04 on Apr 8, 2020
- DrahtBot locked this on Sep 8, 2021
Contributors