2019-07-07

< bitcoin-git> [bitcoin] fanquake opened pull request #16351: 0.18 backport: backport github-merge.py improvements (0.18...backport_github_merge_improvements) https://github.com/bitcoin/bitcoin/pull/16351
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to 0.18: https://github.com/bitcoin/bitcoin/compare/629c7b029cf9...1fb747a8007c
< bitcoin-git> bitcoin/0.18 1dc357d Anthony Towns: signrawtransactionwithkey: report error when missing redeemScript/witnessS...
< bitcoin-git> bitcoin/0.18 1fb747a fanquake: Merge #16342: 0.18 backport: report error when missing redeemScript/witnes...
< bitcoin-git> [bitcoin] fanquake merged pull request #16342: 0.18 backport: report error when missing redeemScript/witnessScript (0.18...201906-signrawerror-regression-018) https://github.com/bitcoin/bitcoin/pull/16342
< bitcoin-git> [bitcoin] hebasto opened pull request #16350: qt: Remove unused guard (master...20190707-remove-unused-guard) https://github.com/bitcoin/bitcoin/pull/16350
< bitcoin-git> [bitcoin] fanquake merged pull request #16344: build: use #if HAVE_SYSTEM instead of defined(HAVE_SYSTEM) (master...2019/07/system-ifdef) https://github.com/bitcoin/bitcoin/pull/16344
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/584168c7f9d9...f373beebbcc0
< bitcoin-git> bitcoin/master 976b034 Sjors Provoost: [build]: use #if HAVE_SYSTEM instead of defined(HAVE_SYSTEM)
< bitcoin-git> bitcoin/master f373bee fanquake: Merge #16344: build: use #if HAVE_SYSTEM instead of defined(HAVE_SYSTEM)
< bitcoin-git> [bitcoin] fanquake merged pull request #16326: [RPC] add new utxoupdatepsbt arguments to the CRPCCommand and CPRCCvertParam tables (master...2019-07-utxoupdatepsbt-args) https://github.com/bitcoin/bitcoin/pull/16326
< bitcoin-git> bitcoin/master 3b11420 John Newbery: [RPC] add new utxoupdatepsbt arguments to the CRPCCommand and CPRCConvertP...
< bitcoin-git> bitcoin/master 91cc18f John Newbery: [docs] Add release notes for PR 15427
< bitcoin-git> bitcoin/master 584168c fanquake: Merge #16326: [RPC] add new utxoupdatepsbt arguments to the CRPCCommand an...
< bitcoin-git> [bitcoin] fanquake pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/8c69fae94410...584168c7f9d9

2019-07-06

< bitcoin-git> [bitcoin] hebasto opened pull request #16349: Fix "opening a wallet" issue #15453 (master...20190706-fix15453) https://github.com/bitcoin/bitcoin/pull/16349
< bitcoin-git> [bitcoin] promag opened pull request #16348: qt: Assert QMetaObject::invokeMethod result (master...2019-07-assert-invoke-method) https://github.com/bitcoin/bitcoin/pull/16348
< bitcoin-git> [bitcoin] dongcarl opened pull request #16347: doc: Include static members in Doxygen (master...2019-07-doxygen-extract-static) https://github.com/bitcoin/bitcoin/pull/16347
< gribble> https://github.com/bitcoin/bitcoin/issues/16341 | WIP: Introduce ScriptPubKeyMan interface and use it for key and script management (aka wallet boxes) by achow101 · Pull Request #16341 · bitcoin/bitcoin · GitHub
< hebasto> promag: hi, could you look into https://travis-ci.org/bitcoin/bitcoin/jobs/554836240 ?
< achow101> anyone familiar with MSVC know what's up with this error: https://ci.appveyor.com/project/DrahtBot/bitcoin/builds/25786259?

2019-07-05

