aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Add getmempooldescendants RPC callSuhas Daftuar2016-06-092-1/+67
|
* Add getmempoolancestors RPC callSuhas Daftuar2016-05-172-0/+66
|
* Refactor logic for converting mempool entries to JSONSuhas Daftuar2016-05-171-39/+51
|
* Merge #7696: Fix de-serialization bug where AddrMan is left corruptedPieter Wuille2016-05-174-0/+146
|\ | | | | | | 1475ecf Fix de-serialization bug where AddrMan is corrupted after exception * CAddrDB modified so that when de-serialization code throws an exception Addrman is reset to a clean state * CAddrDB modified to make unit tests possible * Regression test created to ensure bug is fixed * StartNode modifed to clear adrman if CAddrDB::Read returns an error code. (EthanHeilman)
| * Fix de-serialization bug where AddrMan is corrupted after exceptionEthanHeilman2016-05-044-0/+146
| | | | | | | | | | | | | | * CAddrDB modified so that when de-serialization code throws an exception Addrman is reset to a clean state * CAddrDB modified to make unit tests possible * Regression test created to ensure bug is fixed * StartNode modifed to clear adrman if CAddrDB::Read returns an error code.
* | Merge #8038: [qa, doc] Various minor fixesMarcoFalke2016-05-174-8/+9
|\ \ | | | | | | | | | | | | | | | | | | fa83a5d [qa] wallet: Temporarily disable salvagewallet test (MarcoFalke) fadd048 [doc] Link to clang-format in the developer notes (MarcoFalke) fa72f7d [doc] Remove outdated line from listunspent RPC help, fix typo (MarcoFalke) ac40ed7 Increase timeout waiting for pruned blk00000.dat (error10)
| * | [qa] wallet: Temporarily disable salvagewallet testMarcoFalke2016-05-101-3/+4
| | |
| * | [doc] Link to clang-format in the developer notesMarcoFalke2016-05-101-1/+3
| | |
| * | [doc] Remove outdated line from listunspent RPC help, fix typoMarcoFalke2016-05-101-3/+1
| | |
| * | Increase timeout waiting for pruned blk00000.daterror102016-05-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In my ever-growing list of test failures, I was seeing this one intermittently. ``` Running 2nd level testscript pruning.py... Initializing test directory /tmp/testY5ypCv Warning! This test requires 4GB of disk space and takes over 30 mins (up to 2 hours) Mining a big blockchain of 995 blocks Check that we haven't started pruning yet because we're below PruneAfterHeight Success Though we're already using more than 550MB, current usage: 587 Mining 25 more blocks should cause the first block file to be pruned Assertion failed: blk00000.dat not pruned when it should be File "/home/error/bitcoinxt-0.11D/qa/rpc-tests/test_framework/test_framework.py", line 118, in main self.run_test() File "/home/error/bitcoinxt-0.11D/qa/rpc-tests/pruning.py", line 272, in run_test self.test_height_min() File "/home/error/bitcoinxt-0.11D/qa/rpc-tests/pruning.py", line 94, in test_height_min raise AssertionError("blk00000.dat not pruned when it should be") Stopping nodes Failed ``` After digging into the test, I found that the code is waiting 10 seconds for blk00000.dat to be deleted, and then throwing this failure if it still exists after 10 seconds. I increased this amount, had the script print the actual time taken, and ran the test a few more times. The time taken ranged between 8 to 12 seconds. So, I feel that this timeout is too short. After changing the timeout to 30 seconds, the test passes consistently. (cherry picked from commit 3469911c89a48dd2fefe4d1c2a0c176256e14ee0)
* | | Merge #7994: Add op csv tests to script_tests.jsonPieter Wuille2016-05-171-1/+10
|\ \ \ | | | | | | | | | | | | 10e83d7 Adding basic tests for OP_CSV inside of script_tests.json (Chris Stewart)
| * | | Adding basic tests for OP_CSV inside of script_tests.jsonChris Stewart2016-05-111-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changing NOP3 op name to OP_CHECKSEQUENCEVERIFY, renaming instances of OP_NOP3 in script_tests.json to CHECKSEQUENCEVERIFY Cleaning up NOP3 comment Re-adding test cases that were accidentally deleted, removing dupicated test case, fixing formatting Removing re-labeling of OP_NOP3 to OP_CSV Fixing whitespace issues
* | | | Merge #8059: Remove unneeded feerate param from ↵Pieter Wuille2016-05-167-25/+17
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | RelayTransaction/AcceptToMemoryPool. d87b198 Remove unneeded feerate param from RelayTransaction/AcceptToMemoryPool. (Gregory Maxwell)
| * | | | Remove unneeded feerate param from RelayTransaction/AcceptToMemoryPool.Gregory Maxwell2016-05-167-25/+17
|/ / / /
* | | | Merge #8046: [Qt][OSX] Fix Cmd-Q / Menu Quit shutdown on OSXJonas Schnelli2016-05-121-0/+1
|\ \ \ \ | | | | | | | | | | | | | | | 34ebceb [Qt][OSX] Fix Cmd-Q / Menu Quit shutdown on OSX (Jonas Schnelli)
| * | | | [Qt][OSX] Fix Cmd-Q / Menu Quit shutdown on OSXJonas Schnelli2016-05-121-0/+1
| | | | |
* | | | | Merge #8006: Qt: Add option to disable the system tray iconWladimir J. van der Laan2016-05-127-2/+65
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | 8b0e497 Qt: Add option to hide the system tray icon (Tyler Hardin)
| * | | | | Qt: Add option to hide the system tray iconTyler Hardin2016-05-117-2/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | My changes leave all tray icon and menu creation/initialization logic untouched. It only shows or hides the icon according to the setting. A new checkbox was added to the OptionsDialog under the Window tab. A bool option named "hideTrayIcon" was added to OptionsModel. This checkbox was mapped like other all options to the OptionsModel. A signal was added to the OptionsModel for broadcasting changes the the hideTrayIcon option. This signal was connected to a new slot added to BitcoinGUI named setTrayIconVisible(bool). The slot simply hides or shows the trayIcon in BitcoinGUI according to the parameter recieved.
* | | | | | Merge #8004: signal handling: fReopenDebugLog and fRequestShutdown should be ↵Wladimir J. van der Laan2016-05-123-3/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | type sig_atomic_t 3262316 fReopenDebugLog and fRequestShutdown should be type sig_atomic_t (Chirag Davé)
| * | | | | | fReopenDebugLog and fRequestShutdown should be type sig_atomic_tChirag Davé2016-05-093-3/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows access as an atomic variable in the presence of async interrupts. See issue #7433 for more details fixes: #7433
* | | | | | | Merge #8041: [qa] Fix bip9-softforks blockstore issueWladimir J. van der Laan2016-05-121-0/+3
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | fad60b3 [qa] Fix bip9-softforks blockstore issue (MarcoFalke)
| * | | | | | [qa] Fix bip9-softforks blockstore issueMarcoFalke2016-05-111-0/+3
| | | | | | |
* | | | | | | Merge #8039: bench: Add crypto hash benchmarksPieter Wuille2016-05-122-1/+55
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | 32114dd bench: Add crypto hash benchmarks (Wladimir J. van der Laan)
| * | | | | | bench: Add crypto hash benchmarksWladimir J. van der Laan2016-05-112-1/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add benchmarks for the cryptographic hash algorithms: - RIPEMD160 - SHA1 - SHA256 - SHA512 Continues work on #7883.
* | | | | | | Merge #7972: [qa] pull-tester: Run rpc test in parallelMarcoFalke2016-05-105-21/+121
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | ccccc59 [qa] Add option --portseed to test_framework (MarcoFalke) fa494de [qa] pull-tester: Run rpc test in parallel (MarcoFalke)
| * | | | | | [qa] Add option --portseed to test_frameworkMarcoFalke2016-05-093-9/+17
| | | | | | |
| * | | | | | [qa] pull-tester: Run rpc test in parallelMarcoFalke2016-05-093-13/+105
| | | | | | |
* | | | | | | Merge #8028: Fix insanity of CWalletDB::WriteTx and CWalletTx::WriteToDiskWladimir J. van der Laan2016-05-104-18/+10
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 0fd5997 Fix insanity of CWalletDB::WriteTx and CWalletTx::WriteToDisk (Patrick Strateman)
| * | | | | | | Fix insanity of CWalletDB::WriteTx and CWalletTx::WriteToDiskPatrick Strateman2016-05-094-18/+10
| | | | | | | |
* | | | | | | | Merge #8036: init: Move berkeleydb version reporting to walletWladimir J. van der Laan2016-05-102-5/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 3e2c946 init: Move berkeleydb version reporting to wallet (Wladimir J. van der Laan)
| * | | | | | | | init: Move berkeleydb version reporting to walletWladimir J. van der Laan2016-05-102-5/+1
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move the version reporting to Wallet::Verify, before starting verification of the wallet. This removes the dependency of init on a specific wallet database library. A further, trivial step towards resolving #7965.
* | | | | | | | Merge #8019: Remove state arg from ReconsiderBlock, rename to ↵Wladimir J. van der Laan2016-05-103-7/+5
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ResetBlockFailureFlags 657e07e Rename ReconsiderBlock func to reflect real behavior (instagibbs) addb9d2 Remove state arg from ReconsiderBlock (instagibbs)
| * | | | | | | | Rename ReconsiderBlock func to reflect real behaviorinstagibbs2016-05-093-3/+3
| | | | | | | | |
| * | | | | | | | Remove state arg from ReconsiderBlockinstagibbs2016-05-063-7/+5
| |/ / / / / / /
* | | | | | | | Merge #8016: Fix multithread CScheduler and reenable testWladimir J. van der Laan2016-05-102-2/+1
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | 166e4b0 Notify other serviceQueue thread we are finished to prevent deadlocks. (Pavel Janík) db18ab2 Reenable multithread scheduler test. (Pavel Janík)
| * | | | | | | Notify other serviceQueue thread we are finished to prevent deadlocks.Pavel Janík2016-05-061-0/+1
| | | | | | | |
| * | | | | | | Reenable multithread scheduler test.Pavel Janík2016-05-061-2/+0
| |/ / / / / /
* | | | | | | Merge #8012: Qt: Delay user confirmation of sendJonas Schnelli2016-05-102-4/+71
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | | | | | | | | 3902a29 Qt: Delay user confirmation of send (Tyler Hardin)
| * | | | | | Qt: Delay user confirmation of sendTyler Hardin2016-05-092-4/+71
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I made a subclass of QMessageBox that disables the send button in exec() and starts a timer that calls a slot to re-enable it after a configurable delay. It also has a countdown in the send/yes button while it is disabled to hint to the user why the send button is disabled (and that it is actually supposed to be disabled).
* | | | | | Merge #7971: [qa] Refactor test_framework and pull testerMarcoFalke2016-05-095-57/+79
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fad3366 [qa] pull-tester: Adjust comment (MarcoFalke) fafb33c [qa] Stop other nodes, even when one fails to stop (MarcoFalke) 2222dae [qa] Update README.md (MarcoFalke) fabbf6b [qa] Refactor test_framework and pull tester (MarcoFalke)
| * | | | | | [qa] pull-tester: Adjust commentMarcoFalke2016-05-081-2/+1
| | | | | | |
| * | | | | | [qa] Stop other nodes, even when one fails to stopMarcoFalke2016-05-061-2/+9
| | | | | | |
| * | | | | | [qa] Update README.mdMarcoFalke2016-05-061-5/+15
| | | | | | |
| * | | | | | [qa] Refactor test_framework and pull testerMarcoFalke2016-05-064-50/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * log to stdout * increase range for p2p and rpc ports * UPPERCASE_CONSTANTS * Stop nodes on CTRL+C
* | | | | | | Merge #8018: Autofind rpc tests --srcdirMarcoFalke2016-05-091-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 5ea4508 Autofind rpc tests --srcdir (Jonas Schnelli)
| * | | | | | | Autofind rpc tests --srcdirJonas Schnelli2016-05-091-1/+1
| |/ / / / / /
* | | | | | | Merge #8030: test: Revert fatal-ness of missing python-zmqWladimir J. van der Laan2016-05-091-2/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 65fee8e test: Revert fatal-ness of missing python-zmq (Wladimir J. van der Laan)
| * | | | | | | test: Revert fatal-ness of missing python-zmqWladimir J. van der Laan2016-05-091-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It looks like travis is using the `travis.yml` from the branch, but runs the test script from the branch merged into master. This causes pull requests created before the QA tests python 3 transition to fail. This temporarily reverts fa05e22e919b7e2e816606f0c0d3dea1bd325bfd (#7851). It can be restored when this is no longer an issue.
* | | | | | | | Merge #7958: Remove useless argument to AlertNotify.Wladimir J. van der Laan2016-05-091-9/+6
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | | | | | | | | | b02119e Remove useless argument to AlertNotify. (Pavel Janík)
| * | | | | | | Remove useless argument to AlertNotify.Pavel Janík2016-04-271-9/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is always 'true', so useless.