2017-04-29

< bitcoin-git> [bitcoin] sipa opened pull request #10299: Remove OpenSSL (master...simplerandom) https://github.com/bitcoin/bitcoin/pull/10299
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10256: [test] Add test for gettxout to wallet.py (master...test_gettxout) https://github.com/bitcoin/bitcoin/pull/10256
< bitcoin-git> bitcoin/master 80c3a73 MarcoFalke: Merge #10256: [test] Add test for gettxout to wallet.py...
< bitcoin-git> bitcoin/master dd1ea59 Jimmy Song: [test] Add gettxout call...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/4c924011f535...80c3a734298e

2017-04-28

< bitcoin-git> [bitcoin] fanquake closed pull request #10298: jslee1301 patch 1 (0.11...gh-jslee1301-patch-1) https://github.com/bitcoin/bitcoin/pull/10298
< bitcoin-git> [bitcoin] gh-jslee1301 opened pull request #10298: jslee1301 patch 1 (0.11...gh-jslee1301-patch-1) https://github.com/bitcoin/bitcoin/pull/10298
< bitcoin-git> [bitcoin] sipa opened pull request #10297: Simplify DisconnectBlock arguments/return value (master...disconnect_enum) https://github.com/bitcoin/bitcoin/pull/10297
< bitcoin-git> [bitcoin] ryanofsky opened pull request #10295: [qt] Move some WalletModel functions into CWallet (master...pr/ipc-move) https://github.com/bitcoin/bitcoin/pull/10295
< bitcoin-git> [bitcoin] instagibbs opened pull request #10294: [Wallet] unset change position when there is no change (master...fixchangepos) https://github.com/bitcoin/bitcoin/pull/10294
< bsm117532> bitcoin-cli -regtest fundrawtransaction 01000000000106220000000000001600149bf98693c5892c3e7b6d991cfeebf146285f59bd00000000 '{"changePosition": 1}'
< sipa> SopaXorzTaker: https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2015-July/009697.html read this first before you say great
< sipa> until bitcoin core 0.11
< achow101> SopaXorzTaker: the original bitcoin client was supposed to include a marketplace thing
< SopaXorzTaker> wumpus, sipa: here's an actual question about the bitcoin core
< bitcoin-git> [bitcoin] mm-s opened pull request #10292: Improved efficiency in COutPoint constructors (master...master) https://github.com/bitcoin/bitcoin/pull/10292
< bitcoin-git> [bitcoin] mm-s closed pull request #10277: Improved efficiency in constructors of COutPoint (master...master) https://github.com/bitcoin/bitcoin/pull/10277

2017-04-27

