12016-12-31T00:20:22  <morcos> the reason this doesn't happen with 9375 alone is that peer 0 hasn't requested that peer 1 be a HB peer, so the reorg isn't announced via the fast mechanism, i'm guessing 9400 changed that, but haven't looked into the details
  22016-12-31T00:29:31  *** harrymm has quit IRC
  32016-12-31T00:30:37  *** harrymm has joined #bitcoin-core-dev
  42016-12-31T01:08:31  <gmaxwell> I wondered if that might also force us to cache more than one block.
  52016-12-31T01:09:31  <gmaxwell> e.g. what happens if we get a second block instantly after the first, since they're tied will we still advertise the second?
  62016-12-31T01:10:10  *** juscamarena has quit IRC
  72016-12-31T01:10:50  *** juscamarena has joined #bitcoin-core-dev
  82016-12-31T01:10:50  *** juscamarena_ has joined #bitcoin-core-dev
  92016-12-31T01:18:22  *** juscamarena_ has quit IRC
 102016-12-31T01:26:42  *** Elysus1 has quit IRC
 112016-12-31T01:31:17  *** AaronvanW has quit IRC
 122016-12-31T01:37:12  *** kadoban has joined #bitcoin-core-dev
 132016-12-31T02:34:27  *** juscamarena has quit IRC
 142016-12-31T02:56:16  *** Elysus has joined #bitcoin-core-dev
 152016-12-31T02:57:25  *** Ylbam has quit IRC
 162016-12-31T02:59:38  *** arowser has quit IRC
 172016-12-31T03:01:22  *** fengling has quit IRC
 182016-12-31T03:03:36  *** xiangfu has quit IRC
 192016-12-31T03:04:22  *** xiangfu has joined #bitcoin-core-dev
 202016-12-31T03:05:24  *** arowser has joined #bitcoin-core-dev
 212016-12-31T03:06:34  <morcos> gmaxwell: that sounds annoying.  seems like maybe we need to rework the signals a bit.  maybe make BlockChecked always called after ProcessNewBlock and we can hold a cache of all blocks from NewPowValidBlock until BlockChecked.
 222016-12-31T03:06:38  <morcos> also always cache the tip
 232016-12-31T03:07:14  *** fengling has joined #bitcoin-core-dev
 242016-12-31T03:07:33  <morcos> but then we have to figure out where the right place to call MaybeSetPeer.. is
 252016-12-31T03:08:30  <morcos> But there is already another minor problem with the way the signals work now, mapBlockSource is never cleared of blocks which don't have BlockChecked called on them
 262016-12-31T03:09:31  <morcos> and valid blocks that don't become your tip don't get that I don't think..  probably rare enough not to matter much now, but if we want to be have an alternate block request mechanism for SPV/hybrid whatever or something..
 272016-12-31T03:20:35  *** juscamarena has joined #bitcoin-core-dev
 282016-12-31T05:01:38  *** laurentmt has joined #bitcoin-core-dev
 292016-12-31T05:42:03  *** kadoban has quit IRC
 302016-12-31T05:56:06  *** Alopex has quit IRC
 312016-12-31T05:57:11  *** Alopex has joined #bitcoin-core-dev
 322016-12-31T06:01:17  *** cannon-c has joined #bitcoin-core-dev
 332016-12-31T06:11:54  *** Chris_Stewart_5 has quit IRC
 342016-12-31T06:13:25  *** TomMc has quit IRC
 352016-12-31T06:25:33  *** Raccoon has joined #bitcoin-core-dev
 362016-12-31T06:25:46  *** Raccoon has left #bitcoin-core-dev
 372016-12-31T07:20:54  *** justanotheruser has quit IRC
 382016-12-31T07:29:43  <cfields> sipa: if you're here, please refresh your browser. You're reviewing stale commits :)
 392016-12-31T07:31:40  <sipa> :(
 402016-12-31T07:31:49  <sipa> i was reviewing from the app
 412016-12-31T07:32:49  <cfields> sipa: sorry. no clue what the app sees. I split them up into much more reviewable chunks, i hope.
 422016-12-31T07:33:18  <sipa> i see it now
 432016-12-31T07:33:27  <sipa> i guess it just caches a bit more aggressively
 442016-12-31T07:33:32  <sipa> no worries
 452016-12-31T07:33:49  <sipa> that's also why the result ends up with commit comments
 462016-12-31T07:33:53  <sipa> rather than PR comments
 472016-12-31T07:34:32  <cfields> ah, that's annoying. I was just looking at the page and saw no comments on the pr. Checked my mail on my phone on a whim.
 482016-12-31T07:35:16  <cfields> no worries though, i'll watch for that now
 492016-12-31T07:36:50  *** justanotheruser has joined #bitcoin-core-dev
 502016-12-31T07:49:58  *** fengling has quit IRC
 512016-12-31T08:00:06  *** Alopex has quit IRC
 522016-12-31T08:01:11  *** Alopex has joined #bitcoin-core-dev
 532016-12-31T08:13:25  *** wumpus_ has quit IRC
 542016-12-31T08:15:30  *** wumpus has joined #bitcoin-core-dev
 552016-12-31T08:33:06  *** Alopex has quit IRC
 562016-12-31T08:34:11  *** Alopex has joined #bitcoin-core-dev
 572016-12-31T08:48:48  <sipa> cfields: i was surprised to see that 2*buffersize was still there, tbh
 582016-12-31T08:50:03  <cfields> sipa: i couldn't really imagine a scenario where it could be hit. I just imagined a response of "belt and suspenders"
 592016-12-31T08:50:10  <cfields> should've asked :)
 602016-12-31T08:50:29  <sipa> yeah, belt and suspenders indeed
 612016-12-31T08:51:01  <cfields> will nuke it though. I've certainly never seen it. And I've done enough stupid things in tests that would've prompted it.
 622016-12-31T08:51:03  <sipa> but if you want to keep it, don't just turn it into fPauseSend
 632016-12-31T08:52:02  *** fengling has joined #bitcoin-core-dev
 642016-12-31T08:52:42  <cfields> well, I think the checking now should be tight enough. But I'll add some tests. I think the corks could've been quite broken and gone unnotiiced.
 652016-12-31T08:54:12  <sipa> if you set max send size below the block size, you'll hit it automatically :)
 662016-12-31T08:57:10  <cfields> see 30ad8c069bcaf4ef8b6bf17498386bcb42e39fc3 :(
 672016-12-31T08:58:15  <cfields> s/send/recv/ though
 682016-12-31T08:59:09  <sipa> yeah, i saw
 692016-12-31T09:02:52  <cfields> headed to bed, thanks for having a look
 702016-12-31T09:02:57  <cfields> and happy new year!
 712016-12-31T09:03:17  <wumpus> happy new year!
 722016-12-31T09:03:31  <cfields> wumpus: !
 732016-12-31T09:06:29  <cannon-c> New year not for another day
 742016-12-31T09:06:35  <cannon-c> Happy New Years Eve!
 752016-12-31T09:07:18  <wumpus> yes, same here, it's still a few hours to go, just not sure I'm going to speak to cfields before that
 762016-12-31T09:08:55  <cannon-c> must be in Australia
 772016-12-31T09:12:11  *** cannon-c has left #bitcoin-core-dev
 782016-12-31T09:12:29  *** cannon-c has joined #bitcoin-core-dev
 792016-12-31T09:15:02  *** d9b4bef9 has quit IRC
 802016-12-31T09:16:07  *** d9b4bef9 has joined #bitcoin-core-dev
 812016-12-31T09:28:52  *** cannon-c is now known as cannon-c_AFK
 822016-12-31T09:30:56  *** Ylbam has joined #bitcoin-core-dev
 832016-12-31T10:09:39  *** cannon-c_AFK is now known as cannon-oc
 842016-12-31T10:09:46  *** cannon-oc is now known as cannon-c
 852016-12-31T10:21:17  *** AaronvanW has joined #bitcoin-core-dev
 862016-12-31T10:21:18  *** AaronvanW has joined #bitcoin-core-dev
 872016-12-31T13:02:04  *** Elysus has quit IRC
 882016-12-31T13:06:40  *** Elysus has joined #bitcoin-core-dev
 892016-12-31T13:09:15  *** Elysus1 has joined #bitcoin-core-dev
 902016-12-31T13:12:29  *** fengling has quit IRC
 912016-12-31T13:13:20  *** Elysus has quit IRC
 922016-12-31T13:16:53  *** Elysus1 has quit IRC
 932016-12-31T13:17:49  *** Elysus has joined #bitcoin-core-dev
 942016-12-31T13:19:35  *** Elysus1 has joined #bitcoin-core-dev
 952016-12-31T13:23:10  *** Elysus has quit IRC
 962016-12-31T13:26:53  *** Elysus has joined #bitcoin-core-dev
 972016-12-31T13:30:02  *** Elysus1 has quit IRC
 982016-12-31T13:40:14  *** arowser has quit IRC
 992016-12-31T13:42:32  *** arowser has joined #bitcoin-core-dev
1002016-12-31T14:00:11  *** Guyver2 has joined #bitcoin-core-dev
1012016-12-31T14:14:26  *** Alopex has quit IRC
1022016-12-31T14:15:32  *** Alopex has joined #bitcoin-core-dev
1032016-12-31T14:19:25  <BlueMatt> morcos: hey, sorry, not been around much...
1042016-12-31T14:19:31  <BlueMatt> morcos: I'm looking at the stuff now
1052016-12-31T14:54:53  *** Guyver2 has quit IRC
1062016-12-31T15:15:49  *** Distructo has joined #bitcoin-core-dev
1072016-12-31T15:16:30  *** fengling has joined #bitcoin-core-dev
1082016-12-31T15:16:43  *** arubi has quit IRC
1092016-12-31T15:18:57  *** arubi has joined #bitcoin-core-dev
1102016-12-31T15:21:47  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1112016-12-31T15:22:46  *** Distructo has quit IRC
1122016-12-31T15:28:14  *** Chris_Stewart_5 has quit IRC
1132016-12-31T15:44:02  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1142016-12-31T16:09:19  *** Sosumi has joined #bitcoin-core-dev
1152016-12-31T16:14:17  *** justanotheruser has quit IRC
1162016-12-31T16:15:47  *** laurentmt has quit IRC
1172016-12-31T16:27:23  *** Chris_Stewart_5 has quit IRC
1182016-12-31T17:17:41  *** PRab has quit IRC
1192016-12-31T17:27:17  *** Elysus1 has joined #bitcoin-core-dev
1202016-12-31T17:30:38  *** Elysus has quit IRC
1212016-12-31T17:49:51  *** TomMc has joined #bitcoin-core-dev
1222016-12-31T17:53:29  *** kadoban has joined #bitcoin-core-dev
1232016-12-31T18:06:08  <bitcoin-git> [bitcoin] isle2983 opened pull request #9450: Increment MIT licence copyright header year on files modified in 2016 (master...PR-increment-year) https://github.com/bitcoin/bitcoin/pull/9450
1242016-12-31T18:09:51  <bitcoin-git> [bitcoin] robmcl4 opened pull request #9451: CScript: remove redundant bounds check (master...remove_extra_bounds_check) https://github.com/bitcoin/bitcoin/pull/9451
1252016-12-31T18:21:50  *** Guyver2 has joined #bitcoin-core-dev
1262016-12-31T18:40:11  *** TomMc has quit IRC
1272016-12-31T18:41:30  <bitcoin-git> [bitcoin] isle2983 opened pull request #9452: Use TravisCI to enforce copyright header rules for source files (master...PR-travisci-copyright-enforce) https://github.com/bitcoin/bitcoin/pull/9452
1282016-12-31T18:57:45  *** AaronvanW has quit IRC
1292016-12-31T19:15:40  <btcdrak> Happy New Year everyone
1302016-12-31T19:20:48  *** Cheeseo has joined #bitcoin-core-dev
1312016-12-31T19:20:52  *** Cheeseo has joined #bitcoin-core-dev
1322016-12-31T19:31:18  *** justanotheruser has joined #bitcoin-core-dev
1332016-12-31T19:41:40  <instagibbs> yes happy new years, hoping for another year of successful releases
1342016-12-31T19:44:04  *** fengling has quit IRC
1352016-12-31T19:45:09  <paveljanik> Yay!
1362016-12-31T19:48:01  *** Cheeseo has quit IRC
1372016-12-31T19:48:51  <btcdrak> let's tag 0.13.2 for New Year!
1382016-12-31T19:50:25  *** Cheeseo has joined #bitcoin-core-dev
1392016-12-31T19:50:25  *** Cheeseo has joined #bitcoin-core-dev
1402016-12-31T20:04:54  <bitcoin-git> [bitcoin] jsmith-dev opened pull request #9453: Copyright notice year 2017 increment. Happy New Year! (master...master) https://github.com/bitcoin/bitcoin/pull/9453
1412016-12-31T20:06:41  *** Cheeseo has quit IRC
1422016-12-31T20:12:05  *** fengling has joined #bitcoin-core-dev
1432016-12-31T20:15:09  *** Cheeseo has joined #bitcoin-core-dev
1442016-12-31T20:31:19  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1452016-12-31T20:34:00  *** MarcoFalke has joined #bitcoin-core-dev
1462016-12-31T20:39:30  *** jcorgan has quit IRC
1472016-12-31T20:44:08  *** jcorgan has joined #bitcoin-core-dev
1482016-12-31T20:56:15  *** Cheeseo has quit IRC
1492016-12-31T21:12:32  *** Cheeseo has joined #bitcoin-core-dev
1502016-12-31T21:31:09  *** Elysus1 has quit IRC
1512016-12-31T21:32:57  *** kanu_ has joined #bitcoin-core-dev
1522016-12-31T22:03:52  *** Sosumi has quit IRC
1532016-12-31T22:09:28  *** nickler has quit IRC
1542016-12-31T22:16:48  *** nickler has joined #bitcoin-core-dev
1552016-12-31T23:31:55  *** Guyver2 has quit IRC