1 2017-07-15 00:02:01 0|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?
2 2017-07-15 00:04:24 0|sipa|cfields: works fine here
3 2017-07-15 00:06:26 0|cfields|:(
4 2017-07-15 00:10:16 0|sipa|cfields: in other news, your compiler is retarded
5 2017-07-15 00:10:21 0|sipa|0000000000000011 movq %rsi, -0x38(%rbp)
6 2017-07-15 00:10:26 0|sipa|000000000000001d movq -0x38(%rbp), %rsi
7 2017-07-15 00:12:01 0|cfields|erm, that just moves and moves back?
8 2017-07-15 00:12:13 0|sipa|yes
9 2017-07-15 00:12:23 0|cfields|heh
10 2017-07-15 00:12:46 0|cfields|fulfilling a constraint, maybe?
11 2017-07-15 00:13:16 0|sipa|nope
12 2017-07-15 00:13:26 0|sipa|it could leave everything in the original order even
13 2017-07-15 00:40:39 0|cfields|sipa: weird. based on my dinky little test app, it seems to be working
14 2017-07-15 00:43:01 0|sipa|cfields: eh
15 2017-07-15 00:43:23 0|cfields|sipa: not sure yet. confused.
16 2017-07-15 00:44:16 0|cfields|i wrote a little sha256sum clone that just reads stdin and spits out a hash. matches sha256sum so far.
17 2017-07-15 00:44:46 0|sipa|cfields: does it use the sse code?
18 2017-07-15 00:46:33 0|cfields|Assertion failed: (0), function Transform, file crypto/sha256_sse42.cpp, line 16.
19 2017-07-15 00:46:43 0|cfields|yep. just threw ^^ assert in to verify.
20 2017-07-15 00:50:15 0|bitcoin-git|[13bitcoin] 15sipa pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/b7d6623c76e1...f90603ac6d24
21 2017-07-15 00:50:16 0|bitcoin-git|13bitcoin/06master 143babbcb 15Gregory Maxwell: Remove confusing MAX_BLOCK_BASE_SIZE....
22 2017-07-15 00:50:16 0|bitcoin-git|13bitcoin/06master 14f90603a 15Pieter Wuille: Merge #10618: Remove confusing MAX_BLOCK_BASE_SIZE....
23 2017-07-15 00:50:34 0|bitcoin-git|[13bitcoin] 15sipa closed pull request #10618: Remove confusing MAX_BLOCK_BASE_SIZE. (06master...06size_b_gone) 02https://github.com/bitcoin/bitcoin/pull/10618
24 2017-07-15 00:51:27 0|sipa|BlueMatt: can you fix the typo in the commit title of #10808?
25 2017-07-15 00:51:29 0|gribble|https://github.com/bitcoin/bitcoin/issues/10808 | Avoid some new gcc warnings in 15 by TheBlueMatt ÷ Pull Request #10808 ÷ bitcoin/bitcoin ÷ GitHub
26 2017-07-15 00:52:14 0|BlueMatt|heh, oops
27 2017-07-15 00:52:47 0|BlueMatt|fixed, sipa
28 2017-07-15 01:00:07 0|sipa|cfields: so it seems that the SSE PR compiles fine on travis, but no tests get run...
29 2017-07-15 01:00:41 0|cfields|sipa: right. we'll need to find some other osx sucker
30 2017-07-15 01:02:06 0|sipa|cfields: you're just doing a straightforward clean build of the PR?
31 2017-07-15 01:02:20 0|cfields|sipa: yep
32 2017-07-15 01:02:30 0|cfields|sipa: did a cross build via depends too with the same result
33 2017-07-15 01:02:48 0|sipa|:(
34 2017-07-15 01:08:23 0|bitcoin-git|[13bitcoin] 15sipa pushed 3 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/f90603ac6d24...ec8a50b8d786
35 2017-07-15 01:08:24 0|bitcoin-git|13bitcoin/06master 141016dac 15Matt Corallo: Fix signed/unsigned comparison warning introduced in c8e29d7ff0.
36 2017-07-15 01:08:24 0|bitcoin-git|13bitcoin/06master 14c73b8be 15Matt Corallo: Explicitly initialize prevector::_union to avoid new warning...
37 2017-07-15 01:08:25 0|bitcoin-git|13bitcoin/06master 14ec8a50b 15Pieter Wuille: Merge #10808: Avoid some new gcc warnings in 15...
38 2017-07-15 01:08:53 0|bitcoin-git|[13bitcoin] 15sipa closed pull request #10808: Avoid some new gcc warnings in 15 (06master...062017-07-15-new-warnings) 02https://github.com/bitcoin/bitcoin/pull/10808
39 2017-07-15 01:18:04 0|bitcoin-git|[13bitcoin] 15gmaxwell opened pull request #10831: Batch flushing operations to the walletdb during top up and increase keypool size. (06master...06topup_batch_flush) 02https://github.com/bitcoin/bitcoin/pull/10831
40 2017-07-15 01:22:18 0|sipa|BlueMatt: i am confused by the change in #10235
41 2017-07-15 01:22:19 0|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
42 2017-07-15 01:22:27 0|sipa|LogPrintf("keypool added key %d, size=%u (%u internal), internal=%d\n", nEnd, setInternalKeyPool.size() + setExternalKeyPool.size(), setInternalKeyPool.size(), internal);
43 2017-07-15 01:22:31 0|sipa|2x internal?
44 2017-07-15 01:23:03 0|sipa|oh, one is a bool
45 2017-07-15 01:23:08 0|BlueMatt|oh ffs, ok, i'll fix, sec
46 2017-07-15 01:25:34 0|BlueMatt|fixed
47 2017-07-15 01:46:06 0|promag|What about a wip tag in GitHub?
48 2017-07-15 01:54:34 0|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)
49 2017-07-15 01:54:35 0|gribble|The operation succeeded.
50 2017-07-15 02:08:07 0|bitcoin-git|13bitcoin/06master 14dd97a52 15Alex Morcos: Properly forbid -salvagewallet and -zapwallettxes for multi wallet.
51 2017-07-15 02:08:07 0|bitcoin-git|[13bitcoin] 15sipa pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/ec8a50b8d786...1c011ff43010
52 2017-07-15 02:08:08 0|bitcoin-git|13bitcoin/06master 141c011ff 15Pieter Wuille: Merge #10816: Properly forbid -salvagewallet and -zapwallettxes for multi wallet....
53 2017-07-15 02:08:41 0|bitcoin-git|[13bitcoin] 15sipa closed pull request #10816: Properly forbid -salvagewallet and -zapwallettxes for multi wallet. (06master...06multiwallet_parameters) 02https://github.com/bitcoin/bitcoin/pull/10816
54 2017-07-15 02:13:22 0|gmaxwell|Is there anyone who uses OSX around? #10821 needs testing for sipa.
55 2017-07-15 02:13:23 0|gribble|https://github.com/bitcoin/bitcoin/issues/10821 | [WIP] Add SSE 4.2 optimized SHA256 by sipa ÷ Pull Request #10821 ÷ bitcoin/bitcoin ÷ GitHub
56 2017-07-15 02:24:34 0|bitcoin-git|[13bitcoin] 15sipa pushed 3 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/1c011ff43010...8fdd23a224ba
57 2017-07-15 02:24:35 0|bitcoin-git|13bitcoin/06master 142aef1f1 15Jonas Schnelli: [Qt] migrate old fee slider value to new dropbown...
58 2017-07-15 02:24:35 0|bitcoin-git|13bitcoin/06master 14bc1be90 15Jonas Schnelli: [Qt] replace fee slider with a Dropdown, extend conf. targets
59 2017-07-15 02:24:36 0|bitcoin-git|13bitcoin/06master 148fdd23a 15Pieter Wuille: Merge #10769: [Qt] replace fee slider with a Dropdown, extend conf. targets...
60 2017-07-15 02:25:01 0|bitcoin-git|[13bitcoin] 15sipa closed pull request #10769: [Qt] replace fee slider with a Dropdown, extend conf. targets (06master...062017/07/qt_fee_slider) 02https://github.com/bitcoin/bitcoin/pull/10769
61 2017-07-15 06:20:11 0|cfields|sipa: still around? I managed to piss clang off on linux
62 2017-07-15 06:20:21 0|cfields|not sure if it's the same issue or if I forced another
63 2017-07-15 06:23:27 0|fanquake|gmaxwell cfields what do you want tested? Merging 10821 now.
64 2017-07-15 06:23:59 0|cfields|fanquake: "./bitcoind" is enough :)
65 2017-07-15 06:24:29 0|cfields|for me it segfaults on the genesis block check
66 2017-07-15 06:26:31 0|fanquake|cfields Indeed Assertion failed: (consensus.hashGenesisBlock etc
67 2017-07-15 06:26:54 0|cfields|fanquake: ok, so i'm not alone. great. also, shit.
68 2017-07-15 06:27:57 0|cfields|fanquake: thanks for testing!
69 2017-07-15 06:29:07 0|fanquake|cfields no worries. Ping me if you want anything looked at. Should be around for the rest of the afternoon.
70 2017-07-15 06:29:37 0|sipa|cfields: how did you make clang piss off in linux?
71 2017-07-15 06:29:45 0|sipa|can you create and a send me a static binary?
72 2017-07-15 06:30:07 0|cfields|sipa: still fiddling with it. I'm having troubles with boost as well.
73 2017-07-15 06:30:19 0|sipa|cfields: let's have a look at it in tokyo
74 2017-07-15 06:30:47 0|sipa|i can try a static register allocation first too, and see if that works
75 2017-07-15 06:31:01 0|sipa|why am i talking here? i'm getting drunk
76 2017-07-15 06:31:57 0|cfields|haha. are you irc'ing at a bar?
77 2017-07-15 06:32:33 0|sipa|yes
78 2017-07-15 06:32:40 0|cfields|sipa: using clang with -fintegrated-as caused a new crash in linux
79 2017-07-15 06:32:59 0|cfields|my assumption is that it uses gnu as on linux, and llvm's as for apple
80 2017-07-15 06:33:28 0|cfields|but i'll come up with something solid and ping you then. forget about assembly for the night :)
81 2017-07-15 06:41:36 0|cfields|heh, confirmed
82 2017-07-15 06:41:49 0|cfields|unfortunately, it crashes gdb too :(
83 2017-07-15 08:49:25 0|bitcoin-git|[13bitcoin] 15laanwj opened pull request #10832: init: Factor out AppInitLockDataDirectory (06master...062017_07_appinitlockdatadirectory) 02https://github.com/bitcoin/bitcoin/pull/10832
84 2017-07-15 08:49:55 0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #10818: init: Keep track of whether data directory locked, don't cleanup if not (06master...062017_05_locked_datadir) 02https://github.com/bitcoin/bitcoin/pull/10818
85 2017-07-15 09:46:27 0|wumpus|might be some calling convention difference
86 2017-07-15 09:46:41 0|wumpus|or at least platform ABI
87 2017-07-15 09:47:34 0|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
88 2017-07-15 10:32:45 0|fanquake|what is with this spam
89 2017-07-15 10:53:43 0|fanquake|wumpus Any chance you've tested Boost 1.64.0 on openbsd?
90 2017-07-15 12:32:23 0|bitcoin-git|[13bitcoin] 15practicalswift opened pull request #10833: Fix typos (06master...06typos-201707) 02https://github.com/bitcoin/bitcoin/pull/10833
91 2017-07-15 14:18:41 0|wumpus|fanquake: only 1.63
92 2017-07-15 15:23:27 0|morcos|wumpus: I assume you knew if you give multiple named arguments for the same argument it's the last one that matters?
93 2017-07-15 15:23:46 0|morcos|I'm not sure how much of an issue that is, but seems like it could cause problems with wallet
94 2017-07-15 15:32:06 0|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?
95 2017-07-15 18:02:17 0|bitcoin-git|13bitcoin/06master 140189d8e 15practicalswift: Fix typos
96 2017-07-15 18:02:17 0|bitcoin-git|[13bitcoin] 15sipa pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/8fdd23a224ba...505955052e60
97 2017-07-15 18:02:18 0|bitcoin-git|13bitcoin/06master 145059550 15Pieter Wuille: Merge #10833: Fix typos...
98 2017-07-15 18:02:46 0|bitcoin-git|[13bitcoin] 15sipa closed pull request #10833: Fix typos (06master...06typos-201707) 02https://github.com/bitcoin/bitcoin/pull/10833
99 2017-07-15 18:08:24 0|bitcoin-git|[13bitcoin] 15practicalswift opened pull request #10834: Remove declaration of unused function: void UpdatedTransaction(const uint256 &) (06master...06remove-UpdatedTransaction) 02https://github.com/bitcoin/bitcoin/pull/10834
100 2017-07-15 18:12:48 0|bitcoin-git|13bitcoin/06master 14228987d 15Gregory Sanders: getbalance example covers at least 6 confirms
101 2017-07-15 18:12:48 0|bitcoin-git|[13bitcoin] 15sipa pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/505955052e60...afd2fca911c4
102 2017-07-15 18:12:49 0|bitcoin-git|13bitcoin/06master 14afd2fca 15Pieter Wuille: Merge #10807: getbalance example covers at least 6 confirms...
103 2017-07-15 18:13:18 0|bitcoin-git|[13bitcoin] 15sipa closed pull request #10807: getbalance example covers at least 6 confirms (06master...06getbalexample) 02https://github.com/bitcoin/bitcoin/pull/10807
104 2017-07-15 19:08:48 0|bitcoin-git|[13bitcoin] 15paveljanik opened pull request #10835: Do not shadow a field of 'reverse_range<T>' (06master...0620170715_Wshadow_reverseiterator) 02https://github.com/bitcoin/bitcoin/pull/10835
105 2017-07-15 19:23:17 0|bitcoin-git|13bitcoin/06master 140aadc11 15Pieter Wuille: Avoid dereference-of-casted-pointer
106 2017-07-15 19:23:17 0|bitcoin-git|[13bitcoin] 15sipa pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/afd2fca911c4...10b22e3141a6
107 2017-07-15 19:23:18 0|bitcoin-git|13bitcoin/06master 1410b22e3 15Pieter Wuille: Merge #10760: Avoid dereference-of-casted-pointer...
108 2017-07-15 19:23:42 0|bitcoin-git|[13bitcoin] 15sipa closed pull request #10760: Avoid dereference-of-casted-pointer (06master...0620170707_avoidcastptr) 02https://github.com/bitcoin/bitcoin/pull/10760
109 2017-07-15 19:38:43 0|bitcoin-git|[13bitcoin] 15corebob opened pull request #10837: Fix resource leak (06master...0620170715-fix-leak-1) 02https://github.com/bitcoin/bitcoin/pull/10837
110 2017-07-15 20:27:14 0|bitcoin-git|13bitcoin/06master 14fe4faba 15John Newbery: [refactor] move SplitHostPort() into utilstrencodings...
111 2017-07-15 20:27:14 0|bitcoin-git|[13bitcoin] 15sipa pushed 3 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/10b22e3141a6...c5904e871479
112 2017-07-15 20:27:15 0|bitcoin-git|13bitcoin/06master 145c64324 15John Newbery: [utils] allow square brackets for ipv6 addresses in bitcoin-cli...
113 2017-07-15 20:27:15 0|bitcoin-git|13bitcoin/06master 14c5904e8 15Pieter Wuille: Merge #10812: [utils] Allow bitcoin-cli's -rpcconnect option to be used with square brackets...
114 2017-07-15 20:27:45 0|bitcoin-git|[13bitcoin] 15sipa closed pull request #10812: [utils] Allow bitcoin-cli's -rpcconnect option to be used with square brackets (06master...06bitcoin_cli_ipv6) 02https://github.com/bitcoin/bitcoin/pull/10812
115 2017-07-15 21:02:27 0|bitcoin-git|[13bitcoin] 15sipa pushed 4 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/c5904e871479...5cfdda2503c9
116 2017-07-15 21:02:28 0|bitcoin-git|13bitcoin/06master 1428301b9 15Matt Corallo: Meet code style on lines changed in the previous commit
117 2017-07-15 21:02:28 0|bitcoin-git|13bitcoin/06master 144a3fc35 15Matt Corallo: Track keypool entries as internal vs external in memory...
118 2017-07-15 21:02:29 0|bitcoin-git|13bitcoin/06master 14d40a72c 15Matt Corallo: Clarify *(--.end()) iterator semantics in CWallet::TopUpKeyPool
119 2017-07-15 21:02:42 0|bitcoin-git|[13bitcoin] 15sipa closed pull request #10235: Track keypool entries as internal vs external in memory (06master...062017-04-wallet-more-keypool-cache) 02https://github.com/bitcoin/bitcoin/pull/10235
120 2017-07-15 23:32:41 0|sipa|cfields, fanquake: feel like testing #10821 again?
121 2017-07-15 23:32:43 0|gribble|https://github.com/bitcoin/bitcoin/issues/10821 | [WIP] Add SSE 4.2 optimized SHA256 by sipa ÷ Pull Request #10821 ÷ bitcoin/bitcoin ÷ GitHub
122 2017-07-15 23:33:05 0|sipa|i've added a commit that makes all register choices explicit, but still lets the compiler manage the stack
123 2017-07-15 23:47:16 0|BlueMatt|can we remove getinfo for 0.15?
124 2017-07-15 23:49:03 0|bitcoin-git|[13bitcoin] 15TheBlueMatt opened pull request #10838: (finally) remove the longest-ever-deprecated RPC call getinfo (06master...062017-07-seriously-fuck-getinfo) 02https://github.com/bitcoin/bitcoin/pull/10838
125 2017-07-15 23:54:09 0|BlueMatt|uhhh, sipa, are you sure https://github.com/bitcoin/bitcoin/pull/10770#issuecomment-315558931 is specific to that PR?