< bitcoin-git> [bitcoin] sipsorcery closed pull request #15995: systemd service script: set usable permissions on /etc/bitcoin config dir (master...systemdconfig) https://github.com/bitcoin/bitcoin/pull/15995
< sipa> there's a website which is still actively being changed: http://bitcoin.sipa.be/miniscript
< sipa> this is about the bitcoin scirpt language as it exists today, without any softforks or other changes to consensus needed (but obviously prepared for those)
< sipa> so me and some colleagues of mine have been working on a way to make more complex bitcoin scripts more accessible, in a project called miniscript
< meshcollider> #bitcoin-core-dev Wallet Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier jl2012 achow101 meshcollider jnewbery maaku fanquake promag provoostenator aj Chris_Stewart_5 dongcarl gwillen jamesob ken281221 ryanofsky gleb moneyball
< gribble> https://github.com/bitcoin/bitcoin/issues/16341 | Introduce ScriptPubKeyMan interface and use it for key and script management (aka wallet boxes) by achow101 · Pull Request #16341 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/15457 | Check std::system for -[alert|block|wallet]notify by Sjors · Pull Request #15457 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] emilengler opened pull request #16345: RPC: Add getblockbyheight (master...getblockbyheight) https://github.com/bitcoin/bitcoin/pull/16345
< provoostenator> emilengler: it does that here: https://github.com/bitcoin/bitcoin/blob/master/src/rpc/client.cpp#L31
< sipa> bitcoin-cli knows that the first argument to generatetoaddress is to be interpreted as json instead of as a string
< sipa> emilengler: bitcoin-cli knows which arguments to treat as strings and which as json
< bitcoin-git> [bitcoin] Sjors opened pull request #16344: [build]: use #if HAVE_SYSTEM instead of defined(HAVE_SYSTEM) (master...2019/07/system-ifdef) https://github.com/bitcoin/bitcoin/pull/16344
< emilengler> bitcoin-cli says "JSON value is not an integer as expected"
< emilengler> I currently have problems with bitcoin-cli to read an integer value on a new command I've implemented
< bitcoin-git> [bitcoin] laanwj merged pull request #15457: Check std::system for -[alert|block|wallet]notify (master...2019/02/std__system) https://github.com/bitcoin/bitcoin/pull/15457
< bitcoin-git> [bitcoin] laanwj pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/4f378ac30cf6...8c69fae94410
< bitcoin-git> bitcoin/master c1c91bb Sjors Provoost: [build] detect std::system or ::wsystem
< bitcoin-git> bitcoin/master cc3ad56 Sjors Provoost: [build] MSVC: set HAVE_SYSTEM for desktop apps
< bitcoin-git> bitcoin/master f874e14 Sjors Provoost: [build]: check std::system for -[alert|block|wallet]notify
< gribble> https://github.com/bitcoin/bitcoin/issues/15457 | Check std::system for -[alert|block|wallet]notify by Sjors · Pull Request #15457 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] Sjors closed pull request #15424: [wallet] wallet-tool: command to remove key metadata (master...2019/02/wallet_tool_remove_metadata) https://github.com/bitcoin/bitcoin/pull/15424
< bitcoin-git> [bitcoin] ajtowns opened pull request #16342: Backport #16250 for 0.18 (0.18...201906-signrawerror-regression-018) https://github.com/bitcoin/bitcoin/pull/16342
< fanquake> I've had a look over the build system changes in that PR, https://github.com/bitcoin/bitcoin/pull/13989#issuecomment-508623934, but don't have a machine that supports AVX512 to test any performance improvements on.
< bitcoin-git> [bitcoin] achow101 opened pull request #16341: Introduce ScriptPubKeyMan interface and use it for key and script management (aka wallet boxes) (master...box-the-wallet) https://github.com/bitcoin/bitcoin/pull/16341
< bitcoin-git> [bitcoin] fanquake merged pull request #16308: [MSVC] Copy build output to src/ automatically after build (master...build/copy-in-msbuild) https://github.com/bitcoin/bitcoin/pull/16308
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/1088b90cbae3...4f378ac30cf6
< bitcoin-git> bitcoin/master 08c721d nicolas.dorier: [MSVC] Copy build output to src/ automatically after build
< bitcoin-git> bitcoin/master 4f378ac fanquake: Merge #16308: [MSVC] Copy build output to src/ automatically after build
< bitcoin-git> bitcoin/master 1088b90 fanquake: Merge #16327: scripts and tools: Update ShellCheck linter
< bitcoin-git> [bitcoin] fanquake merged pull request #16327: scripts and tools: Update ShellCheck linter (master...20190702-shellcheck) https://github.com/bitcoin/bitcoin/pull/16327
< bitcoin-git> [bitcoin] fanquake opened pull request #16339: doc: add reduce-memory.md (master...doc-reduce-memory-usage) https://github.com/bitcoin/bitcoin/pull/16339
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/dfdcb3dfe535...1088b90cbae3
< bitcoin-git> bitcoin/master 1ac454a Hennadii Stepanov: Enable ShellCheck rules

2019-07-04

< bitcoin-git> [bitcoin] qmma70 opened pull request #16338: test: Disable other targets when enable-fuzz is set (master...enable-fuzz) https://github.com/bitcoin/bitcoin/pull/16338
< gribble> https://github.com/bitcoin/bitcoin/issues/15759 | [p2p] Add 2 outbound blocks-only connections by sdaftuar · Pull Request #15759 · bitcoin/bitcoin · GitHub
< sipa> sorry i haven't found much time for review; i've been spending a lot of time on improving and implementing miniscript, which i hope will at some point be integratable in bitcoin core
< wumpus> https://github.com/bitcoin/bitcoin/projects/8 5 blockers, still 7 chasing concept ACK
< wumpus> #bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier jl2012 achow101 meshcollider jnewbery maaku fanquake promag provoostenator aj Chris_Stewart_5 dongcarl gwillen jamesob ken281221 ryanofsky gleb moneyball kvaciral
< bitcoin-git> [bitcoin] emilengler closed pull request #16317: RPC: Add feature to getblock by using the index (master...getblock-improvement) https://github.com/bitcoin/bitcoin/pull/16317
< bitcoin-git> [bitcoin] fanquake closed pull request #16336: [lib] Refactor bloom filter class data member names (master...bloom_refactor) https://github.com/bitcoin/bitcoin/pull/16336
< bitcoin-git> [bitcoin] fanquake closed pull request #16077: docs: Follow ISO/IEC 14882 terms and definitions (master...20190523-iso-parameter) https://github.com/bitcoin/bitcoin/pull/16077
< bitcoin-git> [bitcoin] fanquake merged pull request #16330: docs: Use placeholder instead of key expiration date (master...20190702-key-expiration) https://github.com/bitcoin/bitcoin/pull/16330
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/1381ddbcfcb6...dfdcb3dfe535
< bitcoin-git> bitcoin/master 88fd556 Hennadii Stepanov: Use placeholder instead of key expiration date
< bitcoin-git> bitcoin/master dfdcb3d fanquake: Merge #16330: docs: Use placeholder instead of key expiration date
< bitcoin-git> [bitcoin] fanquake closed pull request #16069: test: move-only: Split large tests into smaller compile units (master...1905-testCompFast) https://github.com/bitcoin/bitcoin/pull/16069
< bitcoin-git> [bitcoin] fanquake merged pull request #16035: 0.18.1: Backports (0.18...1906-181b) https://github.com/bitcoin/bitcoin/pull/16035
< bitcoin-git> [bitcoin] fanquake pushed 37 commits to 0.18: https://github.com/bitcoin/bitcoin/compare/9c29bc71dc40...629c7b029cf9
< bitcoin-git> bitcoin/0.18 5935f01 MarcoFalke: build with -fstack-reuse=none
< bitcoin-git> bitcoin/0.18 9c1a607 MarcoFalke: net: Rename ::fRelayTxes to ::g_relay_txes
< bitcoin-git> bitcoin/0.18 8f215c7 MarcoFalke: test: Format predicate source as multiline on error
< bitcoin-git> [bitcoin] qmma70 opened pull request #16336: [lib] Refactor bloom filter class data member names (master...bloom_refactor) https://github.com/bitcoin/bitcoin/pull/16336

