12017-01-10T00:03:01  *** d9b4bef9 has quit IRC
  22017-01-10T00:04:08  *** d9b4bef9 has joined #bitcoin-core-dev
  32017-01-10T00:28:51  *** chjj has quit IRC
  42017-01-10T00:38:05  *** blkdb has quit IRC
  52017-01-10T00:38:25  *** blkdb has joined #bitcoin-core-dev
  62017-01-10T00:43:04  *** Saucery has joined #bitcoin-core-dev
  72017-01-10T00:43:31  *** chjj has joined #bitcoin-core-dev
  82017-01-10T00:47:51  *** chjj has quit IRC
  92017-01-10T00:59:32  *** LeMiner2 has joined #bitcoin-core-dev
 102017-01-10T01:01:16  *** chjj has joined #bitcoin-core-dev
 112017-01-10T01:01:46  *** LeMiner has quit IRC
 122017-01-10T01:01:46  *** LeMiner2 is now known as LeMiner
 132017-01-10T01:01:53  *** abpa has quit IRC
 142017-01-10T01:37:28  *** Ylbam has quit IRC
 152017-01-10T01:37:48  *** Saucery has quit IRC
 162017-01-10T01:40:11  *** abpa has joined #bitcoin-core-dev
 172017-01-10T02:03:14  *** abpa has quit IRC
 182017-01-10T02:25:18  <bitcoin-git> [bitcoin] achow101 opened pull request #9500: [Qt][RPC] Autocomplete commands for 'help' command in debug console (master...help-rpc-autocomplete) https://github.com/bitcoin/bitcoin/pull/9500
 192017-01-10T03:14:50  *** wasi has quit IRC
 202017-01-10T03:34:49  *** jtimon has quit IRC
 212017-01-10T04:20:21  *** chris2000 has joined #bitcoin-core-dev
 222017-01-10T04:23:40  *** chris200_ has quit IRC
 232017-01-10T05:11:52  <Lightsword> what conditions would cause me to hit this inconclusive for submitblock? https://github.com/bitcoin/bitcoin/blob/0.13/src/rpc/mining.cpp#L776
 242017-01-10T05:21:24  <luke-jr> Lightsword: block submitted is not on the longest chain, and therefore was not fully verified
 252017-01-10T05:21:39  <luke-jr> eg, stale block
 262017-01-10T05:24:49  <Lightsword> luke-jr, so it’s possible to hit that when doing a duplicate submit right?
 272017-01-10T05:25:20  <luke-jr> shouldn't be. duplicate is explicitly detected
 282017-01-10T05:25:44  <Lightsword> luke-jr, that’s what I thought…but seems you can get that from a duplicate…probably timing dependent
 292017-01-10T05:26:02  <luke-jr> oh, maybe if the duplicate is of an inconclusive block?
 302017-01-10T05:26:08  <luke-jr> eg, the first submit was inconclusive
 312017-01-10T05:26:41  <Lightsword> actually in this case the block first came in over p2p and was then submitted over the RPC at almost the exact same time
 322017-01-10T05:27:28  <luke-jr> that doesn't change much
 332017-01-10T05:27:54  <Lightsword> hmm, looks like the block came in over p2p in the middle of submitblock actually
 342017-01-10T05:29:26  <phantomcircuit> Lightsword, duplicate-inconclusive is a thing
 352017-01-10T05:30:12  <Lightsword> phantomcircuit, well duplicate-inconclusive was not what submit block returned, it returned a plain “inconclusive”
 362017-01-10T05:31:31  <phantomcircuit> that means the block wasn't already seen was accepted but is not in the main chain
 372017-01-10T05:34:01  <Lightsword> phantomcircuit, so that response is given after being accepted but before updatetip?
 382017-01-10T05:36:32  <phantomcircuit> Lightsword, a block can be accepted even if it doesn't update the tip iirc
 392017-01-10T05:37:24  <profall> I have an issue where my bitcoin core daemon goes out of sync for no reason at all. Plenty of peers and in a proper datacenter. It's a server with an E3 processor and 16GB of Ram, not a resource isssue.
 402017-01-10T05:38:23  <profall> It'll run for 24 or 36 hours on the correct block if you compare it in real-time to blockchain.info or wherever, then randomly it'll just stop syncing.
 412017-01-10T05:39:52  <phantomcircuit> profall, what version?
 422017-01-10T05:40:25  <profall> bitcoin-0.13.2-x86_64-linux-gnu.tar.gz right off the website.
 432017-01-10T05:40:59  <profall> (obviously extracted it...)
 442017-01-10T05:41:46  <phantomcircuit> bitcoin-cli getinfo how many peers?
 452017-01-10T05:42:12  <profall> 69
 462017-01-10T05:47:09  <phantomcircuit> profall, how far behind?
 472017-01-10T05:47:40  <profall> It'll go 50 blocks behind sometimes until I manually intervene. I simply stop bitcoind and start it again.
 482017-01-10T05:47:48  <profall> It'll resync right away.
 492017-01-10T05:48:25  <phantomcircuit> hmm
 502017-01-10T05:48:51  <profall> I mean, maybe the datacenter is having connection issues? It's like the only thing I can think of I guess ill put in a ticket.
 512017-01-10T05:49:22  <profall> Everything else works fine though on that server, so I feel like it's not related to the internet connection.
 522017-01-10T05:52:02  <phantomcircuit> uh
 532017-01-10T05:52:07  <phantomcircuit> is this by chance in the bay area
 542017-01-10T05:52:12  <profall> No
 552017-01-10T05:54:59  <phantomcircuit> BlueMatt, ^
 562017-01-10T05:56:28  <phantomcircuit> profall, if you run getpeerinfo is there one peer with a bunch of blocks in flight?
 572017-01-10T06:12:36  <Lightsword> phantomcircuit, so I guess when parsing submitblock responses inconclusive can potentially mean duplicate and that the tip just hasn’t updated yet?
 582017-01-10T06:15:15  *** wasi has joined #bitcoin-core-dev
 592017-01-10T06:18:51  *** dgenr8 has quit IRC
 602017-01-10T06:19:06  *** dgenr8 has joined #bitcoin-core-dev
 612017-01-10T06:22:23  <gmaxwell> profall: that is very interesting and unusual. Can you check the time on that host (including timezone)?  Do you have a debug log covering one of these events?
 622017-01-10T06:51:05  *** windsok has quit IRC
 632017-01-10T06:52:31  *** tunafizz has quit IRC
 642017-01-10T07:02:30  *** Ylbam has joined #bitcoin-core-dev
 652017-01-10T07:10:04  *** tunafizz has joined #bitcoin-core-dev
 662017-01-10T07:29:28  *** kadoban has quit IRC
 672017-01-10T07:29:37  *** windsok has joined #bitcoin-core-dev
 682017-01-10T07:29:55  *** kadoban has joined #bitcoin-core-dev
 692017-01-10T08:02:59  *** kadoban has quit IRC
 702017-01-10T08:03:24  *** kadoban has joined #bitcoin-core-dev
 712017-01-10T08:07:01  *** BashCo has quit IRC
 722017-01-10T08:07:38  *** BashCo has joined #bitcoin-core-dev
 732017-01-10T08:12:02  *** BashCo has quit IRC
 742017-01-10T08:28:43  *** BashCo has joined #bitcoin-core-dev
 752017-01-10T08:47:24  *** MarcoFalke has joined #bitcoin-core-dev
 762017-01-10T08:49:06  *** justanotheruser has quit IRC
 772017-01-10T08:58:27  *** Guyver2 has joined #bitcoin-core-dev
 782017-01-10T08:59:47  *** justanotheruser has joined #bitcoin-core-dev
 792017-01-10T09:04:19  *** jannes has joined #bitcoin-core-dev
 802017-01-10T09:08:25  *** paveljanik has quit IRC
 812017-01-10T09:15:44  <bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/07fd147b9f12...98c80e374b84
 822017-01-10T09:15:45  <bitcoin-git> bitcoin/master 7df5e38 Pavel Janík: Rename lambda argument name to prevent shadowing.
 832017-01-10T09:15:45  <bitcoin-git> bitcoin/master 98c80e3 MarcoFalke: Merge #9496: Rename lambda argument name to prevent shadowing...
 842017-01-10T09:16:00  <bitcoin-git> [bitcoin] MarcoFalke closed pull request #9496: Rename lambda argument name to prevent shadowing (master...20170109_Wshadow_rpcconsole_lambda) https://github.com/bitcoin/bitcoin/pull/9496
 852017-01-10T09:35:19  *** petertodd has quit IRC
 862017-01-10T09:37:17  *** eenoch has quit IRC
 872017-01-10T09:37:18  *** kallewoof has quit IRC
 882017-01-10T09:37:18  *** warren has quit IRC
 892017-01-10T09:37:19  *** lesderid has quit IRC
 902017-01-10T09:37:19  *** thestringpuller has quit IRC
 912017-01-10T09:37:26  *** lesderid has joined #bitcoin-core-dev
 922017-01-10T09:37:30  *** eenoch has joined #bitcoin-core-dev
 932017-01-10T09:37:31  *** thestringpuller has joined #bitcoin-core-dev
 942017-01-10T09:37:34  *** kallewoof has joined #bitcoin-core-dev
 952017-01-10T09:37:54  *** thestringpuller is now known as Guest94281
 962017-01-10T09:38:25  *** warren has joined #bitcoin-core-dev
 972017-01-10T09:38:47  *** Guyver2 has quit IRC
 982017-01-10T09:38:47  *** Squidicuz has quit IRC
 992017-01-10T09:38:48  *** morcos has quit IRC