< bitcoin-git> [bitcoin] practicalswift reopened pull request #9544: [trivial] Add end of namespace comments. Improve consistency. (master...consistent-use-of-end-of-namespace-comments) https://github.com/bitcoin/bitcoin/pull/9544
< bitcoin-git> [bitcoin] practicalswift closed pull request #9544: [trivial] Add end of namespace comments. Improve consistency. (master...consistent-use-of-end-of-namespace-comments) https://github.com/bitcoin/bitcoin/pull/9544
< jtimon> btw BlueMatt thanks for https://github.com/bitcoin/bitcoin/pull/771 I shouldn't look at it much and look at the new things instead but like these things if I find the time
< jtimon> wumpus: btw, it would be nice to put BlueMatt 's libconsensus-related PRs in https://github.com/bitcoin/bitcoin/projects/6 (and always tag anything there with "consensus")
<@wumpus> see "Ubuntu 16.04 Windows cross-build" on https://github.com/bitcoin/bitcoin/projects for a collection of issues
< gribble> https://github.com/bitcoin/bitcoin/issues/7729 | rpc: introduce label API for wallet by laanwj · Pull Request #7729 · bitcoin/bitcoin · GitHub
< BlueMatt> luke-jr: thats ridiculous, you're living in a world where people have the time go read a ton of bitcoin core docs/code before running it, and do...neither of which are true
< gribble> https://github.com/bitcoin/bitcoin/issues/7729 | rpc: introduce label API for wallet by laanwj · Pull Request #7729 · bitcoin/bitcoin · GitHub
< petertodd> gmaxwell: btw, re: your comment in the meeting, no-one's funding me to do any work on Bitcoin Core these days
< BlueMatt> jtimon: conf flag seems neater, no one checks their bitcoin datadir
< jtimon> it should still be relatively easy for users to get out of the stuck situation in case they can't upgrade in the same system or something, like maybe deleting a filed named ~/.bitcoin/DELETE_ME_ONLY_IF_YOU_CANT_UPGRADE_IN_THIS_SYSTEM or something
< luke-jr> BlueMatt: it's only slightly shorter than Bitcoin's current lifetime
< jtimon> or shouldn't I recommend bitcoin core for a wallet?
< jtimon> luke-jr: let's say my friend remembers what I told him about being up to date 6 years and 11 months after I helped him install bitcoin core
< BlueMatt> luke-jr: 110% sends the wrong message. if i expected any reasonable person to see that and think "I need to think for myself about what consensus of the network is" I'd be happy with it, but realistically the only people reading that will think "oh, I have to switch to the latest thing from Bitcoin Core, for whatever Bitcoin Core is according to my local google server"
< jtimon> I think if it's optional and disabled by default it kind of defeats the point, but I certainly don't want that for myself or the users I recommend to use bitcoin core
< gribble> https://github.com/bitcoin/bitcoin/issues/10282 | Expire bitcoind & bitcoin-qt 7-8 years after its last change by luke-jr · Pull Request #10282 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10195 | Switch chainstate db and cache to per-txout model by sipa · Pull Request #10195 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10148 | [WIP] Use non-atomic flushing with block replay by sipa · Pull Request #10148 · bitcoin/bitcoin · GitHub
< * jtimon> checks https://github.com/bitcoin/bitcoin/pull/8855 is on the priority list
< gribble> https://github.com/bitcoin/bitcoin/issues/10195 | Switch chainstate db and cache to per-txout model by sipa · Pull Request #10195 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10148 | [WIP] Use non-atomic flushing with block replay by sipa · Pull Request #10148 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10285 | net: refactor the connection process. moving towards async connections. by theuni · Pull Request #10285 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/7729 | rpc: introduce label API for wallet by laanwj · Pull Request #7729 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10195 | Switch chainstate db and cache to per-txout model by sipa · Pull Request #10195 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10199 | Better fee estimates by morcos · Pull Request #10199 · bitcoin/bitcoin · GitHub
<@wumpus> anything to add to project https://github.com/bitcoin/bitcoin/projects/8?
<@wumpus> jonasschnelli: I meant in bitcoin core; I don't know about other implementations
< sipa> (by bitcoin core)
< bitcoin-git> [bitcoin] sipa opened pull request #10290: Add -stopatheight for benchmarking (master...shutdown_at_height) https://github.com/bitcoin/bitcoin/pull/10290
< gribble> https://github.com/bitcoin/bitcoin/issues/10279 | Add a CChainState class to validation.cpp to take another step towards clarifying internal interfaces by TheBlueMatt · Pull Request #10279 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/771 | CBlockStore by TheBlueMatt · Pull Request #771 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10240 | [WIP] Add basic HD wallet restore functionality by jonasschnelli · Pull Request #10240 · bitcoin/bitcoin · GitHub
<@wumpus> #bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr btcdrak sdaftuar jtimon cfields petertodd kanzure bluematt
< bitcoin-git> [bitcoin] laanwj closed pull request #10075: Remove unused C++ code not covered by unit tests (master...unused) https://github.com/bitcoin/bitcoin/pull/10075
< bitcoin-git> bitcoin/master 4c92401 Wladimir J. van der Laan: Merge #10075: Remove unused C++ code not covered by unit tests...
< bitcoin-git> bitcoin/master b51aaf1 practicalswift: Remove unused C++ code not covered by unit tests
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a550f6e415fd...4c924011f535
< bitcoin-git> [bitcoin] sipa closed pull request #10283: Cleanup: reduce to one GetMinimumFee call signature (master...oneGetMinimumFee) https://github.com/bitcoin/bitcoin/pull/10283
< bitcoin-git> bitcoin/master a550f6e Pieter Wuille: Merge #10283: Cleanup: reduce to one GetMinimumFee call signature...
< bitcoin-git> bitcoin/master 3edbd79 Alex Morcos: cleanup: reduce to one GetMinimumFee call signature
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/47535d7c3ec7...a550f6e415fd
< bitcoin-git> [bitcoin] jimmysong opened pull request #10287: [tests] Update Unit Test for addrman.h/addrman.cpp (master...test_addrman) https://github.com/bitcoin/bitcoin/pull/10287
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #10286: Call wallet notify callbacks in scheduler thread (without cs_main) (master...2017-01-wallet-cache-inmempool-4) https://github.com/bitcoin/bitcoin/pull/10286
< gribble> https://github.com/bitcoin/bitcoin/issues/9254 | [depends] ZeroMQ 4.2.2 by fanquake · Pull Request #9254 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] theuni opened pull request #10285: net: refactor the connection process. moving towards async connections. (master...connman-events6) https://github.com/bitcoin/bitcoin/pull/10285

2017-04-26

< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10280: [test] Unit test amount.h/amount.cpp (master...test_amount) https://github.com/bitcoin/bitcoin/pull/10280
< bitcoin-git> bitcoin/master 47535d7 MarcoFalke: Merge #10280: [test] Unit test amount.h/amount.cpp...
< bitcoin-git> bitcoin/master dcb69fd Jimmy Song: [test] Unit test amount.h/amount.cpp...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/cf5782508a3f...47535d7c3ec7
< bitcoin-git> [bitcoin] morcos opened pull request #10284: Always log debug information for fee calculation in CreateTransaction (master...debugEstimates) https://github.com/bitcoin/bitcoin/pull/10284
< bitcoin-git> [bitcoin] morcos opened pull request #10283: Cleanup: reduce to one GetMinimumFee call signature (master...oneGetMinimumFee) https://github.com/bitcoin/bitcoin/pull/10283
< bitcoin-git> [bitcoin] luke-jr opened pull request #10282: Expire bitcoind & bitcoin-qt 7-8 years after its last change (master...timebomb) https://github.com/bitcoin/bitcoin/pull/10282
< bitcoin-git> [bitcoin] laanwj closed pull request #9614: [wallet] [refactor] Simplify getbalance implementation (master...pr/getbalance-cleanup) https://github.com/bitcoin/bitcoin/pull/9614
< bitcoin-git> bitcoin/master cf57825 Wladimir J. van der Laan: Merge #9614: [wallet] [refactor] Simplify getbalance implementation...
< bitcoin-git> bitcoin/master 02d9f50 Russell Yanofsky: [wallet] Remove unneeded legacy getbalance code
< bitcoin-git> bitcoin/master 82b7dc3 Russell Yanofsky: [wallet] Add GetLegacyBalance method to simplify getbalance RPC...
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/6fdb31916589...cf5782508a3f
< bitcoin-git> [bitcoin] practicalswift closed pull request #9545: Add final where appropriate (master...add-overrides-where-appropriate) https://github.com/bitcoin/bitcoin/pull/9545
< bitcoin-git> [bitcoin] fanquake closed pull request #9363: Reduce fingerprinting attacks (master...MinChainWorkBlocktxns) https://github.com/bitcoin/bitcoin/pull/9363
< bitcoin-git> [bitcoin] laanwj closed pull request #9743: Fix several potential issues found by sanitizers (master...fsanitize) https://github.com/bitcoin/bitcoin/pull/9743
< bitcoin-git> bitcoin/master 6fdb319 Wladimir J. van der Laan: Merge #9743: Fix several potential issues found by sanitizers...
< bitcoin-git> bitcoin/master 1d31093 Pieter Wuille: fix tsan: utiltime race on nMockTime
< bitcoin-git> bitcoin/master 321bbc2 Pieter Wuille: fix ubsan: bitcoin-tx: not initialize context before IsFullyValid
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/bd9ec0ef1ea7...6fdb31916589
< bitcoin-git> [bitcoin] laanwj closed pull request #9670: contrib: github-merge improvements (master...2017_01_ghmerge_update) https://github.com/bitcoin/bitcoin/pull/9670
< bitcoin-git> bitcoin/master bd9ec0e Wladimir J. van der Laan: Merge #9670: contrib: github-merge improvements...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8979f4569e0e...bd9ec0ef1ea7
< bitcoin-git> bitcoin/master b508424 Wladimir J. van der Laan: contrib: github-merge improvements...
< bitcoin-git> [bitcoin] laanwj closed pull request #10273: [scripts] Minor improvements to `macdeployqtplus` script. (master...macdeployqtplus) https://github.com/bitcoin/bitcoin/pull/10273
< bitcoin-git> bitcoin/master 8979f45 Wladimir J. van der Laan: Merge #10273: [scripts] Minor improvements to `macdeployqtplus` script....
< bitcoin-git> bitcoin/master e8babc4 Chris Gavin: Use `with` in `macdeployqtplus` script.
< bitcoin-git> bitcoin/master 4f3ac7d Chris Gavin: Remove unused variable from `macdeployqtplus` script.
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/8254a8ae213b...8979f4569e0e
< bitcoin-git> [bitcoin] laanwj closed pull request #10278: [test] Add Unit Test for GetListenPort (master...test_listenport) https://github.com/bitcoin/bitcoin/pull/10278
< bitcoin-git> bitcoin/master 1b14449 Jimmy Song: [test] Add Unit Test for GetListenPort...
< bitcoin-git> bitcoin/master 8254a8a Wladimir J. van der Laan: Merge #10278: [test] Add Unit Test for GetListenPort...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9c94fb6c3257...8254a8ae213b
< bitcoin-git> [bitcoin] laanwj closed pull request #9930: Trivial: Correct indentation and bracing (master...fix-indent) https://github.com/bitcoin/bitcoin/pull/9930
< bitcoin-git> [bitcoin] laanwj opened pull request #10281: doc: Add RPC interface guidelines (master...2017_04_rpc_if_guidelines) https://github.com/bitcoin/bitcoin/pull/10281
< bitcoin-git> [bitcoin] jimmysong opened pull request #10280: [test] Unit test amount.h/amount.cpp (master...test_amount) https://github.com/bitcoin/bitcoin/pull/10280
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #10279: Add a CChainState class to validation.cpp to take another step towards clarifying internal interfaces (master...2016-12-cconsensus) https://github.com/bitcoin/bitcoin/pull/10279

2017-04-25