2019-07-03

< bitcoin-git> [bitcoin] dongcarl opened pull request #16334: test: rpc_users: Also test rpcauth.py with password. (master...2019-07-rpcauth-passwd-specified-case) https://github.com/bitcoin/bitcoin/pull/16334
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #16329: test: Add tests for getblockchaininfo.softforks (master...1907-qaSoftforksHeight) https://github.com/bitcoin/bitcoin/pull/16329
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/91c345eb9282...1381ddbcfcb6
< bitcoin-git> bitcoin/master faf6caf MarcoFalke: test: Add tests for getblockchaininfo.softforks
< bitcoin-git> bitcoin/master 1381ddb MarcoFalke: Merge #16329: test: Add tests for getblockchaininfo.softforks
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #16333: test: Set BIP34Height = 2 for regtest (master...1906-bip34H2) https://github.com/bitcoin/bitcoin/pull/16333
< bitcoin-git> [bitcoin] MarcoFalke reopened pull request #16333: test: Set BIP34Height = 2 for regtest (master...1906-bip34H2) https://github.com/bitcoin/bitcoin/pull/16333
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #16299: bench: Move generated data to a dedicated translation unit (master...2019-06-benchmark-data) https://github.com/bitcoin/bitcoin/pull/16299
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/11de669d8b7a...91c345eb9282
< bitcoin-git> bitcoin/master 3d60a03 João Barbosa: bench: Move generated data to a dedicated translation unit
< bitcoin-git> bitcoin/master 91c345e MarcoFalke: Merge #16299: bench: Move generated data to a dedicated translation unit
< aj> MarcoFalke: i've done a backport of #16250 for 0.18 on top of #16035 at https://github.com/ajtowns/bitcoin/tree/201906-signrawerror-regression-018 fwiw
< gribble> https://github.com/bitcoin/bitcoin/issues/16250 | signrawtransactionwithkey: report error when missing redeemScript/witnessScript by ajtowns · Pull Request #16250 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/16035 | 0.18.1: Backports by MarcoFalke · Pull Request #16035 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/16270 | depends: expat 2.2.7 by fanquake · Pull Request #16270 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #16333: test: Set BIP34Height = 2 for regtest (master...1906-bip34H2) https://github.com/bitcoin/bitcoin/pull/16333
< bitcoin-git> [bitcoin] instagibbs opened pull request #16332: Add logpath description for getrpcinfo (master...getrpcinfo_logpath_help) https://github.com/bitcoin/bitcoin/pull/16332
< bitcoin-git> [bitcoin] laanwj merged pull request #16325: rpc: Clarify that block count means height excl genesis (master...1907-rpcBlockCount) https://github.com/bitcoin/bitcoin/pull/16325
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/9339008a9dce...11de669d8b7a
< bitcoin-git> bitcoin/master fab0c82 MarcoFalke: rpc: Clarify that block count means height excl genesis
< bitcoin-git> bitcoin/master 11de669 Wladimir J. van der Laan: Merge #16325: rpc: Clarify that block count means height excl genesis
< bitcoin-git> [bitcoin] laanwj merged pull request #15483: rpc: Adding a 'logpath' entry to getrpcinfo (master...getrpcinfo_logpath) https://github.com/bitcoin/bitcoin/pull/15483
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/085cac6b9043...9339008a9dce
< bitcoin-git> bitcoin/master 8a6810d darosior: Add a 'logpath' field to getrpcinfo
< bitcoin-git> bitcoin/master 9339008 Wladimir J. van der Laan: Merge #15483: rpc: Adding a 'logpath' entry to getrpcinfo
< bitcoin-git> [bitcoin] laanwj merged pull request #14734: fix an undefined behavior in uint::SetHex (master...SetHex-bad-ptr) https://github.com/bitcoin/bitcoin/pull/14734
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/38fbb575e252...085cac6b9043
< bitcoin-git> bitcoin/master 0f459d8 Kaz Wesley: fix an undefined behavior in uint::SetHex
< bitcoin-git> bitcoin/master 085cac6 Wladimir J. van der Laan: Merge #14734: fix an undefined behavior in uint::SetHex
< bitcoin-git> [bitcoin] NicolasDorier closed pull request #16319: [MSVC] Simplify AppVeyor build by using user customized project file (master...msvc/appveyor-refactor) https://github.com/bitcoin/bitcoin/pull/16319
< bitcoin-git> [bitcoin] laanwj merged pull request #16294: qt: test: Create at most one testing setup (master...1906-qtTestOnlyOneTestingSetup) https://github.com/bitcoin/bitcoin/pull/16294
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/7f985d6c8159...38fbb575e252
< bitcoin-git> bitcoin/master faa1e0f MarcoFalke: qt: test: Create at most one testing setup
< bitcoin-git> bitcoin/master 38fbb57 Wladimir J. van der Laan: Merge #16294: qt: test: Create at most one testing setup
< bitcoin-git> [bitcoin] practicalswift opened pull request #16331: build: Add configure option --enable-lowmem to allow for building in low-memory environments (master...low-mem) https://github.com/bitcoin/bitcoin/pull/16331
< bitcoin-git> [bitcoin] laanwj merged pull request #16158: Fix logic of memory_cleanse() on MSVC and clean up docs (master...cleanup-cleanse) https://github.com/bitcoin/bitcoin/pull/16158
< bitcoin-git> [bitcoin] laanwj pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/7d7b832d6707...7f985d6c8159
< bitcoin-git> bitcoin/master cac30a4 Tim Ruffing: Clean up logic in memory_cleanse() for MSVC
< bitcoin-git> bitcoin/master f53a70c Tim Ruffing: Improve documentation of memory_cleanse()
< bitcoin-git> bitcoin/master 7f985d6 Wladimir J. van der Laan: Merge #16158: Fix logic of memory_cleanse() on MSVC and clean up docs
< bitcoin-git> [bitcoin] laanwj merged pull request #16262: rpc: Allow shutdown while in generateblocks (master...2019-06-21-generateblocks) https://github.com/bitcoin/bitcoin/pull/16262
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/e06067387e75...7d7b832d6707
< bitcoin-git> bitcoin/master 3b9bf0e Patrick Strateman: rpc: Allow shutdown while in generateblocks
< bitcoin-git> bitcoin/master 7d7b832 Wladimir J. van der Laan: Merge #16262: rpc: Allow shutdown while in generateblocks
< bitcoin-git> [bitcoin] laanwj closed pull request #14633: Fix height serialization inside script of coinbase input (master...fix_height_serialization_in_coinbase) https://github.com/bitcoin/bitcoin/pull/14633
< bitcoin-git> [bitcoin] fanquake closed pull request #14137: gui: Add Windows taskbar progress (master...qt-windows-progress-bar) https://github.com/bitcoin/bitcoin/pull/14137

