2016-11-12

< bitcoin-git> [bitcoin] luke-jr closed pull request #8388: [0.13] mining: Optimise for typical mining use with blockmaxsize (0.13...blockmaxsize_opti-0.13) https://github.com/bitcoin/bitcoin/pull/8388
< bitcoin-git> [bitcoin] luke-jr closed pull request #7534: Minimal RPC & wallet support for CLTV-enabled multisig addresses (master...cltv_address) https://github.com/bitcoin/bitcoin/pull/7534
< bitcoin-git> [bitcoin] luke-jr opened pull request #9140: Bugfix: Correctly replace generated headers and fail cleanly (master...bugfix_genheaders) https://github.com/bitcoin/bitcoin/pull/9140

2016-11-11

< bitcoin-git> [bitcoin] ryanofsky opened pull request #9139: Change sync_blocks to pick smarter maxheight (master...sync-height) https://github.com/bitcoin/bitcoin/pull/9139
< bitcoin-git> [bitcoin] morcos closed pull request #9123: Remove extraneous LogPrint from fee estimation (master...eliminateFeeWarning) https://github.com/bitcoin/bitcoin/pull/9123
< bitcoin-git> [bitcoin] morcos opened pull request #9138: Improve fee estimation (master...refactorFeeEstimation) https://github.com/bitcoin/bitcoin/pull/9138
< bitcoin-git> [bitcoin] ryanofsky opened pull request #9137: WIP: Allow wallet key import RPCs to track TxOut amounts on -prune nodes (master...import-pruned) https://github.com/bitcoin/bitcoin/pull/9137
< bitcoin-git> [bitcoin] ryanofsky opened pull request #9136: sync_blocks cleanup (master...sync-clean) https://github.com/bitcoin/bitcoin/pull/9136
< GoogleHater> My Bitcoin Core does not finding *.onion peers.
< GoogleHater> Why speed limit isn't implemented in Bitcoin Core?
< bitcoin-git> [bitcoin] morcos opened pull request #9133: Unset fImporting for loading mempool (master...noImportLoadMempool) https://github.com/bitcoin/bitcoin/pull/9133
< bitcoin-git> [bitcoin] jonasschnelli opened pull request #9131: fNetworkActive is not protected by a lock, use an atomic (master...2016/11/net_toggle) https://github.com/bitcoin/bitcoin/pull/9131
< gribble> https://github.com/bitcoin/bitcoin/issues/8456 | [RPC] Simplified bumpfee command. by mrbandrews · Pull Request #8456 · bitcoin/bitcoin · GitHub
< fanquake> One thing I wanted you to take a look at this this diff generated by your diff-tool -> https://github.com/bitcoin/bitcoin/pull/8808#issuecomment-259657066 , the bottom change seems to significant to be part of the PR.
< gribble> https://github.com/bitcoin/bitcoin/issues/9100 | tx_valid: re-order inputs to how they are encoded by dcousens · Pull Request #9100 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] paveljanik opened pull request #9130: Mention the new network toggle functionality in the tooltip. (master...20161111_disable_network_tooltip) https://github.com/bitcoin/bitcoin/pull/9130
< gribble> https://github.com/bitcoin/bitcoin/issues/9100 | tx_valid: re-order inputs to how they are encoded by dcousens · Pull Request #9100 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #9124: Use better name for local variable to prevent -Wshadow compiler warning (master...20161110_Wshadow_benchcheckblock) https://github.com/bitcoin/bitcoin/pull/9124
< bitcoin-git> bitcoin/master 87ab49e Wladimir J. van der Laan: Merge #9124: Use better name for local variable to prevent -Wshadow compiler warning...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/bfc7aad00884...87ab49e4fe38
< bitcoin-git> bitcoin/master bf49f10 Pavel Janík: Use better name for local variable to prevent -Wshadow compiler warning
< gribble> https://github.com/bitcoin/bitcoin/issues/9101 | [Wallet] Do not parse ssValue in CWalletDB::Recover by pstratem · Pull Request #9101 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9124 | Use better name for local variable to prevent -Wshadow compiler warning by paveljanik · Pull Request #9124 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8983 | WIP: Log block height and size when received by rebroad · Pull Request #8983 · bitcoin/bitcoin · GitHub
<@wumpus> ooh capitalizing Bitcoin just now that internet is no longer going to be capitalized :)
< gribble> https://github.com/bitcoin/bitcoin/issues/9064 | unify capitalization of "bitcoin address" by s-matthew-english · Pull Request #9064 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #9115: Mention reporting security issues responsibly (master...20161109_secissues) https://github.com/bitcoin/bitcoin/pull/9115
< bitcoin-git> bitcoin/master bfc7aad Wladimir J. van der Laan: Merge #9115: Mention reporting security issues responsibly...
< bitcoin-git> bitcoin/master 7d1de30 Pavel Janík: Mention reporting security issues responsibly
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ab914a65301b...bfc7aad00884
< gribble> https://github.com/bitcoin/bitcoin/issues/9115 | Mention reporting security issues responsibly by paveljanik · Pull Request #9115 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] luke-jr closed pull request #8889: Qt/ModalOverlay: Use theme tooltip colours (master...overlay_theme) https://github.com/bitcoin/bitcoin/pull/8889
< jonasschnelli> Luke-Jr: how do we continue with https://github.com/bitcoin/bitcoin/pull/8889? Close?
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #8996: Network activity toggle (master...networkactive) https://github.com/bitcoin/bitcoin/pull/8996
< bitcoin-git> bitcoin/master 32efa79 Jon Lund Steffensen: Qt: Add GUI feedback and control of network activity state....
< bitcoin-git> bitcoin/master e38993b Jon Lund Steffensen: RPC: Add "togglenetwork" method to toggle network activity temporarily...
< bitcoin-git> bitcoin/master 7c9a98a Jon Lund Steffensen: Allow network activity to be temporarily suspended....
< bitcoin-git> [bitcoin] jonasschnelli pushed 7 new commits to master: https://github.com/bitcoin/bitcoin/compare/7977a1157a3a...ab914a65301b
<@wumpus> (where "upstream" is the name of your remote for bitcoin/bitcoin, big chance it is "origin" for you)
< bitcoin-git> [bitcoin] laanwj closed pull request #9058: Fixes for p2p-compactblocks.py test timeouts on travis (#8842) (master...fix-8842-sync-timeouts) https://github.com/bitcoin/bitcoin/pull/9058
< bitcoin-git> bitcoin/master dac53b5 Russell Yanofsky: Modify getblocktxn handler not to drop requests for old blocks...
< bitcoin-git> bitcoin/master 55bfddc Russell Yanofsky: [qa] Fix stale data bug in test_compactblocks_not_at_tip...
< bitcoin-git> bitcoin/master 47e9659 Russell Yanofsky: [qa] Fix bug in compactblocks v2 merge...
< bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/46027e8668ec...7977a1157a3a
< fanquake> Have a look at the github-merge script in this dir https://github.com/bitcoin/bitcoin/tree/master/contrib/devtools
< bitcoin-git> [bitcoin] laanwj closed pull request #9113: Return the type when it's unknown instead of throwing an exception (master...ReturnNotThrow) https://github.com/bitcoin/bitcoin/pull/9113
< bitcoin-git> [bitcoin] laanwj closed pull request #9112: Avoid ugly exception in log on unknown inv type (master...2016_11_invtype_debugging) https://github.com/bitcoin/bitcoin/pull/9112
< bitcoin-git> bitcoin/master 46027e8 Wladimir J. van der Laan: Merge #9112: Avoid ugly exception in log on unknown inv type...
< bitcoin-git> bitcoin/master e9f25dd Wladimir J. van der Laan: Avoid ugly exception in log on unknown inv type...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/21e6c6b569c5...46027e8668ec
<@wumpus> phantomcircuit: jonasschnelli made such a tool https://github.com/bitcoin/bitcoin/pull/8745 - but yeah that's out of scope
< gribble> https://github.com/bitcoin/bitcoin/issues/9112 | Avoid ugly exception in log on unknown inv type by laanwj · Pull Request #9112 · bitcoin/bitcoin · GitHub
< jonasschnelli> For multiwallet, this PR is large but simple to review: https://github.com/bitcoin/bitcoin/pull/8764
<@wumpus> bumpfee needs to be on RPC first e.g. see https://github.com/bitcoin/bitcoin/pull/8456
< Victorsueca> wumpus: you mean that bitcoin core should have it's own random service like, for example, OpenPGP that uses other computationally intensive processes running on your system?
< bitcoin-git> [bitcoin] laanwj closed pull request #9114: [depends] Set OSX_MIN_VERSION to 10.8 (master...depends-min-osx-10-8) https://github.com/bitcoin/bitcoin/pull/9114
< bitcoin-git> bitcoin/master 21e6c6b Wladimir J. van der Laan: Merge #9114: [depends] Set OSX_MIN_VERSION to 10.8...
< bitcoin-git> bitcoin/master 617c96d fanquake: [depends] Set OSX_MIN_VERSION to 10.8
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/71bc39eb7483...21e6c6b569c5
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #9129: One fDisconnect to rule them all (master...OnefDisconnect) https://github.com/bitcoin/bitcoin/pull/9129
< bitcoin-git> [bitcoin] rebroad opened pull request #9129: One fDisconnect to rule them all (master...OnefDisconnect) https://github.com/bitcoin/bitcoin/pull/9129
< bitcoin-git> [bitcoin] theuni opened pull request #9128: net: Decouple CConnman and message serialization (master...connman-send) https://github.com/bitcoin/bitcoin/pull/9128
< bitcoin-git> [bitcoin] sipa opened pull request #9125: Make CBlock a vector of shared_ptr of CTransactions (master...sharedblock) https://github.com/bitcoin/bitcoin/pull/9125