< jonasschnelli> testnet-seed.bitcoin.jonasschnelli.ch should be up and running... do you use tor?
< ryan-c> host testnet-seed.bitcoin.jonasschnelli.ch | grep ' has address ' | awk '{print$4}' | xargs -n1 -I IP bitcoin-cli -testnet addnode IP add
< gribble> https://github.com/bitcoin/bitcoin/issues/10210 | Frozen (for 15 minutes) on shutdown · Issue #10210 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10215 | Check interruptNet during dnsseed lookups by TheBlueMatt · Pull Request #10215 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10209 | Stalled shutdown · Issue #10209 · bitcoin/bitcoin · GitHub
< BlueMatt> can we close #10209 now that we fixed the "regenerate bitcoin-config.h" issue and #10210 now that #10215 is merged?
< ryan-c> BlueMatt: It's the binary release from bitcoin.org, sha256sum 860a046514961aeead8b640f1a321da8eb575d727fd8c37de5a4cee5908bbd9d
< ryan-c> I told bitcoind 0.14.0 to shutdown via bitcoin-cli over four hours ago, and it's still running. strace shows it waiting on a futex call. Is there anything I should do before sending it a kill -9? The system is not low on memory.
< sipa> that doesn't mean there is no advantage for HT cores in bitcoin block validation, as ECDSA verification isn't the only thing involved
<@wumpus> cfields: jnewbery had already agreed with that, just he still had some concern: https://github.com/bitcoin/bitcoin/pull/10234#issuecomment-295354958
< bitcoin-git> [bitcoin] jimmysong opened pull request #10278: [test] Add Unit Test for GetListenPort (master...test_listenport) https://github.com/bitcoin/bitcoin/pull/10278
< bitcoin-git> [bitcoin] mm-s opened pull request #10277: Improved efficiency in constructors of COutPoint (master...master) https://github.com/bitcoin/bitcoin/pull/10277
< bitcoin-git> [bitcoin] laanwj closed pull request #10225: [test] Add aborttrescan tests (master...abort-rescan-tests) https://github.com/bitcoin/bitcoin/pull/10225
< bitcoin-git> bitcoin/master e0a7e19 Wladimir J. van der Laan: Merge #10225: [test] Add aborttrescan tests...
< bitcoin-git> bitcoin/master ed60970 Karl-Johan Alm: [test] Test abortrescan command.
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c29a0d48129d...e0a7e1994e6f
< bitcoin-git> [bitcoin] laanwj closed pull request #10265: [wallet] [moveonly] Check non-null pindex before potentially referencing (master...fix-check-pindex-scanforwallettx) https://github.com/bitcoin/bitcoin/pull/10265
< bitcoin-git> bitcoin/master c29a0d4 Wladimir J. van der Laan: Merge #10265: [wallet] [moveonly] Check non-null pindex before potentially referencing...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/cb007e4346cf...c29a0d48129d
< bitcoin-git> bitcoin/master c36ea69 Karl-Johan Alm: [wallet] Make sure pindex is non-null before possibly referencing in LogPrintf call.
< gribble> https://github.com/bitcoin/bitcoin/issues/10234 | [net] listbanned RPC and QT should show correct banned subnets by jnewbery · Pull Request #10234 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10234 | [net] listbanned RPC and QT should show correct banned subnets by jnewbery · Pull Request #10234 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #10263: Trivial: fix fee estimate write error log message (master...coinbase) https://github.com/bitcoin/bitcoin/pull/10263
< bitcoin-git> bitcoin/master cb007e4 Wladimir J. van der Laan: Merge #10263: Trivial: fix fee estimate write error log message...
< bitcoin-git> bitcoin/master 94807be CryptAxe: Trivial: fix fee estimate write error log message
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/95f5e4407502...cb007e4346cf
< bitcoin-git> [bitcoin] knocte opened pull request #10276: contrib/verifybinaries: allow filtering by platform (master...filterByPlatformInVerifySh) https://github.com/bitcoin/bitcoin/pull/10276
< bitcoin-git> [bitcoin] laanwj closed pull request #10270: Remove Clang workaround for Boost 1.46 (master...remove-boost-clang-workaround) https://github.com/bitcoin/bitcoin/pull/10270
< bitcoin-git> bitcoin/master 95f5e44 Wladimir J. van der Laan: Merge #10270: Remove Clang workaround for Boost 1.46...
< bitcoin-git> bitcoin/master 93dbb15 fanquake: Remove Clang workaround for Boost 1.46
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/54e2d87e792f...95f5e4407502
< bitcoin-git> [bitcoin] laanwj closed pull request #10274: Replace by fee v0.11.2 (master...replace-by-fee-v0.11.2) https://github.com/bitcoin/bitcoin/pull/10274
< bitcoin-git> [bitcoin] kallewoof opened pull request #10275: [rpc] Allow fetching tx directly from specified block in getrawtransaction (master...gettx-with-blockhash) https://github.com/bitcoin/bitcoin/pull/10275
< bitcoin-git> [bitcoin] ericchong opened pull request #10274: Replace by fee v0.11.2 (master...replace-by-fee-v0.11.2) https://github.com/bitcoin/bitcoin/pull/10274
< bitcoin-git> [bitcoin] chrisgavin opened pull request #10273: [scripts] Minor improvements to `macdeployqtplus` script. (master...macdeployqtplus) https://github.com/bitcoin/bitcoin/pull/10273
<@wumpus> unless someone implements https://github.com/bitcoin/bitcoin/issues/8973 to make onion connections come in on either an alternative port or a unix socket, it's unsafe to whitelist 127.0.0.1 [by default]
< bitcoin-git> [bitcoin] practicalswift closed pull request #10212: Make sure parameter names in .cpp and .h files are in sync (master...make-doxygen-happy-by-using-consistent-parameter-names) https://github.com/bitcoin/bitcoin/pull/10212
< bitcoin-git> [bitcoin] laanwj closed pull request #10272: [Tests] Prevent warning: variable 'x' is uninitialized (master...20170425_FastRandomContext_test_warnings) https://github.com/bitcoin/bitcoin/pull/10272
< bitcoin-git> bitcoin/master 54e2d87 Wladimir J. van der Laan: Merge #10272: [Tests] Prevent warning: variable 'x' is uninitialized...
< bitcoin-git> bitcoin/master 5ec8836 Pavel Janík: Prevent warning: variable 'x' is uninitialized
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c73af5416b66...54e2d87e792f
< bitcoin-git> [bitcoin] paveljanik opened pull request #10272: [Tests] Prevent warning: variable 'x' is uninitialized (master...20170425_FastRandomContext_test_warnings) https://github.com/bitcoin/bitcoin/pull/10272
< bitcoin-git> [bitcoin] fanquake opened pull request #10271: Use std::thread::hardware_concurrency, instead of Boost, to determine available cores (master...replace-boost-getnumcores) https://github.com/bitcoin/bitcoin/pull/10271
< bitcoin-git> [bitcoin] fanquake opened pull request #10270: Remove Clang workaround for Boost 1.46 (master...remove-boost-clang-workaround) https://github.com/bitcoin/bitcoin/pull/10270