2019-07-02

< bitcoin-git> [bitcoin] MarcoFalke merged pull request #16250: signrawtransactionwithkey: report error when missing redeemScript/witnessScript (master...201906-signrawerror-regression) https://github.com/bitcoin/bitcoin/pull/16250
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/4db2f8cf0f63...e06067387e75
< bitcoin-git> bitcoin/master 0117459 Anthony Towns: signrawtransactionwithkey: report error when missing redeemScript/witnessS...
< bitcoin-git> bitcoin/master e060673 MarcoFalke: Merge #16250: signrawtransactionwithkey: report error when missing redeemS...
< bitcoin-git> [bitcoin] hebasto opened pull request #16330: docs: Update release signing key expiration date (master...20190702-key-expiration) https://github.com/bitcoin/bitcoin/pull/16330
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #16329: test: Add tests for getblockchaininfo.softforks (master...1907-qaSoftforksHeight) https://github.com/bitcoin/bitcoin/pull/16329
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #16328: rpc: Tidy up reporting of buried and ongoing softforks (master...1907-rpcSoftforks) https://github.com/bitcoin/bitcoin/pull/16328
< bitcoin-git> [bitcoin] hebasto opened pull request #16327: scripts and tools: Update ShellCheck linter (master...20190702-shellcheck) https://github.com/bitcoin/bitcoin/pull/16327
< bitcoin-git> [bitcoin] jnewbery opened pull request #16326: [RPC] add new utxoupdatepsbt arguments to the CRPCCommand and CPRCCvertParam tables (master...2019-07-utxoupdatepsbt-args) https://github.com/bitcoin/bitcoin/pull/16326
< gribble> https://github.com/bitcoin/bitcoin/issues/16106 | gui: Sort wallets in open wallet menu by promag · Pull Request #16106 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj merged pull request #16153: Qt: Add antialiasing to traffic graph widget (master...antialiasing) https://github.com/bitcoin/bitcoin/pull/16153
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/c84349a14832...4db2f8cf0f63
< bitcoin-git> bitcoin/master db26e8e Josu Goñi: Add antialiasing to traffic graph widget
< bitcoin-git> bitcoin/master 4db2f8c Wladimir J. van der Laan: Merge #16153: Qt: Add antialiasing to traffic graph widget
< bitcoin-git> [bitcoin] instagibbs closed pull request #16292: wallet_resendwallettransaction.py: fix coinbase height (master...resend_test_height) https://github.com/bitcoin/bitcoin/pull/16292
< gribble> https://github.com/bitcoin/bitcoin/issues/16325 | rpc: Clarify that block count means height excl genesis by MarcoFalke · Pull Request #16325 · bitcoin/bitcoin · GitHub
< instagibbs> promag, you just linked the same PR? :P https://github.com/bitcoin/bitcoin/pull/16292#issuecomment-507754182
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #16325: rpc: Clarify that block count means height excl genesis (master...1907-rpcBlockCount) https://github.com/bitcoin/bitcoin/pull/16325
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #16324: Get cs_main out of the critical path in ProcessMessages (master...2019-07-peerstate-initial-moves) https://github.com/bitcoin/bitcoin/pull/16324
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #16323: Call ProcessNewBlock() asynchronously (master...2019-07-background-pnb) https://github.com/bitcoin/bitcoin/pull/16323
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #16314: scripts and tools: Update copyright_header.py script (master...20190629-copyright-headers) https://github.com/bitcoin/bitcoin/pull/16314
< bitcoin-git> [bitcoin] MarcoFalke pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/2f717fb5cdfc...c84349a14832
< bitcoin-git> bitcoin/master 01fafe5 Hennadii Stepanov: Include Objective-C source files
< bitcoin-git> bitcoin/master 621463d Hennadii Stepanov: Drop no-longer-relevant copyright holder name
< bitcoin-git> bitcoin/master ca11606 Hennadii Stepanov: Fix: "Bitcoin Core" -> "The Bitcoin Core"
< bitcoin-git> [bitcoin] laanwj merged pull request #15427: Add support for descriptors to utxoupdatepsbt (master...201902_utxoupdatepsbtdesc) https://github.com/bitcoin/bitcoin/pull/15427
< bitcoin-git> [bitcoin] laanwj pushed 5 commits to master: https://github.com/bitcoin/bitcoin/compare/c6e42f1ca9c8...2f717fb5cdfc
< bitcoin-git> bitcoin/master eaf4f88 Pieter Wuille: Abstract out IsSegWitOutput from utxoupdatepsbt
< bitcoin-git> bitcoin/master fb90ec3 Pieter Wuille: Abstract out EvalDescriptorStringOrObject from scantxoutset
< bitcoin-git> bitcoin/master 3135c1a Pieter Wuille: Abstract out UpdatePSBTOutput from FillPSBT
< bitcoin-git> [bitcoin] laanwj merged pull request #14193: validation: Add missing mempool locks (master...Mf1809-valLocksMempool) https://github.com/bitcoin/bitcoin/pull/14193
< bitcoin-git> [bitcoin] laanwj pushed 4 commits to master: https://github.com/bitcoin/bitcoin/compare/6c21a801f3df...c6e42f1ca9c8
< bitcoin-git> bitcoin/master fa0c9db MarcoFalke: txpool: Make nTransactionsUpdated atomic
< bitcoin-git> bitcoin/master fabeb1f MarcoFalke: validation: Add missing mempool locks
< bitcoin-git> bitcoin/master fa2b083 MarcoFalke: [test] Add test to check mempool consistency in case of reorgs
< bitcoin-git> [bitcoin] promag opened pull request #16322: wallet: Fix -maxtxfee check by moving it to CWallet::CreateTransaction (master...2019-07-fix-16257) https://github.com/bitcoin/bitcoin/pull/16322
< gribble> https://github.com/bitcoin/bitcoin/issues/16296 | gui: crash with loadwallet & QT_FATAL_WARNINGS · Issue #16296 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/16307 | scheduler: crash after releasing wallet · Issue #16307 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/16261 | gui: Refactor OpenWalletActivity by promag · Pull Request #16261 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj merged pull request #15538: wallet_bumpfee.py: Make sure coin selection produces change (master...bumpfee_change_test) https://github.com/bitcoin/bitcoin/pull/15538
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/3ccab6470a61...6c21a801f3df
< bitcoin-git> bitcoin/master 276972c Gregory Sanders: wallet_bumpfee.py: Make sure coin selection produces change
< bitcoin-git> bitcoin/master 6c21a80 Wladimir J. van der Laan: Merge #15538: wallet_bumpfee.py: Make sure coin selection produces change
< bitcoin-git> [bitcoin] laanwj merged pull request #16212: addrdb: Avoid eating inodes - remove temporary files created by SerializeFileDB in case of errors (master...SerializeFileDB-non-determinism) https://github.com/bitcoin/bitcoin/pull/16212
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/1212808762f6...3ccab6470a61
< bitcoin-git> bitcoin/master d975338 practicalswift: addrdb: Remove temporary files created in SerializeFileDB. Fixes non-deter...
< bitcoin-git> bitcoin/master 3ccab64 Wladimir J. van der Laan: Merge #16212: addrdb: Avoid eating inodes - remove temporary files created...