2016-11-10

< sipa> you're free to ask questions about Bitcoin and Bitcoin Core's development model here
< Lauda> Oops wrong chat, thought this was #bitcoin. Sorry.
< MarcoFalke> FreeUser: The "Bitcoin Core Developers" is just the short version of all the credits (which you can find in the release notes)
< FreeUser> I can see "Bitcoin Core Developers" while loading Bitcoin Core.
< Chris_Stewart_5> Damn! I paid that guy all my doge coin for a bitcoin core dev sticker..
< gmaxwell> Perhaps you're a Bitcoin Core dev, have you submitted a patch? :)
< FreeUser> Official means from Bitcoin Core devs
< gmaxwell> whats a Bitcoin Core devs? there is no such thing as membership.
< sipa> Knots is a fork of Bitcoin Core, maintained by different people
< FreeUser> Is Bitcoin Knots an official wallet?
< FreeUser> How to disable alerts in old Bitcoin Core versions?
< bitcoin-git> [bitcoin] paveljanik opened pull request #9124: Use better name for local variable to prevent -Wshadow compiler warning (master...20161110_Wshadow_benchcheckblock) https://github.com/bitcoin/bitcoin/pull/9124
< gribble> https://github.com/bitcoin/bitcoin/issues/9058 | Fixes for p2p-compactblocks.py test timeouts on travis (#8842) by ryanofsky · Pull Request #9058 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9058 | Fixes for p2p-compactblocks.py test timeouts on travis (#8842) by ryanofsky · Pull Request #9058 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8550 | [Qt] Add interactive mempool graph by jonasschnelli · Pull Request #8550 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8501 | Add mempool statistics collector by jonasschnelli · Pull Request #8501 · bitcoin/bitcoin · GitHub
< jonasschnelli> petertodd: yes. 8501 can work without 8550 and has the most bitcoin-core wide impact.
< gribble> https://github.com/bitcoin/bitcoin/issues/8501 | Add mempool statistics collector by jonasschnelli · Pull Request #8501 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8501 | Add mempool statistics collector by jonasschnelli · Pull Request #8501 · bitcoin/bitcoin · GitHub
< jonasschnelli> BlueMatt: yeah! Bumpfee should get some review. Guys! https://github.com/bitcoin/bitcoin/pull/8456
< gribble> https://github.com/bitcoin/bitcoin/issues/8776 | Wallet refactoring leading up to multiwallet by luke-jr · Pull Request #8776 · bitcoin/bitcoin · GitHub
< jonasschnelli> So.. theres a project: https://github.com/bitcoin/bitcoin/projects/2
< bitcoin-git> [bitcoin] morcos opened pull request #9123: Remove extraneous LogPrint from fee estimation (master...eliminateFeeWarning) https://github.com/bitcoin/bitcoin/pull/9123
< NicolasDorier> well bitcoin core has a wallet part
< sipa> bitcoin core has a perfectly good wallet
< btcdrak> Yeah I also think purespv seems wrong for Bitcoin Core.
< BlueMatt> I'm not sure if its worth the effort to make bitcoin core a competitive spv wallet - they mostly all already support connecting to a trusted node (though need auth)
<@wumpus> being able to use bitcoin core as a full SPV node would be useful, especially with a local node that does validate
< BlueMatt> wumpus: because if you're not gonna be upgrading to full trust model why are you running bitcoin core as a wallet?
< jonasschnelli> For those who haven seen it, there a full working PR for the hybrid SPV: https://github.com/bitcoin/bitcoin/pull/9076
< sipa> ok, "client mode" as bitcoin's creator called it then
<@wumpus> I think the idea has always been to have some kind of SPV mode in bitcoin core, yes
< jonasschnelli> I wanted to ask if we'd like to see some form of "SPV" (wrong term, i agree) mode in Bitcoin-Core, if it's worth to continue the work
<@wumpus> #bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr btcdrak sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier jl2012
< gribble> https://github.com/bitcoin/bitcoin/issues/9075 | Decouple peer-processing-logic from block-connection-logic (#3) by TheBlueMatt · Pull Request #9075 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #9049: Remove duplicatable duplicate-input check from CheckTransaction (master...2016-10-bench-checkblock) https://github.com/bitcoin/bitcoin/pull/9049
< bitcoin-git> bitcoin/master e2b3fb3 Matt Corallo: Optimize vInOutPoints insertion a bit
< bitcoin-git> bitcoin/master eecffe5 Matt Corallo: Remove redundant duplicate-input check from CheckTransaction
< bitcoin-git> bitcoin/master b2e178a Matt Corallo: Add deserialize + CheckBlock benchmarks, and a full block hex
< bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/e5364991daec...71bc39eb7483
< jonasschnelli> Bip151 encrypted message packaging (https://github.com/bitcoin/bips/blob/master/bip-0151.mediawiki#encrypted-messages-structure) allows that. Because from the protocol perspective, combining them seems to have no downside
< bitcoin-git> bitcoin/master e536499 MarcoFalke: Merge #9097: [qa] Rework sync_* and preciousblock.py...
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #9097: [qa] Rework sync_* and preciousblock.py (master...Mf1611-qaSyncAndPrecious) https://github.com/bitcoin/bitcoin/pull/9097
< bitcoin-git> bitcoin/master fac1141 MarcoFalke: [qa] preciousblock: Use assert_equal and BitcoinTestFramework.__init__
< bitcoin-git> bitcoin/master fa97ccb MarcoFalke: [qa] util: Rework sync_*()...
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/4a2b170c075c...e5364991daec
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #9122: fix getnettotals RPC description about timemillis. (master...fix-getnettotals-rpc-desc) https://github.com/bitcoin/bitcoin/pull/9122
< bitcoin-git> bitcoin/master 4a2b170 MarcoFalke: Merge #9122: fix getnettotals RPC description about timemillis....
< bitcoin-git> bitcoin/master a79f864 Masahiko Hyuga: fix getnettotals RPC description about timemillis.
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/aab102cbae6b...4a2b170c075c
< bitcoin-git> [bitcoin] laanwj closed pull request #9121: Initialize variable to prevent compiler warning (master...20161110_rpcdump_uninitialized) https://github.com/bitcoin/bitcoin/pull/9121
< bitcoin-git> bitcoin/master aab102c Wladimir J. van der Laan: Merge #9121: Initialize variable to prevent compiler warning...
< bitcoin-git> bitcoin/master bdcba6d Pavel Janík: Initialize variable to prevent compiler warning
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/537e0cb252a3...aab102cbae6b
< bitcoin-git> [bitcoin] laanwj closed pull request #9120: bug: Missed one "return false" in recent refactoring in #9067 (master...fixExitCodesBitcoinTx) https://github.com/bitcoin/bitcoin/pull/9120
< bitcoin-git> bitcoin/master 537e0cb Wladimir J. van der Laan: Merge #9120: bug: Missed one "return false" in recent refactoring in #9067...
< bitcoin-git> bitcoin/master 45d372f UdjinM6: Missed one "return false" in recent refactoring in #9067
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/faec09bc7f03...537e0cb252a3
< bitcoin-git> [bitcoin] visvirial opened pull request #9122: fix getnettotals RPC description about timemillis. (master...fix-getnettotals-rpc-desc) https://github.com/bitcoin/bitcoin/pull/9122
< bitcoin-git> [bitcoin] paveljanik opened pull request #9121: Initialize variable to prevent compiler warning (master...20161110_rpcdump_uninitialized) https://github.com/bitcoin/bitcoin/pull/9121
< gribble> https://github.com/bitcoin/bitcoin/issues/8895 | Better SigCache Implementation by JeremyRubin · Pull Request #8895 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9075 | Decouple peer-processing-logic from block-connection-logic (#3) by TheBlueMatt · Pull Request #9075 · bitcoin/bitcoin · GitHub

2016-11-09

< gribble> https://github.com/bitcoin/bitcoin/issues/9058 | Fixes for p2p-compactblocks.py test timeouts on travis (#8842) by ryanofsky · Pull Request #9058 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] UdjinM6 opened pull request #9120: bug: Missed one "return false" in recent refactoring in #9067 (master...fixExitCodesBitcoinTx) https://github.com/bitcoin/bitcoin/pull/9120
< bitcoin-git> [bitcoin] laanwj closed pull request #9118: Remove requireGreater argment from TxConfirmStats::EstimateMedianVal() (master...remove-dead-code) https://github.com/bitcoin/bitcoin/pull/9118
< bitcoin-git> [bitcoin] laanwj closed pull request #9048: [0.13 backport] Fix handling of invalid compact blocks (0.13...fix-invalid-cb-ban-0.13) https://github.com/bitcoin/bitcoin/pull/9048
< bitcoin-git> [bitcoin] dooglus opened pull request #9118: Remove requireGreater argment from TxConfirmStats::EstimateMedianVal() (master...remove-dead-code) https://github.com/bitcoin/bitcoin/pull/9118
< bitcoin-git> bitcoin/master faec09b Wladimir J. van der Laan: Merge #8894: [Testing] Include fRelay in mininode version messages...
< bitcoin-git> bitcoin/master e5d682f jnewbery: Fix mininode version message format
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/fb156100f9b4...faec09bc7f03
< bitcoin-git> [bitcoin] laanwj closed pull request #9111: Remove unused variable UNLIKELY_PCT from fees.h (master...remove-unused-unlikely-pct) https://github.com/bitcoin/bitcoin/pull/9111
< bitcoin-git> bitcoin/master d8edf03 fanquake: Remove unused var UNLIKELY_PCT from fees.h
< bitcoin-git> bitcoin/master fb15610 Wladimir J. van der Laan: Merge #9111: Remove unused variable UNLIKELY_PCT from fees.h...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f445d886126c...fb156100f9b4
< bitcoin-git> bitcoin/master f445d88 Wladimir J. van der Laan: Revert "Check and enable -Wshadow by default."...
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/f445d886126c00814c71b855fc2f36fdd7e11098
<@wumpus> not just me *multiple people* tested it here: https://github.com/bitcoin/bitcoin/pull/8794
< gribble> https://github.com/bitcoin/bitcoin/issues/8808 | Do not shadow variables (gcc set) by paveljanik · Pull Request #8808 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8808 | Do not shadow variables (gcc set) by paveljanik · Pull Request #8808 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8808 | Do not shadow variables (gcc set) by paveljanik · Pull Request #8808 · bitcoin/bitcoin · GitHub
< Chris_Stewart_5> Is the CHECKSIG operation missing in BIP141 for P2WPKH & P2WPKH nested inside of P2SH? https://github.com/bitcoin/bips/blob/master/bip-0141.mediawiki#p2wpkh
< bitcoin-git> [bitcoin] theuni opened pull request #9117: net: don't send feefilter messages before the version handshake is complete (master...feefilter-assert) https://github.com/bitcoin/bitcoin/pull/9117
< MarcoFalke> I think I was hitting the assert in net.h https://github.com/bitcoin/bitcoin/blob/e9847303e708ab71b4d2c22ceb7d65c89615e18a/src/net.h#L772 ... I couldn't reproduce and debug.log only says ProcessMessages(version, 102 bytes) FAILED peer=10 http://pastebin.ubuntu.com/23451144/
< jonasschnelli> cfields: here I pass in Lambda https://github.com/bitcoin/bitcoin/pull/9076/files#diff-b2bb174788c7409b671c46ccc86034bdR3824 ... the function structure defines the parameter as std::function, does this always result in a copy for the lambda?
< bitcoin-git> [bitcoin] laanwj closed pull request #8794: Enable -Wshadow by default (master...20160922_Wshadow_enable) https://github.com/bitcoin/bitcoin/pull/8794
< bitcoin-git> bitcoin/master 359bac7 Pavel Janík: Add notes about variable names and shadowing
< bitcoin-git> bitcoin/master e0477f6 Wladimir J. van der Laan: Merge #8794: Enable -Wshadow by default...
< bitcoin-git> bitcoin/master fd5654c Pavel Janík: Check and enable -Wshadow by default.
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/e81df49644c2...e0477f6d2072
< bitcoin-git> [bitcoin] laanwj closed pull request #9039: Various serialization simplifcations and optimizations (master...simpleserial) https://github.com/bitcoin/bitcoin/pull/9039
< bitcoin-git> bitcoin/master fad9b66 Pieter Wuille: Make nType and nVersion private and sometimes const...
< bitcoin-git> bitcoin/master c2c5d42 Pieter Wuille: Make streams' read and write return void...
< bitcoin-git> bitcoin/master 50e8a9c Pieter Wuille: Remove unused ReadVersion and WriteVersion...
< bitcoin-git> [bitcoin] laanwj pushed 10 new commits to master: https://github.com/bitcoin/bitcoin/compare/e9847303e708...e81df49644c2
< bitcoin-git> [bitcoin] ondrejsika opened pull request #9116: Allow getblocktemlate for not connected regtest node (master...master) https://github.com/bitcoin/bitcoin/pull/9116
< bitcoin-git> [bitcoin] paveljanik opened pull request #9115: Mention reporting security issues responsibly (master...20161109_secissues) https://github.com/bitcoin/bitcoin/pull/9115
< bitcoin-git> [bitcoin] fanquake opened pull request #9114: [depends] Set OSX_MIN_VERSION to 10.8 (master...depends-min-osx-10-8) https://github.com/bitcoin/bitcoin/pull/9114
< bitcoin-git> [bitcoin] rebroad opened pull request #9113: Return the type when it's unknown instead of throwing an exception (master...ReturnNotThrow) https://github.com/bitcoin/bitcoin/pull/9113
< bitcoin-git> [bitcoin] laanwj opened pull request #9112: Avoid ugly exception in log on unknown inv type (master...2016_11_invtype_debugging) https://github.com/bitcoin/bitcoin/pull/9112
< bitcoin-git> bitcoin/master 1077577 Andrew Chow: Fix auto-deselection of peers
< bitcoin-git> bitcoin/master e984730 Wladimir J. van der Laan: Merge #8874: Multiple Selection for peer and ban tables...
< bitcoin-git> bitcoin/master addfdeb Andrew Chow: Multiple Selection for peer and ban tables...
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/924de0bd75a7...e9847303e708
< bitcoin-git> [bitcoin] fanquake opened pull request #9111: Remove unused variable UNLIKELY_PCT from fees.h (master...remove-unused-unlikely-pct) https://github.com/bitcoin/bitcoin/pull/9111
< gribble> https://github.com/bitcoin/bitcoin/issues/1625 | Bitcoin-Qt: exception in CInv::GetCommand() when using -onlynet="IPv6" · Issue #1625 · bitcoin/bitcoin · GitHub
< Victorsueca> sounds like bitcoin core fails some assertion in there
< gribble> https://github.com/bitcoin/bitcoin/issues/9110 | CInv::GetCommand(): type=1373179482 unknown type · Issue #9110 · bitcoin/bitcoin · GitHub
< rebroad> gmaxwell, wumpus, sipa, I think I may have found a bug that can crash bitcoin nodes
< gribble> https://github.com/bitcoin/bitcoin/issues/9087 | RPC: why not give more details when "generate" fails? by jtimon · Pull Request #9087 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9075 | Decouple peer-processing-logic from block-connection-logic (#3) by TheBlueMatt · Pull Request #9075 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9087 | RPC: why not give more details when "generate" fails? by jtimon · Pull Request #9087 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] TheBlueMatt closed pull request #9014: Fix block-connection performance regression (master...2016-10-fix-tx-regression) https://github.com/bitcoin/bitcoin/pull/9014

2016-11-08

< gribble> https://github.com/bitcoin/bitcoin/issues/9026 | Fix handling of invalid compact blocks by sdaftuar · Pull Request #9026 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] ryanofsky opened pull request #9108: WIP: Use importmulti timestamp when importing watch only keys (master...watchtime) https://github.com/bitcoin/bitcoin/pull/9108
< bitcoin-git> [bitcoin] morcos opened pull request #9107: Safer modify new coins (master...saferModifyNewCoins) https://github.com/bitcoin/bitcoin/pull/9107
< gribble> https://github.com/bitcoin/bitcoin/issues/9058 | Fixes for p2p-compactblocks.py test timeouts on travis (#8842) by ryanofsky · Pull Request #9058 · bitcoin/bitcoin · GitHub
< 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/9102 | An error has occurred and has been logged. Please contact this bot's administrator for more information.
< 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/9103 | An error has occurred and has been logged. Please contact this bot's administrator for more information.
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #9087: RPC: why not give more details when "generate" fails? (master...0.13-errors-rpc-mining) https://github.com/bitcoin/bitcoin/pull/9087
< bitcoin-git> bitcoin/master 924de0b MarcoFalke: Merge #9087: RPC: why not give more details when "generate" fails?...
< bitcoin-git> bitcoin/master 1adf82a Jorge Timón: RPC: Give more details when "generate" fails
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/4e5782438c3d...924de0bd75a7
< bitcoin-git> [bitcoin] laanwj closed pull request #9088: Reduce ambiguity of warning message (master...LessAmbiguousWalletWarning) https://github.com/bitcoin/bitcoin/pull/9088
< bitcoin-git> bitcoin/master 4e57824 Wladimir J. van der Laan: Merge #9088: Reduce ambiguity of warning message...
< bitcoin-git> bitcoin/master 77cbbd9 R E Broadley: Make warning message about wallet balance possibly...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/71ab476b0482...4e5782438c3d
< bitcoin-git> [bitcoin] laanwj closed pull request #9098: [qa] Handle zombies and cluttered tmpdirs (master...Mf1611-qaZombies) https://github.com/bitcoin/bitcoin/pull/9098
< bitcoin-git> bitcoin/master 71ab476 Wladimir J. van der Laan: Merge #9098: [qa] Handle zombies and cluttered tmpdirs...
< bitcoin-git> bitcoin/master fab0f07 MarcoFalke: [qa] rpc-tests: Apply random offset to portseed...
< bitcoin-git> bitcoin/master fae19aa MarcoFalke: [qa] test_framework: Exit when tmpdir exists
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/f53023dbb860...71ab476b0482
< bitcoin-git> [bitcoin] laanwj closed pull request #9067: Fix exit codes (master...fixExitCodesBitcoin) https://github.com/bitcoin/bitcoin/pull/9067
< bitcoin-git> bitcoin/master f53023d Wladimir J. van der Laan: Merge #9067: Fix exit codes...
< bitcoin-git> bitcoin/master 4441018 UdjinM6: Every main()/exit() should return/use one of EXIT_ codes instead of magic numbers
< bitcoin-git> bitcoin/master bd0de13 UdjinM6: Fix exit codes:...
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/b8f43e33316c...f53023dbb860
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #9095: test: Fix test_random includes (master...Mf1611-testRand) https://github.com/bitcoin/bitcoin/pull/9095
< bitcoin-git> bitcoin/master b8f43e3 Jonas Schnelli: Merge #9095: test: Fix test_random includes...
< bitcoin-git> bitcoin/master fa8278e MarcoFalke: test: Fix test_random includes
< bitcoin-git> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/dc6b9406bdfa...b8f43e33316c
< bitcoin-git> bitcoin/master d4833ff Suhas Daftuar: Bump the protocol version to distinguish new banning behavior....
< bitcoin-git> [bitcoin] sipa closed pull request #9026: Fix handling of invalid compact blocks (master...fix-invalidcb-handling) https://github.com/bitcoin/bitcoin/pull/9026
< bitcoin-git> bitcoin/master 88c3549 Suhas Daftuar: Fix compact block handling to not ban if block is invalid
< bitcoin-git> bitcoin/master c93beac Suhas Daftuar: [qa] Test that invalid compactblocks don't result in ban
< bitcoin-git> [bitcoin] sipa pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/9f554e03ebe5...dc6b9406bdfa
< bitcoin-git> [bitcoin] instagibbs opened pull request #9102: Really don't validate genesis block (master...dontvalidategenesis) https://github.com/bitcoin/bitcoin/pull/9102
< bitcoin-git> [bitcoin] pstratem opened pull request #9101: [Wallet] Do not parse ssValue in CWalletDB::Recover (master...2016-11-07-cwalletdb-recover) https://github.com/bitcoin/bitcoin/pull/9101
< bitcoin-git> [bitcoin] dcousens opened pull request #9100: tx_valid: re-order inputs to how they are encoded (master...testorder) https://github.com/bitcoin/bitcoin/pull/9100

2016-11-07

< gribble> https://github.com/bitcoin/bitcoin/issues/8831 | Replace CWalletDB::ReadKeyValue with CWallet::LoadKeyValue by pstratem · Pull Request #8831 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] sipa closed pull request #9045: Hash P2P messages as they are received instead of at process-time (master...2016-10-p2p-hash) https://github.com/bitcoin/bitcoin/pull/9045
< bitcoin-git> bitcoin/master 9f554e0 Pieter Wuille: Merge #9045: Hash P2P messages as they are received instead of at process-time...
< bitcoin-git> bitcoin/master fe1dc62 Matt Corallo: Hash P2P messages as they are received instead of at process-time
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1253f8692fc3...9f554e03ebe5
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #9098: [qa] Handle zombies and cluttered tmpdirs (master...Mf1611-qaZombies) https://github.com/bitcoin/bitcoin/pull/9098
< satosh-777-xl_> I asked this on bitcoin-dev and was told to check the developer notes and then post on here if I couldn't find anything.. Anyways as core is moving away from boost and BOOST_FOREACH loops in favor of standard C++ (11), is the following for loop syntax acceptable?
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #9097: [qa] Rework sync_* and preciousblock.py (master...Mf1611-qaSyncAndPrecious) https://github.com/bitcoin/bitcoin/pull/9097
< gribble> https://github.com/bitcoin/bitcoin/issues/8708 | net: have CConnman handle message sending by theuni · Pull Request #8708 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #9077: [qa] Increase wallet-dump RPC timeout (master...fix-wallet-dump-timeout) https://github.com/bitcoin/bitcoin/pull/9077
< bitcoin-git> bitcoin/master 1253f86 MarcoFalke: Merge #9077: [qa] Increase wallet-dump RPC timeout...
< bitcoin-git> bitcoin/master 8463aaa Russell Yanofsky: [qa] Increase wallet-dump RPC timeout...
< bitcoin-git> bitcoin/master e89614b Russell Yanofsky: [qa] Add more helpful RPC timeout message...
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/78cdd643d317...1253f8692fc3
< gribble> https://github.com/bitcoin/bitcoin/issues/9095 | test: Fix test_random includes by MarcoFalke · Pull Request #9095 · bitcoin/bitcoin · GitHub
< MarcoFalke> Did anyone receive a email for the edit of https://github.com/bitcoin/bitcoin/releases/tag/v0.13.1?
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #9095: test: Fix test_random includes (master...Mf1611-testRand) https://github.com/bitcoin/bitcoin/pull/9095
< jonasschnelli> MarcoFalke: hmm... rm inputs/bitcoin and do a fresh local heckout?
< jonasschnelli> MarcoFalke: the first errored build was on 19th of october https://bitcoin.jonasschnelli.ch/nightlybuilds/2016-10-19/
< bitcoin-git> [bitcoin] jonasschnelli closed pull request #9094: qt: Use correct conversion function for boost::path datadir (master...2016_11_datadir_in_console) https://github.com/bitcoin/bitcoin/pull/9094
< bitcoin-git> bitcoin/master 78cdd64 Jonas Schnelli: Merge #9094: qt: Use correct conversion function for boost::path datadir...
< bitcoin-git> bitcoin/master e760b30 Wladimir J. van der Laan: qt: Use correct conversion function for boost::path datadir...
< bitcoin-git> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0b2322b144a0...78cdd643d317
< gribble> https://github.com/bitcoin/bitcoin/issues/8981 | Wshadow: Do not shadow argument with a local variable by paveljanik · Pull Request #8981 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #8981: Wshadow: Do not shadow argument with a local variable (master...20161020_Wshadow_rpcdump) https://github.com/bitcoin/bitcoin/pull/8981
< gribble> https://github.com/bitcoin/bitcoin/issues/8981 | Wshadow: Do not shadow argument with a local variable by paveljanik · Pull Request #8981 · bitcoin/bitcoin · GitHub
< bitcoin-git> bitcoin/master 0b2322b Wladimir J. van der Laan: Merge #8981: Wshadow: Do not shadow argument with a local variable...
< bitcoin-git> bitcoin/master ff6639b Pavel Janík: Do not shadow local variable
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8c6218a28ae8...0b2322b144a0
< gribble> https://github.com/bitcoin/bitcoin/issues/8976 | libconsensus: Add input validation of flags by laanwj · Pull Request #8976 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #8976: libconsensus: Add input validation of flags (master...2016_10_bitcoinconsensus_input_checking) https://github.com/bitcoin/bitcoin/pull/8976
< gribble> https://github.com/bitcoin/bitcoin/issues/8976 | libconsensus: Add input validation of flags by laanwj · Pull Request #8976 · bitcoin/bitcoin · GitHub
< bitcoin-git> bitcoin/master 8c6218a Wladimir J. van der Laan: Merge #8976: libconsensus: Add input validation of flags...
< bitcoin-git> bitcoin/master 5ca8ef2 Wladimir J. van der Laan: libconsensus: Add input validation of flags...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/3c03dc2cfc07...8c6218a28ae8
< gribble> https://github.com/bitcoin/bitcoin/issues/7730 | Remove priority estimation by morcos · Pull Request #7730 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #7730: Remove priority estimation (master...removePriEst) https://github.com/bitcoin/bitcoin/pull/7730
< gribble> https://github.com/bitcoin/bitcoin/issues/7730 | Remove priority estimation by morcos · Pull Request #7730 · bitcoin/bitcoin · GitHub
< bitcoin-git> bitcoin/master 3c03dc2 Wladimir J. van der Laan: Merge #7730: Remove priority estimation...
< bitcoin-git> bitcoin/master 0bd581a Alex Morcos: add release notes for removal of priority estimation
< bitcoin-git> bitcoin/master b2322e0 Alex Morcos: Remove priority estimation
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/1e50d22ed2df...3c03dc2cfc07
<@wumpus> can whoever is in charge of gribble please make it ignore user 'bitcoin-git'?
< gribble> https://github.com/bitcoin/bitcoin/issues/8709 | Allow filterclear messages for enabling TX relay only. by rebroad · Pull Request #8709 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #8709: Allow filterclear messages for enabling TX relay only. (master...AllowFilterclear) https://github.com/bitcoin/bitcoin/pull/8709
< gribble> https://github.com/bitcoin/bitcoin/issues/8709 | Allow filterclear messages for enabling TX relay only. by rebroad · Pull Request #8709 · bitcoin/bitcoin · GitHub
< bitcoin-git> bitcoin/master 1e50d22 Wladimir J. van der Laan: Merge #8709: Allow filterclear messages for enabling TX relay only....
< bitcoin-git> bitcoin/master 1f951c6 R E Broadley: Allow filterclear messages for enabling TX relay only....
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c113a651f1f5...1e50d22ed2df
< gribble> https://github.com/bitcoin/bitcoin/issues/9052 | Use RelevantServices instead of node_network in AttemptToEvict. by gmaxwell · Pull Request #9052 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #9052: Use RelevantServices instead of node_network in AttemptToEvict. (master...prefer_relevant2) https://github.com/bitcoin/bitcoin/pull/9052
< gribble> https://github.com/bitcoin/bitcoin/issues/9052 | Use RelevantServices instead of node_network in AttemptToEvict. by gmaxwell · Pull Request #9052 · bitcoin/bitcoin · GitHub
< bitcoin-git> bitcoin/master c113a65 Wladimir J. van der Laan: Merge #9052: Use RelevantServices instead of node_network in AttemptToEvict....
< bitcoin-git> bitcoin/master d32036a Gregory Maxwell: Use RelevantServices instead of node_network in AttemptToEvict....
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/078900df75f1...c113a651f1f5
< gribble> https://github.com/bitcoin/bitcoin/issues/8568 | new var DIST_CONTRIB adds useful things for packagers from contrib by nomnombtc · Pull Request #8568 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #8568: new var DIST_CONTRIB adds useful things for packagers from contrib (master...DIST_CONTRIB) https://github.com/bitcoin/bitcoin/pull/8568
< GitHub126> bitcoin/master 078900d Wladimir J. van der Laan: Merge #8568: new var DIST_CONTRIB adds useful things for packagers from contrib...
< GitHub126> bitcoin/master 1ee6f91 nomnombtc: new var DIST_CONTRIB adds useful things for packagers from contrib/ to EXTRA_DIST
< GitHub126> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2fae5b93468c...078900df75f1
< GitHub31> [bitcoin] laanwj closed pull request #9093: [doc] release-process: Mention GitHub release and archived release notes (master...Mf1611-docRel) https://github.com/bitcoin/bitcoin/pull/9093
< GitHub3> bitcoin/master 2fae5b9 Wladimir J. van der Laan: Merge #9093: [doc] release-process: Mention GitHub release and archived release notes...
< GitHub3> bitcoin/master faead5e MarcoFalke: [doc] release-process: Mention GitHub release and archived release notes
< GitHub3> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2b799ae9e1e0...2fae5b93468c
< GitHub23> bitcoin/master 2b799ae MarcoFalke: Merge #9083: Enforcing consistency, 'gitian' to 'Gitian'...
< GitHub179> [bitcoin] MarcoFalke closed pull request #9083: Enforcing consistency, 'gitian' to 'Gitian' (master...patch-9) https://github.com/bitcoin/bitcoin/pull/9083
< GitHub23> bitcoin/master 66ca6cd S. Matthew English: Enforcing consistency, 'gitian' to 'Gitian'...
< GitHub23> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/44f2df613f23...2b799ae9e1e0
< GitHub181> bitcoin/master 44f2df6 Wladimir J. van der Laan: Merge #8675: Make copyright header lines uniform...
< GitHub151> [bitcoin] laanwj closed pull request #8675: Make copyright header lines uniform (master...copyright-made-uniform) https://github.com/bitcoin/bitcoin/pull/8675
< GitHub181> bitcoin/master 4b04e32 isle2983: [copyright] copyright header style uniform...
< GitHub181> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/5fa7b07565d2...44f2df613f23
< GitHub154> [bitcoin] laanwj closed pull request #8158: Simplify calls to retrieve credit and balance (master...enhancement/unification-wallet-balance) https://github.com/bitcoin/bitcoin/pull/8158
< btcdrak> luke-jr: I think Andreas has a point https://github.com/bitcoin/bips/pull/472#issuecomment-258794458
< GitHub10> [bitcoin] laanwj opened pull request #9094: qt: Use correct conversion function for boost::path datadir (master...2016_11_datadir_in_console) https://github.com/bitcoin/bitcoin/pull/9094
< GitHub141> [bitcoin] laanwj closed pull request #8736: base58: Improve DecodeBase58 performance. (master...speedup-decodebase58) https://github.com/bitcoin/bitcoin/pull/8736