2017-04-24

< bitcoin-git> [bitcoin] sipa closed pull request #10249: Switch CCoinsMap from boost to std unordered_map (master...stdcoinmap) https://github.com/bitcoin/bitcoin/pull/10249
< bitcoin-git> bitcoin/master c73af54 Pieter Wuille: Merge #10249: Switch CCoinsMap from boost to std unordered_map...
< bitcoin-git> bitcoin/master e6756ad Pieter Wuille: Switch CCoinsMap from boost to std unordered_map
< bitcoin-git> bitcoin/master 344a2c4 Pieter Wuille: Add support for std::unordered_{map,set} to memusage.h
< bitcoin-git> [bitcoin] sipa pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/fa1ac2881f2a...c73af5416b66
< gribble> https://github.com/bitcoin/bitcoin/issues/10260 | [doc] Minor corrections to osx dependencies by fanquake · Pull Request #10260 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #9951: Wallet database handling abstractions/simplifications (master...2017_03_wallet_dbwrapper) https://github.com/bitcoin/bitcoin/pull/9951
< bitcoin-git> bitcoin/master 071c955 Wladimir J. van der Laan: wallet: Get rid of fFileBacked...
< bitcoin-git> bitcoin/master be9e1a9 Wladimir J. van der Laan: wallet: Reduce references to global bitdb environment
< bitcoin-git> bitcoin/master 71afe3c Wladimir J. van der Laan: wallet: Introduce database handle wrapper...
< bitcoin-git> [bitcoin] laanwj pushed 7 new commits to master: https://github.com/bitcoin/bitcoin/compare/342b9bc3907e...fa1ac2881f2a
< gribble> https://github.com/bitcoin/bitcoin/issues/10192 | Cache full script execution results in addition to signatures by TheBlueMatt · Pull Request #10192 · bitcoin/bitcoin · GitHub
< fanquake> Since we seem to be giving Boost removal a good shot for 0.15, does anyone have suggestions for replacing GetNumCores? https://github.com/bitcoin/bitcoin/blob/master/src/util.cpp#L881
<@wumpus> "bitcoin mining on mars"
<@wumpus> BlueMatt: ah yes the "apply all the buzzwords to bitcoin mining, see what sticks" approach
< BlueMatt> wumpus: the same bottom-of-the-barrel buzzword bingo that came up with "using normal computers to simulate quantum computers to do bitcoin mining"
< bitcoin-git> [bitcoin] laanwj closed pull request #9792: FastRandomContext improvements and switch to ChaCha20 (master...chacha) https://github.com/bitcoin/bitcoin/pull/9792
< bitcoin-git> bitcoin/master e04326f Pieter Wuille: Add ChaCha20
< bitcoin-git> bitcoin/master 663fbae Pieter Wuille: FastRandom benchmark
< bitcoin-git> bitcoin/master c21cbe6 Pieter Wuille: Introduce FastRandomContext::randbool()
< bitcoin-git> [bitcoin] laanwj pushed 6 new commits to master: https://github.com/bitcoin/bitcoin/compare/1b25b6df0f08...342b9bc3907e
<@wumpus> "using deep learning frameworks in bitcoin mining", that's hilarious, what bottom-of-the-barrel buzzword factory did that come out of
< bitcoin-git> [bitcoin] kallewoof opened pull request #10267: New -readconfig argument for including external configuration files (master...feature-config-readconfig) https://github.com/bitcoin/bitcoin/pull/10267
< bitcoin-git> [bitcoin] fanquake closed pull request #10266: Update policy.h (master...patch-1) https://github.com/bitcoin/bitcoin/pull/10266
< bitcoin-git> [bitcoin] 1mbtxn opened pull request #10266: Update policy.h (master...patch-1) https://github.com/bitcoin/bitcoin/pull/10266

2017-04-23