2019-07-01

< wumpus> pinheadmz: that was always the problem, I'm usually happy to help people but it was always the same discussions about bitcoin over and over again, that just wears you out
< pinheadmz> The room "#bitcoin" on "chat.freenode.net" is invite only. -- getting this message today all of a sudden, anyone else?
< gribble> https://github.com/bitcoin/bitcoin/issues/16277 | [Tests] Suppress output in test_bitcoin for expected errors by gertjaap · Pull Request #16277 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/16035 | 0.18.1: Backports by MarcoFalke · Pull Request #16035 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj merged pull request #16257: [wallet] abort when attempting to fund a transaction above -maxtxfee (master...2019/06/max_fee) https://github.com/bitcoin/bitcoin/pull/16257
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/b3edacb5299b...1212808762f6
< bitcoin-git> bitcoin/master 806b005 Sjors Provoost: [wallet] abort when attempting to fund a transaction above maxtxfee
< bitcoin-git> bitcoin/master 1212808 Wladimir J. van der Laan: Merge #16257: [wallet] abort when attempting to fund a transaction above -...
< bitcoin-git> [bitcoin] practicalswift opened pull request #16320: ci: Add Travis check to catch accidental introduction of non-determinism in unit tests (master...test-deterministic-coverage-in-travis) https://github.com/bitcoin/bitcoin/pull/16320
< bitcoin-git> [bitcoin] NicolasDorier opened pull request #16319: [MSVC] Simplify AppVeyor build by using user customized project file (master...msvc/appveyor-refactor) https://github.com/bitcoin/bitcoin/pull/16319
< bitcoin-git> [bitcoin] fanquake merged pull request #16309: [MSVC] allow user level project customization (master...msvc/user-build) https://github.com/bitcoin/bitcoin/pull/16309
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/6e9eda556717...b3edacb5299b
< bitcoin-git> bitcoin/master c83f0ac nicolas.dorier: [MSVC] allow user level project customization
< bitcoin-git> bitcoin/master b3edacb fanquake: Merge #16309: [MSVC] allow user level project customization

