< promag> off topic, but I see lots of different conventions for commit messages, branch names and PR titles. should there be such conventions in developer notes for instance?
< sipa> cfields: works fine here
< cfields> :(
< sipa> cfields: in other news, your compiler is retarded
< sipa> 0000000000000011movq%rsi, -0x38(%rbp)
< sipa> 000000000000001dmovq-0x38(%rbp), %rsi
< cfields> erm, that just moves and moves back?
< sipa> yes
< cfields> heh
< cfields> fulfilling a constraint, maybe?
< sipa> nope
< sipa> it could leave everything in the original order even
< cfields> sipa: weird. based on my dinky little test app, it seems to be working
< sipa> cfields: eh
< cfields> sipa: not sure yet. confused.
< cfields> i wrote a little sha256sum clone that just reads stdin and spits out a hash. matches sha256sum so far.
< sipa> cfields: does it use the sse code?
< cfields> Assertion failed: (0), function Transform, file crypto/sha256_sse42.cpp, line 16.
< cfields> yep. just threw ^^ assert in to verify.
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b7d6623c76e1...f90603ac6d24
< bitcoin-git> bitcoin/master 3babbcb Gregory Maxwell: Remove confusing MAX_BLOCK_BASE_SIZE....
< bitcoin-git> bitcoin/master f90603a Pieter Wuille: Merge #10618: Remove confusing MAX_BLOCK_BASE_SIZE....
< bitcoin-git> [bitcoin] sipa closed pull request #10618: Remove confusing MAX_BLOCK_BASE_SIZE. (master...size_b_gone) https://github.com/bitcoin/bitcoin/pull/10618
< sipa> BlueMatt: can you fix the typo in the commit title of #10808?
< gribble> https://github.com/bitcoin/bitcoin/issues/10808 | Avoid some new gcc warnings in 15 by TheBlueMatt · Pull Request #10808 · bitcoin/bitcoin · GitHub
< BlueMatt> heh, oops
< BlueMatt> fixed, sipa
< sipa> cfields: so it seems that the SSE PR compiles fine on travis, but no tests get run...
< cfields> sipa: right. we'll need to find some other osx sucker
< sipa> cfields: you're just doing a straightforward clean build of the PR?
< cfields> sipa: yep
< cfields> sipa: did a cross build via depends too with the same result
< sipa> :(
< bitcoin-git> [bitcoin] sipa pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/f90603ac6d24...ec8a50b8d786
< bitcoin-git> bitcoin/master 1016dac Matt Corallo: Fix signed/unsigned comparison warning introduced in c8e29d7ff0.
< bitcoin-git> bitcoin/master c73b8be Matt Corallo: Explicitly initialize prevector::_union to avoid new warning...
< bitcoin-git> bitcoin/master ec8a50b Pieter Wuille: Merge #10808: Avoid some new gcc warnings in 15...
< bitcoin-git> [bitcoin] sipa closed pull request #10808: Avoid some new gcc warnings in 15 (master...2017-07-15-new-warnings) https://github.com/bitcoin/bitcoin/pull/10808
< bitcoin-git> [bitcoin] gmaxwell opened pull request #10831: Batch flushing operations to the walletdb during top up and increase keypool size. (master...topup_batch_flush) https://github.com/bitcoin/bitcoin/pull/10831
< sipa> BlueMatt: i am confused by the change in #10235
< gribble> https://github.com/bitcoin/bitcoin/issues/10235 | Track keypool entries as internal vs external in memory by TheBlueMatt · Pull Request #10235 · bitcoin/bitcoin · GitHub
< sipa> LogPrintf("keypool added key %d, size=%u (%u internal), internal=%d\n", nEnd, setInternalKeyPool.size() + setExternalKeyPool.size(), setInternalKeyPool.size(), internal);
< sipa> 2x internal?
< sipa> oh, one is a bool
< BlueMatt> oh ffs, ok, i'll fix, sec
< BlueMatt> fixed
< promag> What about a wip tag in GitHub?
< BlueMatt> ;;later tell promag problem with github tags is you cant have regular folks adding tags (ie you cant limit tag-edits to people who own the issue/pr)
< gribble> The operation succeeded.
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ec8a50b8d786...1c011ff43010
< bitcoin-git> bitcoin/master dd97a52 Alex Morcos: Properly forbid -salvagewallet and -zapwallettxes for multi wallet.
< bitcoin-git> bitcoin/master 1c011ff Pieter Wuille: Merge #10816: Properly forbid -salvagewallet and -zapwallettxes for multi wallet....
< bitcoin-git> [bitcoin] sipa closed pull request #10816: Properly forbid -salvagewallet and -zapwallettxes for multi wallet. (master...multiwallet_parameters) https://github.com/bitcoin/bitcoin/pull/10816
< gmaxwell> Is there anyone who uses OSX around? #10821 needs testing for sipa.
< gribble> https://github.com/bitcoin/bitcoin/issues/10821 | [WIP] Add SSE 4.2 optimized SHA256 by sipa · Pull Request #10821 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] sipa pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/1c011ff43010...8fdd23a224ba
< bitcoin-git> bitcoin/master bc1be90 Jonas Schnelli: [Qt] replace fee slider with a Dropdown, extend conf. targets
< bitcoin-git> bitcoin/master 2aef1f1 Jonas Schnelli: [Qt] migrate old fee slider value to new dropbown...
< bitcoin-git> bitcoin/master 8fdd23a Pieter Wuille: Merge #10769: [Qt] replace fee slider with a Dropdown, extend conf. targets...
< bitcoin-git> [bitcoin] sipa closed pull request #10769: [Qt] replace fee slider with a Dropdown, extend conf. targets (master...2017/07/qt_fee_slider) https://github.com/bitcoin/bitcoin/pull/10769
< cfields> sipa: still around? I managed to piss clang off on linux
< cfields> not sure if it's the same issue or if I forced another
< fanquake> gmaxwell cfields what do you want tested? Merging 10821 now.
< cfields> fanquake: "./bitcoind" is enough :)
< cfields> for me it segfaults on the genesis block check
< fanquake> cfields Indeed Assertion failed: (consensus.hashGenesisBlock etc
< cfields> fanquake: ok, so i'm not alone. great. also, shit.
< cfields> fanquake: thanks for testing!
< fanquake> cfields no worries. Ping me if you want anything looked at. Should be around for the rest of the afternoon.
< sipa> cfields: how did you make clang piss off in linux?
< sipa> can you create and a send me a static binary?
< cfields> sipa: still fiddling with it. I'm having troubles with boost as well.
< sipa> cfields: let's have a look at it in tokyo
< sipa> i can try a static register allocation first too, and see if that works
< sipa> why am i talking here? i'm getting drunk
< cfields> haha. are you irc'ing at a bar?
< sipa> yes
< cfields> sipa: using clang with -fintegrated-as caused a new crash in linux
< cfields> my assumption is that it uses gnu as on linux, and llvm's as for apple
< cfields> but i'll come up with something solid and ping you then. forget about assembly for the night :)
< cfields> heh, confirmed
< cfields> unfortunately, it crashes gdb too :(
< bitcoin-git> [bitcoin] laanwj opened pull request #10832: init: Factor out AppInitLockDataDirectory (master...2017_07_appinitlockdatadirectory) https://github.com/bitcoin/bitcoin/pull/10832
< bitcoin-git> [bitcoin] laanwj closed pull request #10818: init: Keep track of whether data directory locked, don't cleanup if not (master...2017_05_locked_datadir) https://github.com/bitcoin/bitcoin/pull/10818
< wumpus> might be some calling convention difference
< wumpus> or at least platform ABI
< wumpus> gdb is not very robust against such things (accidental or to step reverse-engineering) gets easily confused if certain registers are not set up as they should
< fanquake> what is with this spam
< fanquake> wumpus Any chance you've tested Boost 1.64.0 on openbsd?
< bitcoin-git> [bitcoin] practicalswift opened pull request #10833: Fix typos (master...typos-201707) https://github.com/bitcoin/bitcoin/pull/10833
< wumpus> fanquake: only 1.63
< morcos> wumpus: I assume you knew if you give multiple named arguments for the same argument it's the last one that matters?
< morcos> I'm not sure how much of an issue that is, but seems like it could cause problems with wallet
< jamesob> If I wanted to do some kind of analysis on the UTXO set, is my best bet forking and making use of CCoinsView somehow? is https://github.com/bitcoin/bitcoin/pull/7759 worth resurrecting?
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8fdd23a224ba...505955052e60
< bitcoin-git> bitcoin/master 0189d8e practicalswift: Fix typos
< bitcoin-git> bitcoin/master 5059550 Pieter Wuille: Merge #10833: Fix typos...
< bitcoin-git> [bitcoin] sipa closed pull request #10833: Fix typos (master...typos-201707) https://github.com/bitcoin/bitcoin/pull/10833
< bitcoin-git> [bitcoin] practicalswift opened pull request #10834: Remove declaration of unused function: void UpdatedTransaction(const uint256 &) (master...remove-UpdatedTransaction) https://github.com/bitcoin/bitcoin/pull/10834
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/505955052e60...afd2fca911c4
< bitcoin-git> bitcoin/master 228987d Gregory Sanders: getbalance example covers at least 6 confirms
< bitcoin-git> bitcoin/master afd2fca Pieter Wuille: Merge #10807: getbalance example covers at least 6 confirms...
< bitcoin-git> [bitcoin] sipa closed pull request #10807: getbalance example covers at least 6 confirms (master...getbalexample) https://github.com/bitcoin/bitcoin/pull/10807
< bitcoin-git> [bitcoin] paveljanik opened pull request #10835: Do not shadow a field of 'reverse_range<T>' (master...20170715_Wshadow_reverseiterator) https://github.com/bitcoin/bitcoin/pull/10835
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/afd2fca911c4...10b22e3141a6
< bitcoin-git> bitcoin/master 0aadc11 Pieter Wuille: Avoid dereference-of-casted-pointer
< bitcoin-git> bitcoin/master 10b22e3 Pieter Wuille: Merge #10760: Avoid dereference-of-casted-pointer...
< bitcoin-git> [bitcoin] sipa closed pull request #10760: Avoid dereference-of-casted-pointer (master...20170707_avoidcastptr) https://github.com/bitcoin/bitcoin/pull/10760
< bitcoin-git> [bitcoin] corebob opened pull request #10837: Fix resource leak (master...20170715-fix-leak-1) https://github.com/bitcoin/bitcoin/pull/10837
< bitcoin-git> [bitcoin] sipa pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/10b22e3141a6...c5904e871479
< bitcoin-git> bitcoin/master fe4faba John Newbery: [refactor] move SplitHostPort() into utilstrencodings...
< bitcoin-git> bitcoin/master 5c64324 John Newbery: [utils] allow square brackets for ipv6 addresses in bitcoin-cli...
< bitcoin-git> bitcoin/master c5904e8 Pieter Wuille: Merge #10812: [utils] Allow bitcoin-cli's -rpcconnect option to be used with square brackets...
< bitcoin-git> [bitcoin] sipa closed pull request #10812: [utils] Allow bitcoin-cli's -rpcconnect option to be used with square brackets (master...bitcoin_cli_ipv6) https://github.com/bitcoin/bitcoin/pull/10812
< bitcoin-git> [bitcoin] sipa pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/c5904e871479...5cfdda2503c9
< bitcoin-git> bitcoin/master 4a3fc35 Matt Corallo: Track keypool entries as internal vs external in memory...
< bitcoin-git> bitcoin/master 28301b9 Matt Corallo: Meet code style on lines changed in the previous commit
< bitcoin-git> bitcoin/master d40a72c Matt Corallo: Clarify *(--.end()) iterator semantics in CWallet::TopUpKeyPool
< bitcoin-git> [bitcoin] sipa closed pull request #10235: Track keypool entries as internal vs external in memory (master...2017-04-wallet-more-keypool-cache) https://github.com/bitcoin/bitcoin/pull/10235
< sipa> cfields, fanquake: feel like testing #10821 again?
< gribble> https://github.com/bitcoin/bitcoin/issues/10821 | [WIP] Add SSE 4.2 optimized SHA256 by sipa · Pull Request #10821 · bitcoin/bitcoin · GitHub
< sipa> i've added a commit that makes all register choices explicit, but still lets the compiler manage the stack
< BlueMatt> can we remove getinfo for 0.15?
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #10838: (finally) remove the longest-ever-deprecated RPC call getinfo (master...2017-07-seriously-fuck-getinfo) https://github.com/bitcoin/bitcoin/pull/10838
< BlueMatt> uhhh, sipa, are you sure https://github.com/bitcoin/bitcoin/pull/10770#issuecomment-315558931 is specific to that PR?