< bitcoin-git> [bitcoin] kallewoof opened pull request #10265: [wallet] Check non-null pindex before potentially referencing (master...fix-check-pindex-scanforwallettx) https://github.com/bitcoin/bitcoin/pull/10265
< gmaxwell> Can someone please close: https://github.com/bitcoin/bitcoin/issues/6876
< bitcoin-git> [bitcoin] jimmysong opened pull request #10264: [test] Add tests for getconnectioncount, getnettotals and ping (master...test_netrpccalls) https://github.com/bitcoin/bitcoin/pull/10264
< bitcoin-git> [bitcoin] CryptAxe reopened pull request #10263: Trivial: fix fee estimate write error log message (master...coinbase) https://github.com/bitcoin/bitcoin/pull/10263
< bitcoin-git> [bitcoin] CryptAxe closed pull request #10263: Trivial: fix fee estimate write error log message (master...coinbase) https://github.com/bitcoin/bitcoin/pull/10263
< bitcoin-git> [bitcoin] CryptAxe opened pull request #10263: Trivial: fix fee estimate write error log message (master...coinbase) https://github.com/bitcoin/bitcoin/pull/10263
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10097: Move zmq test skipping logic into individual test case. (master...zmq_optional) https://github.com/bitcoin/bitcoin/pull/10097
< bitcoin-git> bitcoin/master 1b25b6d MarcoFalke: Merge #10097: Move zmq test skipping logic into individual test case....
< bitcoin-git> bitcoin/master 6803e09 John Newbery: Move zmq test skipping logic into individual test case....
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c530c1518063...1b25b6df0f08
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10023: [tests] remove maxblocksinflight.py (functionality covered by other test) (master...improvemaxblocksinflight) https://github.com/bitcoin/bitcoin/pull/10023
< bitcoin-git> bitcoin/master c530c15 MarcoFalke: Merge #10023: [tests] remove maxblocksinflight.py (functionality covered by other test)...
< bitcoin-git> bitcoin/master 5f4bcf2 John Newbery: [tests] Remove maxblocksinflight testcase...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2723bcdce324...c530c1518063
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10224: [test] Add test for getaddednodeinfo (master...test_getaddednodeinfo) https://github.com/bitcoin/bitcoin/pull/10224
< bitcoin-git> bitcoin/master 2723bcd MarcoFalke: Merge #10224: [test] Add test for getaddednodeinfo...
< bitcoin-git> bitcoin/master bc53752 Jimmy Song: Tests: Add simple test for getaddednodeinfo...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ed22eb4a62bd...2723bcdce324
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10258: Fixed typo in documentation for merkleblock.h (master...master) https://github.com/bitcoin/bitcoin/pull/10258
< bitcoin-git> bitcoin/master dd07068 Mikerah: Fixed typo in documentation for merkleblock.h
< bitcoin-git> bitcoin/master ed22eb4 MarcoFalke: Merge #10258: Fixed typo in documentation for merkleblock.h...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f8feaa463626...ed22eb4a62bd
< bitcoin-git> [bitcoin] fanquake opened pull request #10260: [doc] Minor corrections to osx dependencies (master...osx-doc-update) https://github.com/bitcoin/bitcoin/pull/10260

2017-04-22

