aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Add logging to bitcoin-util-test.pyjnewbery2016-11-022-24/+54
| | | | | | - Use the python standard logging library - Run all tests and report all failing test-cases (rather than stop after one test case fails) - If output is different from expected output, log a contextual diff.
* Merge #8828: Move CWalletDB::ReorderTransactions to CWalletWladimir J. van der Laan2016-11-023-80/+74
|\ | | | | | | 86029e7 Move CWalletDB::ReorderTransactions to CWallet (Patrick Strateman)
| * Move CWalletDB::ReorderTransactions to CWalletPatrick Strateman2016-10-303-80/+74
| |
* | Merge #8674: tools for analyzing, updating and adding copyright headers in ↵Wladimir J. van der Laan2016-11-023-76/+662
|\ \ | | | | | | | | | | | | | | | source files 159597a [devtools] script support for managing source file copyright headers (isle2983)
| * | [devtools] script support for managing source file copyright headersisle29832016-09-103-76/+662
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Three subcommands to this script: 1) ./copyright_header.py report Examines git-tracked files with extensions that match: INCLUDE = ['*.h', '*.cpp', '*.cc', '*.c', '*.py'] Helps to: -> Identify source files without copyright -> Identify source files added with something other than "The Bitcoin Core developers" holder so we can be sure it is appropriate -> Identify unintentional typos in the copyright line 2) ./copyright_header.py update Replaces fix-copyright-headers.py. It does file editing in native python rather than subprocessing out to perl as was the case with fix-copyright-headers.py. It also shares code with the 'report' functions. 3) ./copyright_header.py insert Inserts a copyright header into a source file with the proper format and dates.
* | | Merge #9060: trivial: fix bloom filter init to isEmpty = trueWladimir J. van der Laan2016-11-021-1/+1
|\ \ \ | | | | | | | | | | | | cccf73d trivial: fix bloom filter init to isEmpty = true (Robert McLaughlin)
| * | | trivial: fix bloom filter init to isEmpty = trueRobert McLaughlin2016-11-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes newly initialized bloom filters being constructed with isEmpty(false), which still works but loses the possible speedup when checking for key membership in an empty filter.
* | | | Merge #9063: Do not shadow variable, use deprecated MAP_ANON if ↵Wladimir J. van der Laan2016-11-021-3/+10
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | MAP_ANONYMOUS is not defined 4a9f3c5 Do not shadow variable, use deprecated MAP_ANON if MAP_ANONYMOUS is not defined. (Pavel Janík)
| * | | | Do not shadow variable, use deprecated MAP_ANON if MAP_ANONYMOUS is not defined.Pavel Janík2016-11-021-3/+10
|/ / / /
* | | | Merge #8753: Locked memory managerWladimir J. van der Laan2016-11-0215-410/+966
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 444c673 bench: Add benchmark for lockedpool allocation/deallocation (Wladimir J. van der Laan) 6567999 rpc: Add `getmemoryinfo` call (Wladimir J. van der Laan) 4536148 support: Add LockedPool (Wladimir J. van der Laan) f4d1fc2 wallet: Get rid of LockObject and UnlockObject calls in key.h (Wladimir J. van der Laan) 999e4c9 wallet: Change CCrypter to use vectors with secure allocator (Wladimir J. van der Laan)
| * | | | bench: Add benchmark for lockedpool allocation/deallocationWladimir J. van der Laan2016-10-272-1/+49
| | | | |
| * | | | rpc: Add `getmemoryinfo` callWladimir J. van der Laan2016-10-271-0/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ``` getmemoryinfo Returns an object containing information about memory usage. Result: { "locked": { (json object) Information about locked memory manager "used": xxxxx, (numeric) Number of bytes used "free": xxxxx, (numeric) Number of bytes available in current arenas "total": xxxxxxx, (numeric) Total number of bytes managed "locked": xxxxxx, (numeric) Amount of bytes that succeeded locking. If this number is smaller than total, locking pages failed at some point and key data could be swapped to disk. } } Examples: > bitcoin-cli getmemoryinfo > curl --user myusername --data-binary '{"jsonrpc": "1.0", "id":"curltest", "method": "getmemoryinfo", "params": [] }' -H 'content-type: text/plain;' http://127.0.0.1:8332/ ```
| * | | | support: Add LockedPoolWladimir J. van der Laan2016-10-277-328/+832
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a pool for locked memory chunks, replacing LockedPageManager. This is something I've been wanting to do for a long time. The current approach of locking objects where they happen to be on the stack or heap in-place causes a lot of mlock/munlock system call overhead, slowing down any handling of keys. Also locked memory is a limited resource on many operating systems (and using a lot of it bogs down the system), so the previous approach of locking every page that may contain any key information (but also other information) is wasteful.
| * | | | wallet: Get rid of LockObject and UnlockObject calls in key.hWladimir J. van der Laan2016-10-193-55/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace these with vectors allocated from the secure allocator. This avoids mlock syscall churn on stack pages, as well as makes it possible to get rid of these functions. Please review this commit and the previous one carefully that no `sizeof(vectortype)` remains in the memcpys and memcmps usage (ick!), and `.data()` or `&vec[x]` is used as appropriate instead of &vec.
| * | | | wallet: Change CCrypter to use vectors with secure allocatorWladimir J. van der Laan2016-10-193-26/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change CCrypter to use vectors with secure allocator instead of buffers on in the object itself which will end up on the stack. This avoids having to call LockedPageManager to lock stack memory pages to prevent the memory from being swapped to disk. This is wasteful.
* | | | | Merge #9032: test: Add format-dependent comparison to bctestWladimir J. van der Laan2016-11-021-3/+27
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | 6c5cd9d test: Add format-dependent comparison to bctest (Wladimir J. van der Laan)
| * | | | | test: Add format-dependent comparison to bctestWladimir J. van der Laan2016-10-281-3/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This splits the output comparison for `bitcoin-tx` into two steps: - First, check for data mismatch, parsing the data as json or hex depending on the extension of the output file - Then, check if the literal string matches For either of these cases give a different error. This prevents wild goose chases when e.g. a trailing space doesn't match exactly, and makes sure that both test output and examples are valid data of the purported format.
* | | | | | Merge #8448: Store mempool and prioritization data to diskWladimir J. van der Laan2016-11-026-7/+147
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 582068a Add mempool.dat to doc/files.md (Pieter Wuille) 3f78562 Add DumpMempool and LoadMempool (Pieter Wuille) ced7c94 Add AcceptToMemoryPoolWithTime function (Pieter Wuille) c3efb58 Add feedelta to TxMempoolInfo (Pieter Wuille)
| * | | | | Add mempool.dat to doc/files.mdPieter Wuille2016-10-311-0/+1
| | | | | |
| * | | | | Add DumpMempool and LoadMempoolPieter Wuille2016-10-313-0/+122
| | | | | |
| * | | | | Add AcceptToMemoryPoolWithTime functionPieter Wuille2016-10-302-5/+15
| | | | | |
| * | | | | Add feedelta to TxMempoolInfoPieter Wuille2016-10-302-2/+9
| | |_|_|/ | |/| | |
* | | | | Merge #9043: [qt] Return useful error message on ATMP failureJonas Schnelli2016-11-013-4/+8
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | 3333e5a [qt] Return useful error message on ATMP failure (MarcoFalke)
| * | | | | [qt] Return useful error message on ATMP failureMarcoFalke2016-10-303-4/+8
| |/ / / /
* | | | | Merge #9041: keypoololdest denote Unix epoch, not GMTMarcoFalke2016-10-312-2/+2
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | 7f61b49 Change all instance of 'GMT epoch' to 'Unix epoch' (matthias)
| * | | | Change all instance of 'GMT epoch' to 'Unix epoch'matthias2016-10-312-2/+2
|/ / / /
* | | | Merge #8989: [Qt] overhaul smart-fee slider, adjust default confirmation targetWladimir J. van der Laan2016-10-287-13/+64
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | cfe77ef [Qt] overhaul smart-fee slider, adjust default confirmation target (Jonas Schnelli) 6f02899 [Qt] Hide nTxConfirmTarget behind WalletModel (Jonas Schnelli) 004168d CoinControl: add option for custom confirmation target (Jonas Schnelli)
| * | | | [Qt] overhaul smart-fee slider, adjust default confirmation targetJonas Schnelli2016-10-283-11/+47
| | | | |
| * | | | [Qt] Hide nTxConfirmTarget behind WalletModelJonas Schnelli2016-10-282-0/+7
| | | | |
| * | | | CoinControl: add option for custom confirmation targetJonas Schnelli2016-10-282-2/+10
| | | | |
* | | | | Merge #9016: Return useful error message on ATMP failureWladimir J. van der Laan2016-10-284-13/+22
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | 169bdab Return useful error message on ATMP failure (instagibbs)
| * | | | Return useful error message on ATMP failureinstagibbs2016-10-264-13/+22
| | | | |
* | | | | Merge #9029: instance of 'mem pool' to 'mempool'Wladimir J. van der Laan2016-10-272-2/+2
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | 1c3ecc7 instance of 'mem pool' to 'mempool' (S. Matthew English)
| * | | | | instance of 'mem pool' to 'mempool'S. Matthew English2016-10-272-2/+2
| | |/ / / | |/| | | | | | | | | | | | | there was only one instance of 'mem pool' and not 'mempool', so I changed it to conform to the others
* | | | | Merge #9033: Update build notes for dropping osx 10.7 support (fanquake)Wladimir J. van der Laan2016-10-271-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | ba26d41 Update build notes for dropping osx 10.7 support (Michael Ford)
| * | | | | Update build notes for dropping osx 10.7 supportMichael Ford2016-10-271-1/+1
| |/ / / / | | | | | | | | | | | | | | | | | | | | Github-Pull: #9022 Rebased-From: 1d1246314f761b3f84c869cb7b0af49cc25b55b3
* / / / / doc: Add release notes for 0.13.1 releaseWladimir J. van der Laan2016-10-271-0/+410
|/ / / /
* | | | Merge #9020: rpc: Remove invalid explanation from wallet fee messageWladimir J. van der Laan2016-10-261-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | 04c1c15 rpc: Remove invalid explanation from wallet fee message (Wladimir J. van der Laan)
| * | | | rpc: Remove invalid explanation from wallet fee messageWladimir J. van der Laan2016-10-261-1/+1
|/ / / /
* | | | Merge #9015: release: bump required osx version to 10.8. (jonasschnelli)Wladimir J. van der Laan2016-10-261-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | 339c4b6 release: bump required osx version to 10.8. Credit jonasschnelli. (Cory Fields)
| * | | | release: bump required osx version to 10.8. Credit jonasschnelli.Cory Fields2016-10-251-1/+1
|/ / / / | | | | | | | | | | | | | | | | | | | | libc++ on 10.7 causes too many issues. See #8577 for discussion/details.
* | | | Merge #8515: A few mempool removal optimizationsWladimir J. van der Laan2016-10-257-50/+51
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0334430 Add some missing includes (Pieter Wuille) 4100499 Return shared_ptr<CTransaction> from mempool removes (Pieter Wuille) 51f2783 Make removed and conflicted arguments optional to remove (Pieter Wuille) f48211b Bypass removeRecursive in removeForReorg (Pieter Wuille)
| * | | | Add some missing includesPieter Wuille2016-10-211-0/+4
| | | | |
| * | | | Return shared_ptr<CTransaction> from mempool removesPieter Wuille2016-10-215-15/+15
| | | | |
| * | | | Make removed and conflicted arguments optional to removePieter Wuille2016-10-217-35/+31
| | | | |
| * | | | Bypass removeRecursive in removeForReorgPieter Wuille2016-10-211-7/+8
| |/ / /
* | | | Merge #8948: [TRIVIAL] reorder Windows gitian build order to match LinuxWladimir J. van der Laan2016-10-251-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | 3f7581d [TRIVIAL] reorder Windows gitian build order to match Linux (Micha)
| * | | | [TRIVIAL] reorder Windows gitian build order to match LinuxMicha2016-10-181-1/+1
| | | | | | | | | | | | | | | | | | | | The consistency is helpful for gauging Gitian build progress. Right now it's necessary to remember which platform builds in which order, which can be confusing if you're attempting to get a quick idea of how far along your builds are.
* | | | | Merge #9008: [net] Remove assert(nMaxInbound > 0)Wladimir J. van der Laan2016-10-251-1/+0
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | fa1c3c2 [net] Remove assert(nMaxInbound > 0) (MarcoFalke)
| * | | | | [net] Remove assert(nMaxInbound > 0)MarcoFalke2016-10-241-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nMaxInbound might very well be 0 or -1, if the user prefers to keep a small number of maxconnections. Note: nMaxInbound of -1 means that the user set maxconnections to 8 or less, but we still want to keep an additional slot for the feeler connection.