2019-06-30

< bitcoin-git> [bitcoin] MarcoFalke merged pull request #16271: build: remove -Wall from rapidcheck build flags (master...rapidcheck_no_wall) https://github.com/bitcoin/bitcoin/pull/16271
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/e7f3450b7d36...6e9eda556717
< bitcoin-git> bitcoin/master 63d0a07 fanquake: build: dont compile rapidcheck with -Wall
< bitcoin-git> bitcoin/master 6e9eda5 MarcoFalke: Merge #16271: build: remove -Wall from rapidcheck build flags
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #16302: test: Add missing syncwithvalidationinterfacequeue to wallet_balance test (master...1906-testWalletBal) https://github.com/bitcoin/bitcoin/pull/16302
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/fd925e9b1cca...e7f3450b7d36
< bitcoin-git> bitcoin/master fa81525 MarcoFalke: test: Add missing sync_all to wallet_balance test
< bitcoin-git> bitcoin/master e7f3450 MarcoFalke: Merge #16302: test: Add missing syncwithvalidationinterfacequeue to wallet...
< bitcoin-git> [bitcoin] promag closed pull request #12578: gui: Add transaction record type Fee (master...2018-03-fee-transaction-record) https://github.com/bitcoin/bitcoin/pull/12578
< bitcoin-git> [bitcoin] emilengler opened pull request #16317: RPC: Add feature to getblock by using the index (master...getblock-improvement) https://github.com/bitcoin/bitcoin/pull/16317
< phantomcircuit> it was called "Bitcoin Wallet" by ... however you spell his name
< sipa> phantomcircuit, gwillen: well... i don't think; bitcoin wallet for android started as a project when afaik literally it was the first android wallet (it may have been the first wallet software besides core entirely); i don't think in that context it was unreasonable to call it that way
< bitcoin-git> [bitcoin] fanquake merged pull request #16313: doc: Fix broken link in doc/build-osx.md (master...fix-broken-link-in-build-osx-docs) https://github.com/bitcoin/bitcoin/pull/16313
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/04710b6d8c9e...fd925e9b1cca
< bitcoin-git> bitcoin/master 90b5c4e Jon Atack: doc: Fix broken link in doc/build-osx.md
< bitcoin-git> bitcoin/master fd925e9 fanquake: Merge #16313: doc: Fix broken link in doc/build-osx.md
< sipa> this person seems confused too: https://bitcoin.stackexchange.com/q/88716/208
< gwillen> phantomcircuit: calling it "bitcoin wallet developers" is? Yeah it's pretty dubious.
< phantomcircuit> gwillen, bitcoin wallet by bitcoin wallet developers is schilbach i think
< harding> gwillen: that email address isn't legit. I let the owner of bitcoin.org know so he could file his own report (I filed a report myself).

2019-06-29

< gwillen> luke-jr: I have no idea, that would make sense but "Bitcoin Wallet developers" is very vague so it made me suspicious. It has strong reviews though.
< gwillen> hm, the developer contact email is listed as "adminsupport@bitcoin.org"
< luke-jr> "Bitcoin Wallet" is the original Android wallet, right?
< gwillen> (this is on "Bitcoin Core - BTC Wallet")
< gwillen> midnightmagic: there is another one called "Bitcoin Wallet" by "Bitcoin Wallet developers"
< midnightmagic> er. the bitcoin core - btc wallet one I mean is just a rebranded breadwallet.
< gwillen> midnightmagic: I assume you are looking at "Bitcoin Core - BTC Wallet" by "Core Org"? (I have no idea what it is, I endorse marking it malware if nobody knows what it is.)
< bitcoin-git> [bitcoin] hebasto opened pull request #16314: scripts and tools: Update copyright_header.py script (master...20190629-copyright-headers) https://github.com/bitcoin/bitcoin/pull/16314
< bitcoin-git> [bitcoin] jonatack opened pull request #16313: doc: Fix broken link in doc/build-osx.md (master...fix-broken-link-in-build-osx-docs) https://github.com/bitcoin/bitcoin/pull/16313
< bitcoin-git> [bitcoin] practicalswift opened pull request #16312: tests: Reduce compile-time memory usage by 0.5 GB when compiling script_tests.cpp (from 1.4 GB to 0.9 GB) (master...fix-absurd-memory-usage-when-compiling-script_build) https://github.com/bitcoin/bitcoin/pull/16312
< bitcoin-git> [bitcoin] practicalswift closed pull request #15814: Add tests for (or remove) untested unused methods (master...untested-unused-methods) https://github.com/bitcoin/bitcoin/pull/15814
< bitcoin-git> [bitcoin] shargon opened pull request #16310: Small optimization for IsHex method (master...master) https://github.com/bitcoin/bitcoin/pull/16310
< bitcoin-git> [bitcoin] NicolasDorier opened pull request #16309: [MSVC] allow user level project customization (master...msvc/user-build) https://github.com/bitcoin/bitcoin/pull/16309
< bitcoin-git> [bitcoin] fanquake merged pull request #15964: Docs: Improve build-osx document formatting (master...feature/update-macOS-doc) https://github.com/bitcoin/bitcoin/pull/15964
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/0f309541aa6d...04710b6d8c9e
< bitcoin-git> bitcoin/master dbd137a Giulio Lombardo: Improve build-osx formatting
< bitcoin-git> bitcoin/master 04710b6 fanquake: Merge #15964: Docs: Improve build-osx document formatting
< bitcoin-git> [bitcoin] NicolasDorier opened pull request #16308: [MSVC] Copy build output to src/ automatically after build (master...build/copy-in-msbuild) https://github.com/bitcoin/bitcoin/pull/16308
< bitcoin-git> [bitcoin] fanquake merged pull request #16258: [MSVC]: Create the config.ini as part of bitcoind build (master...msvc/build-config-ini) https://github.com/bitcoin/bitcoin/pull/16258
< bitcoin-git> [bitcoin] fanquake pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/935cd6b1ec08...0f309541aa6d
< bitcoin-git> bitcoin/master e47e793 nicolas.dorier: [MSVC]: Create the config.ini as part of bitcoind build
< bitcoin-git> bitcoin/master 819c5dd nicolas.dorier: [MSVC] Enable Fuzz for functional tests
< bitcoin-git> bitcoin/master 0f30954 fanquake: Merge #16258: [MSVC]: Create the config.ini as part of bitcoind build
< gribble> https://github.com/bitcoin/bitcoin/issues/16035 | 0.18.1: Backports by MarcoFalke · Pull Request #16035 · bitcoin/bitcoin · GitHub