< bitcoin-git> [bitcoin] jonasschnelli closed pull request #10254: [Qt] Remove shutdown poll timer and replace it with a signal (master...2017/04/qt_shutdown) https://github.com/bitcoin/bitcoin/pull/10254
< gribble> https://github.com/bitcoin/bitcoin/issues/8855 | Use a proper factory for creating chainparams by jtimon · Pull Request #8855 · bitcoin/bitcoin · GitHub
< bitcoin-git> bitcoin/0.14 f2a96e7 Wladimir J. van der Laan: doc: clean out release notes
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to 0.14: https://github.com/bitcoin/bitcoin/commit/f2a96e7d0289682eb87877a3528cb6a170f8b786
< bitcoin-git> bitcoin/master f8feaa4 Wladimir J. van der Laan: doc: Add historical release notes for 0.14.1
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/f8feaa4636260b599294c7285bcf1c8b7737f74e
< BashCo> getting a 403 on the torrent file? https://bitcoin.org/bin/bitcoin-core-0.14.1/bitcoin-0.14.1.torrent
< sipa> i promise i'll personally try to answer your questions on https://bitcoin.stackexchange.com
<@wumpus> channel topic: Bitcoin Core development discussion and commit log | This is the channel for developing Bitcoin Core. Feel free to watch, but please take commentary and usage questions to #bitcoin | Channel logs: https://botbot.me/freenode/bitcoin-core-dev, http://www.erisian.com.au/bitcoin-core-dev/
<@wumpus> just ask these things in #bitcoin okay - more people that can answer, or use the stack overflow
< sipa> this channel is aboit development of bitcoin-core
< SopaXorzTaker> sipa, I have to disagree. The question I asked *is* about the Bitcoin Core
< sipa> SopaXorzTaker: bitcoin.stackexchange.com
< SopaXorzTaker> sipa, a question related to the bitcoin core: is the UTXO database initialized when loading the blockchain?
<@wumpus> this channel is about development of the bitcoin core code base, nothing else
< sipa> if you have more questions like this in the future, please go to bitcoin.stackexchange.com... i'll gladly answer there
< SopaXorzTaker> I am asking about the motivation of some Bitcoin design choices
< bitcoin-git> [bitcoin] Mikerah opened pull request #10258: Fixed typo in documentation for merkleblock.h (master...master) https://github.com/bitcoin/bitcoin/pull/10258
< bitcoin-git> [bitcoin] jimmysong opened pull request #10257: [test] Add test for getmemoryinfo (master...test_getmemoryinfo) https://github.com/bitcoin/bitcoin/pull/10257
< achow101> I made dummy github user non-github-bitcoin (https://github.com/non-github-bitcoin) to account for the commits made by the people who committed but don't have github accounts. right now it has sirius-m (github removed it from saracen to allow me to add that account) and lasloh. I'll be asking github to also let me add s_nakamoto which is currently under invalid-email-address. I think it would be prudent to give the repo maintainers
< bitcoin-git> [bitcoin] jimmysong opened pull request #10256: [test] Add gettxout call to wallet.py (master...test_gettxout) https://github.com/bitcoin/bitcoin/pull/10256

2017-04-21

< bitcoin-git> [bitcoin] jimmysong opened pull request #10255: [test] Add test for listaddressgroupings (master...test_listaddressgroupings) https://github.com/bitcoin/bitcoin/pull/10255
< bitcoin-git> [bitcoin] jonasschnelli opened pull request #10254: [Qt] Remove shutdown poll timer and replace it with a signal (master...2017/04/qt_shutdown) https://github.com/bitcoin/bitcoin/pull/10254
< bitcoin-git> [bitcoin] jtimon closed pull request #10227: Make functions in validation.cpp static: (master...b14-chainparams-validation-static) https://github.com/bitcoin/bitcoin/pull/10227
< jtimon_> wumpus: it seems you added a commit to https://github.com/bitcoin/bitcoin/pull/10201 by mistake?
< bitcoin-git> [bitcoin] jimmysong opened pull request #10253: Tests: Add test for getnetworkhashps (master...test_getnetworkhashps) https://github.com/bitcoin/bitcoin/pull/10253
< bitcoin-git> [bitcoin] luke-jr opened pull request #10252: RPC/Mining: Restore API compatibility for prioritisetransaction (master...rpc_prioritise_api) https://github.com/bitcoin/bitcoin/pull/10252
< bitcoin-git> bitcoin/master a6548a4 Wladimir J. van der Laan: Merge #10201: pass Consensus::Params& to ReceivedBlockTransactions()...
< bitcoin-git> bitcoin/master 25660e9 Mario Dian: pass Consensus::Params& to ReceivedBlockTransactions()
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1428f3030d99...a6548a47a554
< gribble> https://github.com/bitcoin/bitcoin/issues/10199 | Better fee estimates by morcos · Pull Request #10199 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #10250: Fix some empty vector references (master...nonnullref) https://github.com/bitcoin/bitcoin/pull/10250
< bitcoin-git> bitcoin/master 1428f30 Wladimir J. van der Laan: Merge #10250: Fix some empty vector references...
< bitcoin-git> bitcoin/master f478d98 Pieter Wuille: Fix some empty vector references...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/5352e5e75da9...1428f3030d99
< stoner19> can anyone link me to the actual code for segwit in bitcoin core?
< bitcoin-git> [bitcoin] laanwj closed pull request #10223: Tests: Refactor to create witness script creation function (master...refactor_blocktools_for_segwit) https://github.com/bitcoin/bitcoin/pull/10223
< bitcoin-git> bitcoin/master 5352e5e Wladimir J. van der Laan: Merge #10223: Tests: Refactor to create witness script creation function...
< bitcoin-git> bitcoin/master c39a6b9 Jimmy Song: Tests: Refactor to create witness script creation function...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f3db4c601385...5352e5e75da9
< bitcoin-git> [bitcoin] laanwj closed pull request #10229: Tests: Add test for getdifficulty (master...test_getdifficulty) https://github.com/bitcoin/bitcoin/pull/10229
< bitcoin-git> bitcoin/master f3db4c6 Wladimir J. van der Laan: Merge #10229: Tests: Add test for getdifficulty...
< bitcoin-git> bitcoin/master 821dd5e Jimmy Song: Tests: Add test for getdifficulty...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/27faa6cccd8d...f3db4c601385
< bitcoin-git> [bitcoin] jonasschnelli opened pull request #10251: Add balances cache / GUI: use a signal instead of a poll thread (master...2017/04/gui_rm_locks) https://github.com/bitcoin/bitcoin/pull/10251
< jonasschnelli> I'm always surprised what kind of hack from the CoinControl implementation I find: https://github.com/bitcoin/bitcoin/blame/master/src/qt/walletmodel.cpp#L64
<@wumpus> but if it doesn't write bitcoin.conf, well, it's not too bad
< jonasschnelli> I once started a new GUI wizard with pruning: https://github.com/jonasschnelli/bitcoin/commit/7d8798d4942b7d7980a4c4f90cdd714432696ea7
< bitcoin-git> [bitcoin] sipa opened pull request #10250: Fix some empty vector references (master...nonnullref) https://github.com/bitcoin/bitcoin/pull/10250
< sipa> there is something crazy in this reasoning: bitcoin core is too annoying to use due to resource usage, thus people switch to electrum, but then complain they can't easily run their own server... which is even more expensive than just running a full node wallet in the first place
<@wumpus> of course there are more electrum users, that doesn't invalidate anything I said, most people running bitcoin core do not run an electrum server
< bitcoin-git> [bitcoin] sipa opened pull request #10249: Switch CCoinsMap from boost to std unordered_map (master...stdcoinmap) https://github.com/bitcoin/bitcoin/pull/10249
< bitcoin-git> [bitcoin] sipa opened pull request #10248: Introduce CHashVerifier to hash while deserializing (master...chashverifier) https://github.com/bitcoin/bitcoin/pull/10248
< gribble> https://github.com/bitcoin/bitcoin/issues/10244 | [qt] Add abstraction layer for accessing node and wallet functionality from gui by ryanofsky · Pull Request #10244 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #10228: build: regenerate bitcoin-config.h as necessary (master...fix-config-h) https://github.com/bitcoin/bitcoin/pull/10228
< bitcoin-git> bitcoin/master 27faa6c Wladimir J. van der Laan: Merge #10228: build: regenerate bitcoin-config.h as necessary...
< bitcoin-git> bitcoin/master 91ab8f5 Cory Fields: build: fix bitcoin-config.h regeneration after touching build files...
< bitcoin-git> bitcoin/master 3577603 Cory Fields: build: remove wonky auto top-level convenience targets...
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/f6f3b58a7250...27faa6cccd8d
< bitcoin-git> bitcoin/master f6f3b58 Wladimir J. van der Laan: Merge #10242: [qt] Don't call method on null WalletModel object...
< bitcoin-git> [bitcoin] laanwj closed pull request #10242: [qt] Don't call method on null WalletModel object (master...pr/rbfnull) https://github.com/bitcoin/bitcoin/pull/10242
< bitcoin-git> bitcoin/master fb463d1 Russell Yanofsky: [qt] Don't call method on null WalletModel object...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0416ea9f743f...f6f3b58a7250
< bitcoin-git> bitcoin/master 0416ea9 Wladimir J. van der Laan: Merge #10239: Make Boost use std::atomic internally...
< bitcoin-git> bitcoin/master 394ccf7 Pieter Wuille: Make Boost use std::atomic internally
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/694062eafec5...0416ea9f743f
< bitcoin-git> [bitcoin] laanwj closed pull request #10245: Minor fix in build documentation for FreeBSD 11 (master...freebsd-11-build-doc-fix) https://github.com/bitcoin/bitcoin/pull/10245
< bitcoin-git> bitcoin/master 694062e Wladimir J. van der Laan: Merge #10245: Minor fix in build documentation for FreeBSD 11...
< bitcoin-git> bitcoin/master 0611bc3 Shigeya Suzuki: Minor fix in build documentation for FreeBSD 11...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/86ea3c2ff247...694062eafec5
< luke-jr> https://github.com/bitcoin/bitcoin/pull/7107 should probably be reopened

2017-04-20

< bitcoin-git> [bitcoin] practicalswift closed pull request #10246: Silence "warning: "MSG_DONTWAIT" redefined" when compiling under Linux (master...silence-msg_dontwait-warning) https://github.com/bitcoin/bitcoin/pull/10246
< gribble> https://github.com/bitcoin/bitcoin/issues/8994 | Testchains: Introduce custom chain whose constructor... by jtimon · Pull Request #8994 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8855 | Use a proper factory for creating chainparams by jtimon · Pull Request #8855 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] practicalswift opened pull request #10246: Silence "warning: "MSG_DONTWAIT" redefined" when compiling under Linux (master...silence-msg_dontwait-warning) https://github.com/bitcoin/bitcoin/pull/10246
< bitcoin-git> [bitcoin] shigeya opened pull request #10245: Minor fix in build documentation for FreeBSD 11 (master...freebsd-11-build-doc-fix) https://github.com/bitcoin/bitcoin/pull/10245
< gribble> https://github.com/bitcoin/bitcoin/issues/10189 | devtools/net: add a verifier for scriptable changes. Use it to make CNode::id private. by theuni · Pull Request #10189 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9343 | Dont create change at dust limit by morcos · Pull Request #9343 · bitcoin/bitcoin · GitHub
< jtimon> wumpus: it seems to interfere with prevector templates in prevector_tests.cpp, see https://github.com/bitcoin/bitcoin/pull/10193/commits/cfef34884684e71c6f43ef3e4f2e87590fc87c9e for the trick to make the PR pass travis. probably I should remove that commit already, but I wanted to make sure the commits after removing BOOST_REVERSE_FOREACH weren't breaking something else, plus that commit is kind of the link that maybe answers your
< jtimon> wumpus: I'm trying to replace it with https://github.com/bitcoin/bitcoin/pull/10193/commits/1e59de57eecc1a61b036d7f89ff6bf918c2c7f67, which I found in the interwebs but although I thought I fully understood, it seems I don't
< jtimon> regarding https://github.com/bitcoin/bitcoin/pull/10193 I'm still failing at replacing BOOST_REVERSE_FOREACH and I don't understand why, maybe I should reduce the scope of the PR (I moved from working stuff to trying to fully remove boost/foreach.hpp by popular demand)
< bitcoin-git> [bitcoin] laanwj closed pull request #10181: Include cleanup (master...2017-04-10-includes) https://github.com/bitcoin/bitcoin/pull/10181
< bitcoin-git> bitcoin/master 86ea3c2 Wladimir J. van der Laan: Merge #10181: Include cleanup...
< bitcoin-git> bitcoin/master 1c897fc Jorge Timón: Missing includes
< bitcoin-git> bitcoin/master a1fd450 Jorge Timón: Trivial: Remove unneeded includes from .h:...
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/14c948987f0b...86ea3c2ff247
< gribble> https://github.com/bitcoin/bitcoin/issues/9343 | An error has occurred and has been logged. Please contact this bot's administrator for more information.
< gribble> https://github.com/bitcoin/bitcoin/issues/9343 | An error has occurred and has been logged. Please contact this bot's administrator for more information.
< bitcoin-git> [bitcoin] ryanofsky opened pull request #10244: [qt] Add abstraction layer for accessing node and wallet functionality from gui (master...pr/ipc-local) https://github.com/bitcoin/bitcoin/pull/10244