< bitcoin-git> [bitcoin] sipa pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/fbf5d3ba1516...1ad3d4e1261f
< bitcoin-git> bitcoin/master a5410ac Jorge Timón: Small preparations for Q_FOREACH, PAIRTYPE and #include <boost/foreach.hpp> removal
< bitcoin-git> bitcoin/master 7c00c26 Jorge Timón: scripted-diff: Fully remove BOOST_FOREACH...
< bitcoin-git> bitcoin/master 18dc3c3 Jorge Timón: scripted-diff: Remove Q_FOREACH...
< bitcoin-git> [bitcoin] sipa closed pull request #10502: scripted-diff: Remove BOOST_FOREACH, Q_FOREACH and PAIRTYPE (master...b15-boost-foreach) https://github.com/bitcoin/bitcoin/pull/10502
< kvnn> gmaxwell : thank you.
< jtimon> \open_poll Would you (A) or (B) #9176 once and for all? It's never costing rebase time but potentially review time just by being open: A: CLOSE, B: MERGE
< gribble> https://github.com/bitcoin/bitcoin/issues/9176 | Globals: Pass Consensus::Params through CBlockTreeDB::LoadBlockIndexGuts() by jtimon · Pull Request #9176 · bitcoin/bitcoin · GitHub
< gmaxwell> uh. what is up with #10584
< gribble> https://github.com/bitcoin/bitcoin/issues/10584 | Remove unused ResendWalletTransactions notification by ryanofsky · Pull Request #10584 · bitcoin/bitcoin · GitHub
< phantomcircuit> gmaxwell, i think it's just wrong?
< sipa> it looks to me he's removing only unused code
< sipa> which it shouldn't
< sipa> be
< sipa> but maybe i miss something
< gmaxwell> sipa: hm? it's hooked up to the Broadcast signal, which is invoked from SendMessages.
< sipa> oh
< gmaxwell> have I said before that I do not slots and signals they and overhead and _obscure control flow_? :P
< sipa> gmaxwell: well i'm confused by ryanofsky thinks it's unused
< gmaxwell> maybe there is some pattern used in the codebase which makes the control flow less clear to him? :P (I thought we had tests though for wallet retransmission, but seems the tests are passing. :( )
< jonasschnelli> Anyone up for a quick wallet PR review with great positive performance impacts: #10251
< gribble> https://github.com/bitcoin/bitcoin/issues/10251 | Add balances cache / GUI: use a signal instead of a poll thread by jonasschnelli · Pull Request #10251 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #10584: Remove unused ResendWalletTransactions notification (master...pr/noresend) https://github.com/bitcoin/bitcoin/pull/10584
< bitcoin-git> [bitcoin] kallewoof opened pull request #10585: [WIP] PoW connection slots (master...pow-connection-slots) https://github.com/bitcoin/bitcoin/pull/10585
< bitcoin-git> [bitcoin] NicolasDorier closed pull request #9991: listreceivedbyaddress Filter Address (master...listreceivedbyaddress-filtered) https://github.com/bitcoin/bitcoin/pull/9991
< wumpus> any problems with 0.14.2rc2 reported yet? anyone have anything to add to the release notes? https://github.com/bitcoin/bitcoin/blob/0.14/doc/release-notes.md
< wumpus> if not, it may be time to cut the release
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1ad3d4e1261f...b63be2c6852f
< bitcoin-git> bitcoin/master cb24c85 Pieter Wuille: Use rdrand as entropy source on supported platforms
< bitcoin-git> bitcoin/master b63be2c Wladimir J. van der Laan: Merge #10377: Use rdrand as entropy source on supported platforms...
< bitcoin-git> [bitcoin] laanwj closed pull request #10377: Use rdrand as entropy source on supported platforms (master...hwrand) https://github.com/bitcoin/bitcoin/pull/10377
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b63be2c6852f...6702617c868a
< bitcoin-git> bitcoin/master ec98b78 Andrew Chow: Tests for wallet encryption stuff...
< bitcoin-git> bitcoin/master 6702617 Wladimir J. van der Laan: Merge #10551: [Tests] Wallet encryption functional tests...
< bitcoin-git> [bitcoin] laanwj closed pull request #10551: [Tests] Wallet encryption functional tests (master...wallet-encrypt-test) https://github.com/bitcoin/bitcoin/pull/10551
< bitcoin-git> [bitcoin] laanwj closed pull request #10081: Add 'dumpbootstrap' RPC for creating bootstrap.dat files. (master...dumpbootstrap) https://github.com/bitcoin/bitcoin/pull/10081
< ryanofsky> re: "well i'm confused by ryanofsky thinks it's unused" just a mistake, my brain wasn't working, and the naming for that signal doesn't follow the normal pattern
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/6702617c868a...c94b89e90d6a
< bitcoin-git> bitcoin/master 6294f32 John Newbery: gettxoutproof() should return consistent result...
< bitcoin-git> bitcoin/master c94b89e Wladimir J. van der Laan: Merge #9738: gettxoutproof() should return consistent result...
< bitcoin-git> [bitcoin] laanwj closed pull request #9738: gettxoutproof() should return consistent result (master...fixgettxoutproof) https://github.com/bitcoin/bitcoin/pull/9738
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c94b89e90d6a...228c319a944b
< bitcoin-git> bitcoin/master 1d1ea9f Marko Bencun: Turn TryCreateDirectory() into TryCreateDirectories()...
< bitcoin-git> bitcoin/master 228c319 Wladimir J. van der Laan: Merge #9895: Turn TryCreateDirectory() into TryCreateDirectories()...
< bitcoin-git> [bitcoin] laanwj closed pull request #9895: Turn TryCreateDirectory() into TryCreateDirectories() (master...appinitmain) https://github.com/bitcoin/bitcoin/pull/9895
< bitcoin-git> [bitcoin] morcos opened pull request #10586: More economical fee estimates for opt-in-RBF transactions (master...aggressiveEstimates) https://github.com/bitcoin/bitcoin/pull/10586
< morcos> sipa: Ruling requested
< morcos> What is the style guide for an rpc named argument?
< morcos> conf_target?
< morcos> we already have a confTarget inside the options for bumpfee, but its inside the options, and it seems most named argumetns aren't camelCase
< sipa> from developer-notes.md, under RPC:
< sipa> Argument naming: use snake case fee_delta (and not, e.g. camel case feeDelta)
< sipa> Rationale: Consistency with existing interface.
< bitcoin-git> [bitcoin] practicalswift opened pull request #10587: Net: Fix resource leak in ReadBinaryFile(...) (master...fopen-not-followed-by-fclose-in-all-states-of-the-universe) https://github.com/bitcoin/bitcoin/pull/10587
< morcos> sipa: oops, sorry i didn't read
< cfields> jtimon: ping
< jtimon> cfields: pong
< cfields> jtimon: I'm confused. At one point, you had a version of the reverse_iterator that compiled, but failed tests. no?
< jtimon> yes, and then I followed your advice for the prevector tests, which was the part that didn't compile unless I commented those lines (but if I did comment them, then the tests failed as they should)
< cfields> jtimon: can you point me to a failing revision?
< cfields> jtimon: oooooh
< cfields> jtimon: the tests failed because some things were commented out, not because of new breakage?
< jtimon> right the breakage was a compile error
< jtimon> sorry, I shouldn't have commented the lines, and should had left the compile error there, I just wanted to see what else would fail
< cfields> jtimon: ok, i misunderstood, then
< cfields> jtimon: i wrote some quick tests locally, and everything passed. So I'm not so concerned about the scope issue anymore
< cfields> jtimon: compile fix is here: https://pastebin.com/raw/2zscDfbq
< jtimon> cfields: oh, thanks! I tried to do the reverse iterator more const, but not the prevector!
< bitcoin-git> [bitcoin] morcos opened pull request #10588: Note preexisting bug in display of fee calculation in coin control (0.14...notebug) https://github.com/bitcoin/bitcoin/pull/10588
< instagibbs> anyone know the reasoning for the "keyword definition" stuff in importmulti.py? A couple of those strings don't even occur, and none of the variables are ever used.
< bitcoin-git> [bitcoin] morcos opened pull request #10589: Add RPC options for RBF, confirmation target and conservative fee estimates (master...rpcestimatechoice) https://github.com/bitcoin/bitcoin/pull/10589
< instagibbs> morcos, really wish we could get some basic effective value logic in so we could easily improve bumpfee as well.
< morcos> how does that help with improving bumpfee?
< instagibbs> Well, I guess we could just lamely loop and keep increasing until it grabs enough effective value...
< instagibbs> the sickness spreading :P
< instagibbs> right now bumpfee must have a change output to be successful
< morcos> instagibbs: yes but i don't understand how thats related to the effective value logic?
< instagibbs> if we had effective value logic, it would be easier? Sorry I'm missing what you're missing.
< morcos> i thought the reason bumpfee had to have change had something to do with the complication of adding new inputs which may mean there are multiple txs being replaced
< instagibbs> ah, maybe that's a reason
< morcos> i dont' think the requirement that we had change to reduce in order to bump fee had anything to do with our stupid coin selection
< morcos> in any case, i'm all for improving coin selection.
< instagibbs> why would adding confirmed inputs replace multiple txs?
< instagibbs> maybe I should go hunt down the bumpfee PR, find the undocumented assumptions
< morcos> but limited cycles.. i owe sipa and bluematt tons of review too, but i'm about to be away from work for a bit, so i just wanted to push out the fee changes i think we need for 0.15
< instagibbs> Understood
< morcos> instagibbs: i don't remember exactly now, but i thin kthe idea was just to limit it to the simplest possible case for starters
< morcos> not that we can't do other things , but just to get the first round in
< instagibbs> I totally agree, which is why I was thinking about coin selection in that respect.
< instagibbs> anyways I'll revisit it post-0.15
< sipa> instagibbs, morcos: achow101 and i are looking at murch's branch&bound algorithm for coin selection
< instagibbs> great. We might not care about exact matches when hitting `bumpfee`, since we may want followup change
< instagibbs> without linking a bunch of inputs together greedily
< murch> @sipa: What are you guys doing this evening?
< murch> I assume you have found the branch in my github repo?
< sipa> murch: yup
< sipa> see pm
< bitcoin-git> [bitcoin] jnewbery opened pull request #10591: [tests] make pruning.py faster (master...fastprune) https://github.com/bitcoin/bitcoin/pull/10591
< bitcoin-git> [bitcoin] jnewbery opened pull request #10592: [trivial] fix indentation for ArgsManager class (master...argsmanager) https://github.com/bitcoin/bitcoin/pull/10592
< bitcoin-git> [bitcoin] luke-jr closed pull request #10512: Rework same-chain from abusing DoS banning, to explicit checks (master...samechain_rework) https://github.com/bitcoin/bitcoin/pull/10512
< bitcoin-git> [bitcoin] luke-jr opened pull request #10593: Relax punishment for peers relaying invalid blocks and headers (master...relax_invblk_punishment) https://github.com/bitcoin/bitcoin/pull/10593
< gmaxwell> instagibbs: I believe the reason for fixing the inputs was just to simplify the first implementation.
< bitcoin-git> [bitcoin] luke-jr opened pull request #10594: Bugfix: net: Apply whitelisting criteria to outgoing connections (master...whitelist_outgoing) https://github.com/bitcoin/bitcoin/pull/10594
< gmaxwell> Also, it's preferable to not use extra inputs if you can avoid it, because you'll burn down your unspent outputs available for other transactions. (keep in mind you really don't want to spend the bump change, since you can't be sure which version will confirm)