2019-06-28

< bitcoin-git> [bitcoin] MarcoFalke closed pull request #16303: bench: Replace <wallet/crypter.h> with <keystore.h> in src/bench/ccoins_caching.cpp (master...patch/replace_crypter_include) https://github.com/bitcoin/bitcoin/pull/16303
< bitcoin-git> [bitcoin] l2a5b1 opened pull request #16303: bench: Replace <wallet/crypter.h> with <keystore.h> in src/bench/ccoins_caching.cpp (master...patch/replace_crypter_include) https://github.com/bitcoin/bitcoin/pull/16303
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #16302: test: Add missing sync_all to wallet_balance test (master...1906-testWalletBal) https://github.com/bitcoin/bitcoin/pull/16302
< bitcoin-git> [bitcoin] laanwj merged pull request #16300: util: Explain why the path is cached (master...1906-utilPathWhy) https://github.com/bitcoin/bitcoin/pull/16300
< bitcoin-git> [bitcoin] laanwj pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/7400135b7918...935cd6b1ec08
< bitcoin-git> bitcoin/master fa69c3e MarcoFalke: util: Explain why the path is cached
< bitcoin-git> bitcoin/master 935cd6b Wladimir J. van der Laan: Merge #16300: util: Explain why the path is cached
< bitcoin-git> [bitcoin] achow101 opened pull request #16301: Use CWallet::Import* functions in all import* RPCs (master...imports-use-cwallet-funcs) https://github.com/bitcoin/bitcoin/pull/16301

2019-06-27

< bitcoin-git> [bitcoin] meshcollider closed pull request #16192: Wallet: Catches situations where capping on maxtxfee drops the fee too low (master...issue-10122) https://github.com/bitcoin/bitcoin/pull/16192
< gribble> https://github.com/bitcoin/bitcoin/issues/16285 | rpc: Improve scantxoutset response and help message by promag · Pull Request #16285 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13339 | wallet: Replace %w by wallet name in -walletnotify script by promag · Pull Request #13339 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13339 | wallet: Replace %w by wallet name in -walletnotify script by promag · Pull Request #13339 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/16035 | 0.18.1: Backports by MarcoFalke · Pull Request #16035 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/16035 | 0.18.1: Backports by MarcoFalke · Pull Request #16035 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/16257 | [wallet] abort when attempting to fund a transaction above -maxtxfee by Sjors · Pull Request #16257 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/15450 | [GUI] Create wallet menu option by achow101 · Pull Request #15450 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/16227 | Refactor CWallets inheritance chain by achow101 · Pull Request #16227 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/16035 | 0.18.1: Backports by MarcoFalke · Pull Request #16035 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #16300: util: Explain why the path is cached (master...1906-utilPathWhy) https://github.com/bitcoin/bitcoin/pull/16300
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #16255: util: Remove code to cache datadir (master...1906-utilNoPath) https://github.com/bitcoin/bitcoin/pull/16255
< bitcoin-git> [bitcoin] MarcoFalke reopened pull request #16294: qt: test: Create at most one testing setup (master...1906-qtTestOnlyOneTestingSetup) https://github.com/bitcoin/bitcoin/pull/16294
< gribble> https://github.com/bitcoin/bitcoin/issues/16292 | wallet_resendwallettransaction.py: fix coinbase height by instagibbs · Pull Request #16292 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #16278: tests: Remove unused includes (master...cut-compilation-bloat-in-unit-tests) https://github.com/bitcoin/bitcoin/pull/16278
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/3077f11dadff...7400135b7918
< bitcoin-git> bitcoin/master 9a84169 practicalswift: tests: Reduce compilation time and unneccessary recompiles by removing unu...
< bitcoin-git> bitcoin/master 7400135 MarcoFalke: Merge #16278: tests: Remove unused includes
< bitcoin-git> [bitcoin] promag opened pull request #16299: bench: Move generated data to a dedicated translation unit (master...2019-06-benchmark-data) https://github.com/bitcoin/bitcoin/pull/16299
< bitcoin-git> [bitcoin] promag closed pull request #12419: Force distinct destinations in CWallet::CreateTransaction (master...2018-02-distinct-destinations) https://github.com/bitcoin/bitcoin/pull/12419
< gribble> https://github.com/bitcoin/bitcoin/issues/16027 | client 0.18.0 crashes when computer wakes up from hibernation · Issue #16027 · bitcoin/bitcoin · GitHub