1002017-01-10T09:38:49  *** amiller has quit IRC
1012017-01-10T09:38:50  *** mr_burdell has quit IRC
1022017-01-10T09:38:56  *** mr_burdell has joined #bitcoin-core-dev
1032017-01-10T09:39:07  *** echonaut has quit IRC
1042017-01-10T09:39:07  *** thermoman has quit IRC
1052017-01-10T09:39:08  *** jeremyrubin has quit IRC
1062017-01-10T09:39:11  *** niska` has quit IRC
1072017-01-10T09:39:14  *** jeremyru1in has joined #bitcoin-core-dev
1082017-01-10T09:39:16  *** thermoman has joined #bitcoin-core-dev
1092017-01-10T09:39:17  *** niska has joined #bitcoin-core-dev
1102017-01-10T09:39:23  *** echonaut has joined #bitcoin-core-dev
1112017-01-10T09:39:49  *** Guest5911 has joined #bitcoin-core-dev
1122017-01-10T09:41:32  *** profall has quit IRC
1132017-01-10T09:42:27  *** LeMiner2 has joined #bitcoin-core-dev
1142017-01-10T09:43:35  *** justanotheruser has quit IRC
1152017-01-10T09:44:00  *** morcos has joined #bitcoin-core-dev
1162017-01-10T09:44:03  *** profall has joined #bitcoin-core-dev
1172017-01-10T09:44:54  *** LeMiner has quit IRC
1182017-01-10T09:44:54  *** LeMiner2 is now known as LeMiner
1192017-01-10T09:55:00  *** justanotheruser has joined #bitcoin-core-dev
1202017-01-10T10:06:32  *** BashCo_ has joined #bitcoin-core-dev
1212017-01-10T10:09:52  *** BashCo has quit IRC
1222017-01-10T10:20:07  *** Guyver2 has joined #bitcoin-core-dev
1232017-01-10T10:49:04  *** petertodd has joined #bitcoin-core-dev
1242017-01-10T11:09:15  *** laurentmt has joined #bitcoin-core-dev
1252017-01-10T11:10:08  *** laurentmt has quit IRC
1262017-01-10T11:30:22  *** emmmork has quit IRC
1272017-01-10T11:58:36  *** BashCo has joined #bitcoin-core-dev
1282017-01-10T12:01:35  *** BashCo_ has quit IRC
1292017-01-10T12:08:37  *** emmmork has joined #bitcoin-core-dev
1302017-01-10T12:35:10  <bitcoin-git> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/68eb56203be17066af4e37837703490af4d4f40c
1312017-01-10T12:35:10  <bitcoin-git> bitcoin/master 68eb562 Wladimir J. van der Laan: qt: periodic translations update
1322017-01-10T12:37:24  *** LeMiner2 has joined #bitcoin-core-dev
1332017-01-10T12:39:54  *** LeMiner has quit IRC
1342017-01-10T12:39:54  *** LeMiner2 is now known as LeMiner
1352017-01-10T12:45:16  *** AaronvanW has joined #bitcoin-core-dev
1362017-01-10T12:45:17  *** AaronvanW has quit IRC
1372017-01-10T12:45:17  *** AaronvanW has joined #bitcoin-core-dev
1382017-01-10T13:14:29  *** wump is now known as wumpus
1392017-01-10T13:15:15  <bitcoin-git> [bitcoin] laanwj pushed 14 new commits to master: https://github.com/bitcoin/bitcoin/compare/68eb56203be1...5754e0341b7c
1402017-01-10T13:15:16  <bitcoin-git> bitcoin/master 5865d41 Wladimir J. van der Laan: authproxy: Add support for RPC named arguments
1412017-01-10T13:15:16  <bitcoin-git> bitcoin/master 6f1c76a Wladimir J. van der Laan: rpc: Support named arguments...
1422017-01-10T13:15:17  <bitcoin-git> bitcoin/master 495eb44 Wladimir J. van der Laan: rpc: Named arguments for blockchain calls
1432017-01-10T13:24:32  *** chjj has quit IRC
1442017-01-10T13:26:32  *** Chris_Stewart_5 has quit IRC
1452017-01-10T13:41:02  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1462017-01-10T13:41:20  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1472017-01-10T13:49:56  *** jtimon has joined #bitcoin-core-dev
1482017-01-10T13:55:39  *** Sosumi has joined #bitcoin-core-dev
1492017-01-10T13:57:56  *** chjj has joined #bitcoin-core-dev
1502017-01-10T14:11:29  *** Chris_Stewart_5 has quit IRC
1512017-01-10T14:27:25  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1522017-01-10T14:29:35  *** wasi has quit IRC
1532017-01-10T14:30:00  *** wasi has joined #bitcoin-core-dev
1542017-01-10T14:31:38  *** wasi has quit IRC
1552017-01-10T14:32:01  *** wasi has joined #bitcoin-core-dev
1562017-01-10T14:48:06  *** Chris_Stewart_5 has quit IRC
1572017-01-10T15:00:46  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1582017-01-10T15:13:53  *** Chris_Stewart_5 has quit IRC
1592017-01-10T15:27:18  *** BashCo has quit IRC
1602017-01-10T15:28:04  *** BashCo has joined #bitcoin-core-dev
1612017-01-10T15:28:07  *** Squidicuz has joined #bitcoin-core-dev
1622017-01-10T15:29:51  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1632017-01-10T15:32:17  *** BashCo has quit IRC
1642017-01-10T15:33:32  <jonasschnelli> instagibbs: can you help me parsing your comment... :)
1652017-01-10T15:33:32  <jonasschnelli> https://github.com/bitcoin/bitcoin/pull/9294/files#r95169601
1662017-01-10T15:33:35  <jonasschnelli> (or anyone)
1672017-01-10T15:33:43  <jonasschnelli> I don't understand it
1682017-01-10T15:35:08  <instagibbs> jonasschnelli, in one commit you go from 0 internal to 50/50 external/internal, to 100/20
1692017-01-10T15:35:13  <instagibbs> in the next
1702017-01-10T15:35:21  <instagibbs> I just find that a bit odd
1712017-01-10T15:35:30  <jonasschnelli> Ah. The commit order...
1722017-01-10T15:35:46  <jonasschnelli> Yes. I force pushed it in a confusing way.
1732017-01-10T15:35:46  <instagibbs> yeah, not blocking, I just got confused when reviewing commit by commit
1742017-01-10T15:36:06  <instagibbs> I think logically it's not broken at any point
1752017-01-10T15:39:23  <jonasschnelli> I think I squash the commits.
1762017-01-10T15:39:32  <jonasschnelli> It's one single change
1772017-01-10T15:51:03  <instagibbs> is #9400 reasonable for 0.14? It's gotten decent amount of review, and is a pretty small change
1782017-01-10T15:51:06  <gribble> https://github.com/bitcoin/bitcoin/issues/9400 | Set peers as HB peers upon full block validation by instagibbs · Pull Request #9400 · bitcoin/bitcoin · GitHub
1792017-01-10T15:54:05  * jonasschnelli invites instagibbs to re-review #9294 (thanks!)
1802017-01-10T15:54:07  <gribble> https://github.com/bitcoin/bitcoin/issues/9294 | Use internal HD chain for change outputs (hd split) by jonasschnelli · Pull Request #9294 · bitcoin/bitcoin · GitHub
1812017-01-10T16:00:22  <instagibbs> well since you asked nicely
1822017-01-10T16:05:47  *** Guest5911 has quit IRC
1832017-01-10T16:05:47  *** Guest5911 has joined #bitcoin-core-dev
1842017-01-10T16:05:47  *** Guest5911 is now known as amiller
1852017-01-10T16:08:44  <jonasschnelli> Thanks for the review instagibbs: I owe you a review now!
1862017-01-10T16:09:21  *** BashCo has joined #bitcoin-core-dev
1872017-01-10T16:09:22  *** xiangfu has quit IRC
1882017-01-10T16:09:51  *** fengling has quit IRC
1892017-01-10T16:19:26  *** fengling has joined #bitcoin-core-dev
1902017-01-10T16:21:53  *** xiangfu has joined #bitcoin-core-dev
1912017-01-10T16:27:34  *** xinxi has joined #bitcoin-core-dev
1922017-01-10T16:27:56  <xinxi> cfields: ping. I have a question about codesigning binaries using gitian.
1932017-01-10T16:28:49  <cfields> xinxi: hi, shoot
1942017-01-10T16:29:50  <xinxi> ./bin/gbuild -i --commit signature=master ../litecoin/contrib/gitian-descriptors/gitian-win-signer.yml
1952017-01-10T16:29:50  <xinxi> --- Building for trusty amd64 ---
1962017-01-10T16:29:51  <xinxi> Stopping target if it is up
1972017-01-10T16:29:52  <xinxi> Starting target
1982017-01-10T16:29:54  <xinxi> Checking if target is up..............................
1992017-01-10T16:29:56  <xinxi> sudo: unknown user: ubuntu
2002017-01-10T16:29:58  <xinxi> sudo: unable to initialize policy plugin
2012017-01-10T16:30:00  <xinxi> ./bin/gbuild:21:in `system!': failed to run on-target true (RuntimeError)
2022017-01-10T16:30:02  <xinxi>     from ./bin/gbuild:73:in `build_one_configuration'
2032017-01-10T16:30:04  <xinxi>     from ./bin/gbuild:285:in `block (2 levels) in <main>'
2042017-01-10T16:30:06  <xinxi>     from ./bin/gbuild:280:in `each'
2052017-01-10T16:30:08  <xinxi>     from ./bin/gbuild:280:in `block in <main>'
2062017-01-10T16:30:10  <xinxi>     from ./bin/gbuild:278:in `each'
2072017-01-10T16:30:12  <xinxi>     from ./bin/gbuild:278:in `<main>'
2082017-01-10T16:30:14  <xinxi> [11:43]
2092017-01-10T16:30:16  <xinxi> I changed ubuntu -> debian
2102017-01-10T16:30:19  <xinxi> I got the above error when I was compiling Litecoin.
2112017-01-10T16:30:20  <cfields> xinxi: ahh, paste somewhere please!
2122017-01-10T16:30:29  <xinxi> oops
2132017-01-10T16:31:03  <cfields> xinxi: need to look at the log, the above doesn't tell much
2142017-01-10T16:32:15  <xinxi> where is the log? no path is shown above.
2152017-01-10T16:32:46  <cfields> xinxi: ./var/build.log
2162017-01-10T16:33:16  <xinxi> there is only var/install.log
2172017-01-10T16:33:23  <xinxi> but it seems not relevant.
2182017-01-10T16:34:03  <achow101> that means that the build hasn't started yet and it was still updating stuff
2192017-01-10T16:34:34  <cfields> xinxi: wait, why did you change ubuntu -> debian?
2202017-01-10T16:34:55  <cfields> xinxi: unknown ubuntu user seems relevant in that case :)
2212017-01-10T16:34:59  <xinxi> I am compiling on debian.
2222017-01-10T16:35:23  <xinxi> let me change back.
2232017-01-10T16:35:35  <cfields> xinxi: doesn't matter, this is creating a vm for you
2242017-01-10T16:36:51  <xinxi> I just changed back and am compiling again. it seems a bit different now.
2252017-01-10T16:37:33  <btcdrak> xinxi: use pastebin/0bin please
2262017-01-10T16:37:50  <xinxi> yeah, just noticed it. sorry about that.
2272017-01-10T16:40:14  <xinxi> cfields: got a new error: http://pastebin.com/JA4rtscY
2282017-01-10T16:42:21  <cfields> xinxi: have you signed the binaries and created detached sigs?
2292017-01-10T16:42:35  <xinxi> yeah, I have.
2302017-01-10T16:43:22  <xinxi> I got the .assert and .assert.sig files.
2312017-01-10T16:43:47  <cfields> xinxi: since this isn't bitcoin related, let's go to pm
2322017-01-10T16:43:58  <instagibbs> 0.13.1 and other NODE_WITNESS only ask for txn from witness peers, right?
2332017-01-10T16:45:16  <sipa> instagibbs: i don't think so, but i forget
2342017-01-10T16:50:46  *** abpa has joined #bitcoin-core-dev
2352017-01-10T16:54:58  <phantomcircuit> cfields, on 9441 the indentation in ba4cae284f6acac4bbbfe08c89dcb8c2ace5da83 is screwed
2362017-01-10T16:55:33  <cfields> phantomcircuit: yea, i left the indentation alone, otherwise the diff would've been a bloodbath
2372017-01-10T16:55:54  <cfields> i'll do a whitespace follow-up
2382017-01-10T16:55:57  <phantomcircuit> cfields, also please use {} for all if statements
2392017-01-10T16:56:09  <phantomcircuit> kind of makes indentation issues less relevant
2402017-01-10T16:56:30  <phantomcircuit> the way it is now it looks like nothing will ever run because of the indentation
2412017-01-10T16:58:00  <cfields> phantomcircuit: fair enough. where in particular?
2422017-01-10T16:59:04  <phantomcircuit> the fDisconnect check mainly
2432017-01-10T16:59:34  <phantomcircuit> also probably swap the fDisconnect and !vRecvGetData.empty() checks since ProcessGetData can set fDisconnect
2442017-01-10T17:00:16  <phantomcircuit> (left these comments on github too)
2452017-01-10T17:01:36  <cfields> phantomcircuit: ah, they make more sense in context. thanks :)
2462017-01-10T17:05:24  <phantomcircuit> cfields, it seems like we should disconnect on checksum and header validation issues actually
2472017-01-10T17:05:32  <phantomcircuit> (we currently ignore them and go to the next message)
2482017-01-10T17:05:55  <cfields> phantomcircuit: there are several things that should change about the behavior there, but i tried to leave as much alone as i could for this PR
2492017-01-10T17:06:04  <phantomcircuit> alright
2502017-01-10T17:06:10  <phantomcircuit> i'll refrain
2512017-01-10T17:06:30  <cfields> phantomcircuit: agreed ofc, btw
2522017-01-10T17:36:14  <bitcoin-git> [bitcoin] jonasschnelli opened pull request #9502: [Qt] Add option to pause/resume block downloads (master...2017/01/autodownload) https://github.com/bitcoin/bitcoin/pull/9502
2532017-01-10T17:54:21  <instagibbs> sipa, oh right, definitely not the case since they wont relay non-std txns in general
2542017-01-10T17:54:52  <sipa> indeed
2552017-01-10T17:57:06  <jtimon> BlueMatt: it seems #9486 could use a better description, the description seems to only talk about the last commit, which seems trivial and overall unrelated to the rest, certainly I'm missing something
2562017-01-10T17:57:08  <gribble> https://github.com/bitcoin/bitcoin/issues/9486 | Make peer=%d log prints consistent by TheBlueMatt · Pull Request #9486 · bitcoin/bitcoin · GitHub
2572017-01-10T17:57:22  <sdaftuar> jtimon: see #9375
2582017-01-10T17:57:25  <gribble> https://github.com/bitcoin/bitcoin/issues/9375 | Relay compact block messages prior to full block connection by TheBlueMatt · Pull Request #9375 · bitcoin/bitcoin · GitHub
2592017-01-10T17:57:42  <sipa> jtimon: from the description "Based on #9375"
2602017-01-10T17:57:46  <gribble> https://github.com/bitcoin/bitcoin/issues/9375 | Relay compact block messages prior to full block connection by TheBlueMatt · Pull Request #9375 · bitcoin/bitcoin · GitHub
2612017-01-10T17:57:51  <sipa> ah, sdaftuar beat me
2622017-01-10T17:58:26  <jtimon> yeah, thanks, reading now, read that and then forgot and tried to exact the leitmotiv from looking at the commits
2632017-01-10T18:01:11  <jtimon> does https://github.com/bitcoin/bitcoin/pull/9486/commits/e6111b2398ca21f0e38333236abb0be7fa48c95f really need to be based on #9375 ? it seems it could be trivially merged independently
2642017-01-10T18:01:13  <gribble> https://github.com/bitcoin/bitcoin/issues/9375 | Relay compact block messages prior to full block connection by TheBlueMatt · Pull Request #9375 · bitcoin/bitcoin · GitHub
2652017-01-10T18:27:03  <bitcoin-git> [bitcoin] JeremyRubin opened pull request #9503: listreceivedbyaddress Filter Address (master...listreceivedbyaddress-filtered) https://github.com/bitcoin/bitcoin/pull/9503
2662017-01-10T18:34:29  *** xinxi has quit IRC
2672017-01-10T18:34:56  *** xinxi has joined #bitcoin-core-dev
2682017-01-10T18:39:13  *** xinxi has quit IRC
2692017-01-10T19:02:25  *** gmaxwell has quit IRC
2702017-01-10T19:08:46  *** xinxi has joined #bitcoin-core-dev
2712017-01-10T19:12:49  *** gmaxwell has joined #bitcoin-core-dev
2722017-01-10T19:18:50  <bitcoin-git> [bitcoin] achow101 opened pull request #9504: [RPC] dumpmasterprivkey command (master...dumpmasterprivkey) https://github.com/bitcoin/bitcoin/pull/9504
2732017-01-10T19:42:07  *** PaulCapestany has quit IRC
2742017-01-10T19:42:44  <BlueMatt> gmaxwell: heh, yea, so extra is worth it as-is, will push the use-less-memory thing for 0.15?
2752017-01-10T19:44:25  <gmaxwell> BlueMatt: seems like it.
2762017-01-10T19:44:30  <BlueMatt> cool
2772017-01-10T19:46:29  *** LeMiner has quit IRC
2782017-01-10T19:48:43  *** GAit has joined #bitcoin-core-dev
2792017-01-10T19:52:24  *** PaulCapestany has joined #bitcoin-core-dev
2802017-01-10T20:01:13  *** justanotheruser has quit IRC
2812017-01-10T20:02:09  <bitcoin-git> [bitcoin] JeremyRubin opened pull request #9505: Prevector Quick Destruct (master...prevector-quick-destruct) https://github.com/bitcoin/bitcoin/pull/9505
2822017-01-10T20:06:31  <bitcoin-git> [bitcoin] sipa opened pull request #9506: RFC: Improve style for if indentation (master...newstyle) https://github.com/bitcoin/bitcoin/pull/9506
2832017-01-10T20:19:22  *** Guest94281 is now known as thestringpuller
2842017-01-10T20:19:29  *** thestringpuller has joined #bitcoin-core-dev
2852017-01-10T20:28:58  *** dermoth_ has joined #bitcoin-core-dev
2862017-01-10T20:29:26  *** dermoth has quit IRC
2872017-01-10T20:29:28  *** dermoth_ is now known as dermoth
2882017-01-10T20:31:35  *** cbits has joined #bitcoin-core-dev
2892017-01-10T20:33:06  <bitcoin-git> [bitcoin] sdaftuar opened pull request #9507: Fix use-after-free in CTxMemPool::removeConflicts() (master...fix-mempool-useafterfree) https://github.com/bitcoin/bitcoin/pull/9507
2902017-01-10T20:34:24  <bitcoin-git> [bitcoin] practicalswift opened pull request #9508: [gardening] Remove unused Python imports (master...remove-unused-python-import) https://github.com/bitcoin/bitcoin/pull/9508
2912017-01-10T20:35:50  *** paveljanik has joined #bitcoin-core-dev
2922017-01-10T20:58:28  <bitcoin-git> [bitcoin] theuni opened pull request #9509: build: fix qt distdir builds (master...out-of-tree-build) https://github.com/bitcoin/bitcoin/pull/9509
2932017-01-10T21:02:21  *** xinxi has quit IRC
2942017-01-10T21:04:59  *** cbits has quit IRC
2952017-01-10T21:07:07  *** Sosumi has quit IRC
2962017-01-10T21:44:57  *** harrymm has joined #bitcoin-core-dev
2972017-01-10T21:48:39  <instagibbs> what's the functional/use difference between CCoinsViewCache::GetCoins and CCoinsViewCache::AccessCoins?
2982017-01-10T21:49:04  <sipa> the latter is much faster
2992017-01-10T21:49:10  <sipa> as it doesn't copy
3002017-01-10T21:49:34  <sipa> so why would you use the first? AccessCoins only exists in CCoinsViewCache, while GetCoins exists in every CCoins implementation
3012017-01-10T21:49:41  <sipa> *every CCoinsView
3022017-01-10T21:49:56  *** waxwing has joined #bitcoin-core-dev
3032017-01-10T21:50:19  <instagibbs> thanks, let me see if that explains its various uses to me
3042017-01-10T21:56:09  <bitcoin-git> [bitcoin] practicalswift opened pull request #9510: [gardening] Fix typos in comments (master...fix-typos-in-comments) https://github.com/bitcoin/bitcoin/pull/9510
3052017-01-10T22:00:41  <bitcoin-git> [bitcoin] morcos opened pull request #9511: Don't overwrite validation state with corruption check (master...ATMPstate) https://github.com/bitcoin/bitcoin/pull/9511
3062017-01-10T22:02:44  *** xinxi has joined #bitcoin-core-dev
3072017-01-10T22:06:03  *** Guyver2 has quit IRC
3082017-01-10T22:10:35  *** molz has quit IRC
3092017-01-10T22:13:14  <gmaxwell> sipa: BlueMatt has a nit on #9472
3102017-01-10T22:13:16  <gribble> https://github.com/bitcoin/bitcoin/issues/9472 | Disentangle progress estimation from checkpoints and update it by sipa · Pull Request #9472 · bitcoin/bitcoin · GitHub
3112017-01-10T22:14:59  <BlueMatt> are we gonna try to merge that for 14 too?
3122017-01-10T22:15:25  <gmaxwell> sure, it's done its ... has about zero potential to break anything.
3132017-01-10T22:15:53  <sipa> it would be nice to say that we're only using checkpoints for avoiding a memory-bloating low-difficulty header attack
3142017-01-10T22:16:01  <sipa> it would be even nicer to say we're not using them at all anymore
3152017-01-10T22:16:19  *** AaronvanW has quit IRC
3162017-01-10T22:16:50  <gmaxwell> yes, well... we could have been there, but
3172017-01-10T22:17:51  <gmaxwell> I don't have a solution for that which doesn't need something which is technically a consensus rule change.
3182017-01-10T22:18:47  <gmaxwell> https://github.com/gmaxwell/bitcoin/commit/09cef642e7d7866cb7b7026a453e1c791c191ddf
3192017-01-10T22:18:55  <gmaxwell> oops wrong commit
3202017-01-10T22:19:02  <gmaxwell> https://github.com/gmaxwell/bitcoin/commit/2db190b183c5204da23191ca642c7f6cad412ae3
3212017-01-10T22:20:29  *** emmmork has quit IRC
3222017-01-10T22:22:04  *** xinxi has quit IRC
3232017-01-10T22:25:57  *** cbits has joined #bitcoin-core-dev
3242017-01-10T23:05:51  *** belcher has quit IRC
3252017-01-10T23:15:29  *** fresn3ll has joined #bitcoin-core-dev
3262017-01-10T23:17:54  <bitcoin-git> [bitcoin] sipa opened pull request #9512: Fix various things -fsanitize complains about (master...sanitize) https://github.com/bitcoin/bitcoin/pull/9512
3272017-01-10T23:19:00  *** belcher has joined #bitcoin-core-dev
3282017-01-10T23:19:01  *** xinxi has joined #bitcoin-core-dev
3292017-01-10T23:21:44  *** NicolasDorier has quit IRC
3302017-01-10T23:22:17  *** chjj has quit IRC
3312017-01-10T23:23:13  *** mappum has quit IRC
3322017-01-10T23:25:56  *** NicolasDorier has joined #bitcoin-core-dev
3332017-01-10T23:25:56  *** mappum has joined #bitcoin-core-dev
3342017-01-10T23:36:10  *** xinxi has quit IRC
3352017-01-10T23:36:56  <dcousens> can prioritisetransaction work on txids that aren't yet in the mempool?
3362017-01-10T23:37:17  <dcousens> I assume no,  in which case,  it might an idea to allow that as an option or maybe allow sendrawtransaction to have a feedelta field?
3372017-01-10T23:37:28  <dcousens> (uh,  a 'virtual fee' field)
3382017-01-10T23:40:03  <sipa> yes, it can
3392017-01-10T23:40:57  <dcousens> sipa: awesome :)
3402017-01-10T23:42:08  <gmaxwell> dcousens: it writes it into mapDeltas.
3412017-01-10T23:48:57  *** fresn3ll has quit IRC
3422017-01-10T23:51:26  *** cbits has quit IRC
3432017-01-10T23:57:10  *** chjj has joined #bitcoin-core-dev