1 2016-12-03 02:25:30	0|bitcoin-git|[13bitcoin] 15sipa pushed 4 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/c36229b0b2e9...2efcfa5acfac
 2 2016-12-03 02:25:31	0|bitcoin-git|13bitcoin/06master 1487c35f5 15Matt Corallo: Remove orphan state wipe from UnloadBlockIndex....
 3 2016-12-03 02:25:31	0|bitcoin-git|13bitcoin/06master 14e736772 15Matt Corallo: Move network-msg-processing code out of main to its own file
 4 2016-12-03 02:25:32	0|bitcoin-git|13bitcoin/06master 1476faa3c 15Matt Corallo: Rename the remaining main.{h,cpp} to validation.{h,cpp}
 5 2016-12-03 02:25:46	0|bitcoin-git|[13bitcoin] 15sipa closed pull request #9260: Mrs Peacock in The Library with The Candlestick (killed main.{h,cpp}) (06master...06net_processing_file) 02https://github.com/bitcoin/bitcoin/pull/9260
 6 2016-12-03 02:25:54	0|cfields|let the rebasing begin :)
 7 2016-12-03 02:25:58	0|sipa|now: the big rebasing game
 8 2016-12-03 02:26:01	0|sipa|jinx
 9 2016-12-03 02:26:04	0|cfields|haha