2019-06-26

< bitcoin-git> [bitcoin] MarcoFalke closed pull request #16294: qt: test: Create at most one testing setup (master...1906-qtTestOnlyOneTestingSetup) https://github.com/bitcoin/bitcoin/pull/16294
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #16289: test: Add missing ECC_Stop() in GUI rpcnestedtests.cpp (master...2019/06/fix_rpcnested_test) https://github.com/bitcoin/bitcoin/pull/16289
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/1b28bca04c27...3077f11dadff
< bitcoin-git> bitcoin/master f466c4c Jonas Schnelli: Add missing ECC_Stop(); in GUI rpcnestedtests.cpp
< bitcoin-git> bitcoin/master 3077f11 MarcoFalke: Merge #16289: test: Add missing ECC_Stop() in GUI rpcnestedtests.cpp
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #16294: qt: test: Create at most one testing setup (master...1906-qtTestOnlyOneTestingSetup) https://github.com/bitcoin/bitcoin/pull/16294
< dongcarl> sipa: The rationale here is that SHA-NI seems to be faster than SSE4/AVX2 in all cases? https://github.com/bitcoin/bitcoin/pull/13386/files#diff-cf563899256ae1ec7ee78b61806e40caR608
< dongcarl> For #13989, I'm wondering what the policy is for adding intrinsics like this. According to jamesob's benchmarks here https://github.com/bitcoin/bitcoin/pull/13989#issuecomment-497381424. I see that: 1. The PR significantly improves SHA-intensive microbenchmarks: `micro.gcc.MerkleRoot` and `micro.gcc.SHA256D64_1024`. 2. It does not speed up IBD performance significantly.
< gribble> https://github.com/bitcoin/bitcoin/issues/13989 | add avx512 instrinsic by fingera · Pull Request #13989 · bitcoin/bitcoin · GitHub
< MarcoFalke> E.g the arm hash: https://travis-ci.org/bitcoin/bitcoin/jobs/550863420#L160 is 42c149c5429f559658e55580a560c8f631c5a8c8bb2db228b3b73635f0e90558
< MarcoFalke> And the win64 hash: https://travis-ci.org/bitcoin/bitcoin/jobs/550927504#L159 is d42e27fad05b3f827b638d9dda4edcac0b86aeefd6658d03f48468bbde9b1845
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #16293: test: Make test cases separate functions (master...1906-testNoScopeLeak) https://github.com/bitcoin/bitcoin/pull/16293
< bitcoin-git> [bitcoin] practicalswift closed pull request #16045: Skip redundant memset(p, 0, 0) calls where p is not valid for writing (master...redundant-memsets) https://github.com/bitcoin/bitcoin/pull/16045
< jtimon> regarding https://github.com/bitcoin/bitcoin/pull/8994 I'm not sure it needs conceptual review (I'm sure there's many concept acks buried in the comments), but mostly review and re-review
< bitcoin-git> [bitcoin] instagibbs opened pull request #16292: wallet_resendwallettransaction.py: fix coinbase height (master...resend_test_height) https://github.com/bitcoin/bitcoin/pull/16292
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #16291: gui: Stop translating PACKAGE_NAME (master...1906-noTranslatePackageName) https://github.com/bitcoin/bitcoin/pull/16291
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #16287: refactor: remove extra CBlockIndex declaration (master...remove-extra-cblockindex) https://github.com/bitcoin/bitcoin/pull/16287
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/ca80fec973f0...1b28bca04c27
< bitcoin-git> bitcoin/master 9824a0d RJ Rybarczyk: Remove extra CBlockIndex declaration
< bitcoin-git> bitcoin/master 1b28bca MarcoFalke: Merge #16287: refactor: remove extra CBlockIndex declaration
< fanquake> If anyone is interested in some build system review. Cory has a PR open on the bitcoin-core/univalue repo: https://github.com/bitcoin-core/univalue/pull/19
< gribble> https://github.com/bitcoin/bitcoin/issues/16035 | 0.18.1: Backports by MarcoFalke · Pull Request #16035 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] fanquake closed pull request #16290: NULL to nullptr (master...null-to-nullptr) https://github.com/bitcoin/bitcoin/pull/16290
< bitcoin-git> [bitcoin] Bushstar opened pull request #16290: NULL to nullptr (master...null-to-nullptr) https://github.com/bitcoin/bitcoin/pull/16290
< bitcoin-git> [bitcoin] jonasschnelli opened pull request #16289: Add missing ECC_Stop(); in GUI rpcnestedtests.cpp (master...2019/06/fix_rpcnested_test) https://github.com/bitcoin/bitcoin/pull/16289
< bitcoin-git> [bitcoin] fanquake merged pull request #16286: refactoring: wallet: Fix GCC 7.4.0 warning (master...20190625-fix-warning) https://github.com/bitcoin/bitcoin/pull/16286
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/332c6134bb15...ca80fec973f0
< bitcoin-git> bitcoin/master d8bd97d Hennadii Stepanov: Fix GCC 7.4.0 warning