10 2016-12-03 02:26:10	0|BlueMatt|hey-o!
11 2016-12-03 02:27:16	0|cfields|BlueMatt: there are a few includes cleanups that can be done that should help with memory
12 2016-12-03 02:27:40	0|cfields|BlueMatt: not sure if you had a few in the queue, or if i should go ahead and PR
13 2016-12-03 02:28:01	0|BlueMatt|cfields: I dont have a queue of them, but was playing around at https://github.com/TheBlueMatt/bitcoin/commits/2016-12-memusage
14 2016-12-03 02:28:16	0|BlueMatt|feel free to peruse that and PR whatever you did
15 2016-12-03 02:28:36	0|sipa|damn, validation.cpp is *still* 8000 lines
16 2016-12-03 02:28:40	0|cfields|ok
17 2016-12-03 02:29:28	0|cfields|BlueMatt: boost/filesystem/path.hpp in validation.h was the only one i really had my eye on
18 2016-12-03 02:30:24	0|BlueMatt|now that I read the feedback, cfields' comments pointed out that filterRounder changed behavior just slightly - instead of rounding based on DEFAULT_MIN_RELAY_TX_FEE it now rounds based on -minrelaytxfee, oops
19 2016-12-03 02:30:40	0|BlueMatt|doesnt matter all that much, but the original behavior is probably more correct
20 2016-12-03 02:30:45	0|BlueMatt|welll, actually
21 2016-12-03 02:31:03	0|BlueMatt|hum, dunno, needs pointed out for discussion, not sure if its better that users can change that rounding or not
22 2016-12-03 02:31:15	0|BlueMatt|(given hardcoded-value-avoidance-policies)
23 2016-12-03 02:31:24	0|BlueMatt|I'll file an issue and open it up
24 2016-12-03 02:31:27	0|sipa|users changing the rounding is probably a slight privacy leak
25 2016-12-03 02:31:37	0|sipa|which is afaik the reason for rounding in the first place
26 2016-12-03 02:31:38	0|cfields|BlueMatt: hmm, i thought i verified the behavior didn't change there. Not sure how i noticed the mempool weirdness without seeing that too
27 2016-12-03 02:31:55	0|BlueMatt|it is, technically, but I'd be surprised if it wasnt already visible what your -minrelaytxfee is
28 2016-12-03 02:32:00	0|BlueMatt|but, ok, will pr a fix and see what folks say
29 2016-12-03 02:36:27	0|bitcoin-git|[13bitcoin] 15TheBlueMatt opened pull request #9268: Fix rounding privacy leak introduced in #9260 (06master...062016-12-feefilterrounder) 02https://github.com/bitcoin/bitcoin/pull/9268
30 2016-12-03 02:44:31	0|morcos|woo hoo!
31 2016-12-03 03:00:20	0|sipa|jtimon: do you know the game cluedo?
32 2016-12-03 03:00:46	0|jtimon|I don't think have played, but yes, it's famous
33 2016-12-03 03:01:15	0|sipa|the game ends when a player figures out who the murdered is, where the murder happened, and with what weapon
34 2016-12-03 03:01:22	0|jtimon|just enjoy with anything make main smaller
35 2016-12-03 03:01:24	0|sipa|*murderer
36 2016-12-03 03:01:35	0|alpalp|the game also ends if you guess wrong
37 2016-12-03 03:01:37	0|sipa|jtimon: well, main.cpp now has size 0 :p
38 2016-12-03 03:01:58	0|jtimon|right, calling main validation is going to take some time...
39 2016-12-03 03:04:24	0|jtimon|btw, I was serious about rewritting any part of https://github.com/bitcoin/bitcoin/pull/8328 that can be agreed on, just without any clear agreement, there's no point and keeping that open for long (since it's guaranteed to be arebase hell)
40 2016-12-03 03:04:53	0|jtimon|what about moving bitcoinconsensus.o from script to consensus?
41 2016-12-03 03:05:08	0|jtimon|anyway, first rebase
42 2016-12-03 03:06:42	0|sipa|BlueMatt: how do you measure compiler memory usage?
43 2016-12-03 03:29:19	0|cfields|BlueMatt: see what you think about https://github.com/theuni/bitcoin/commit/3f598dbe7100c7c6c7bfb7e10210585327ed9d31
44 2016-12-03 03:29:56	0|bitcoin-git|[13bitcoin] 15sipa opened pull request #9269: Align struct COrphan definition (06master...06oneorphan) 02https://github.com/bitcoin/bitcoin/pull/9269
45 2016-12-03 03:29:56	0|cfields|BlueMatt: I just hacked it up after the main split because it looked kinda trivial, not sure if that's a direction worth going in or not
46 2016-12-03 03:32:05	0|sipa|BlueMatt: i succesfully compiled with flto, and i never saw memory usage of a single process go above ~700MB (i was just watching with top, this may not be very accurate)
47 2016-12-03 03:32:09	0|cfields|hmm, actually, CConnman could easily just hold the interface pointer, since it's abstract
48 2016-12-03 03:32:44	0|cfields|sipa: isn't lto much less intensive on the individual compilation units?
49 2016-12-03 03:32:59	0|cfields|or was that your point?
50 2016-12-03 03:33:02	0|sipa|yes
51 2016-12-03 03:33:20	0|cfields|ok, nm me
52 2016-12-03 03:53:37	0|sipa|BlueMatt: https://www.zerobin.net/?cd7cc1b7c1d2c689#/IYxSnkHTRRd5FdHJRRPwmRNZKdq+YPvKOW6rt0acTQ=
53 2016-12-03 04:26:59	0|BlueMatt|sipa: you want grep VmPeak /proc/`pidof cc1plus`/status
54 2016-12-03 04:30:29	0|sipa|this is made using /usr/bin/time -f "%M" g++
55 2016-12-03 04:30:31	0|BlueMatt|cfields: concept ack, but I'd say yes to just about anything that removed boost in any context
56 2016-12-03 04:30:39	0|BlueMatt|sipa: oh, even better
57 2016-12-03 04:31:06	0|sipa|BlueMatt: i made a tiny wrapper around g++ that calls time, and passed that as CXX= to configure
58 2016-12-03 04:31:26	0|BlueMatt|yea
59 2016-12-03 04:33:59	0|sipa|oh, the numbers for the binaries are likely inaccurate, as gcc spawns child processes that do the linking/compiling
60 2016-12-03 04:34:20	0|BlueMatt|yea, i mean thats where lto uses all its memory/time, though?
61 2016-12-03 04:34:27	0|sipa|time, yes
62 2016-12-03 04:34:30	0|sipa|memory, no
63 2016-12-03 04:34:37	0|BlueMatt|oh? the streaming shit?
64 2016-12-03 04:35:02	0|sipa|the memory used by those processes seems much lower than the typical first step compilation
65 2016-12-03 04:38:02	0|sipa|like 60MB or so
66 2016-12-03 05:48:00	0|bitcoin-git|[13bitcoin] 15jtimon opened pull request #9271: Discusion : 0.13 consensus flags error (06master...060.13-consensus-flags-error) 02https://github.com/bitcoin/bitcoin/pull/9271
67 2016-12-03 09:31:13	0|bitcoin-git|[13bitcoin] 15jtimon closed pull request #9271: Discusion : 0.13 consensus flags error (06master...060.13-consensus-flags-error) 02https://github.com/bitcoin/bitcoin/pull/9271
68 2016-12-03 09:49:28	0|bitcoin-git|[13bitcoin] 15jtimon reopened pull request #9271: Discusion : 0.13 consensus flags error (06master...060.13-consensus-flags-error) 02https://github.com/bitcoin/bitcoin/pull/9271
69 2016-12-03 10:58:33	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #9267: Disable fee estimates for a confirm target of 1 block (060.13...06backport9239) 02https://github.com/bitcoin/bitcoin/pull/9267
70 2016-12-03 11:15:26	0|wumpus|strange, bitcoin-git shows the close notice but not the merge on the 0.13 branch
71 2016-12-03 11:15:41	0|wumpus|first thought I forgot to push, but I didn't
72 2016-12-03 11:16:15	0|btcdrak|wumpus: I have the same problem with notifications for another repo.
73 2016-12-03 11:16:33	0|wumpus|btcdrak: may be just an intermittent issue, it's the first time I notice at least
74 2016-12-03 12:23:34	0|bitcoin-git|[13bitcoin] 15jtimon closed pull request #7779: Discussion: Consensus: There's only one type of consensus flags (06master...060.12.99-consensus-unify-flags) 02https://github.com/bitcoin/bitcoin/pull/7779