12017-05-02T00:07:15  *** jcliff42 has joined #bitcoin-core-dev
  22017-05-02T00:17:40  *** jcliff42 has quit IRC
  32017-05-02T00:21:14  *** Ylbam has quit IRC
  42017-05-02T00:22:08  *** laurentmt has joined #bitcoin-core-dev
  52017-05-02T00:22:58  *** laurentmt has quit IRC
  62017-05-02T00:24:43  *** tw2006 has joined #bitcoin-core-dev
  72017-05-02T00:29:20  *** tw2006 has quit IRC
  82017-05-02T00:31:57  <bitcoin-git> [bitcoin] kallewoof closed pull request #10304: [rpc] Allow a txid param in getrawmempool (master...getrawmempool-include-txid) https://github.com/bitcoin/bitcoin/pull/10304
  92017-05-02T00:34:05  *** jcliff42 has joined #bitcoin-core-dev
 102017-05-02T00:34:45  *** jcliff42 has joined #bitcoin-core-dev
 112017-05-02T00:37:55  *** dclxvi has joined #bitcoin-core-dev
 122017-05-02T00:42:15  *** DCLXVI` has quit IRC
 132017-05-02T00:43:09  <bitcoin-git> [bitcoin] kallewoof opened pull request #10310: [doc] Add hint about getmempoolentry to getrawmempool help. (master...doc-getrawmempool-getmempoolentry) https://github.com/bitcoin/bitcoin/pull/10310
 142017-05-02T01:08:57  *** jcliff42 has quit IRC
 152017-05-02T01:09:11  *** jcliff42 has joined #bitcoin-core-dev
 162017-05-02T01:20:00  *** jcliff42 has quit IRC
 172017-05-02T01:24:27  *** tripleslash has quit IRC
 182017-05-02T01:41:05  *** tripleslash has joined #bitcoin-core-dev
 192017-05-02T01:46:42  *** altoz_ has joined #bitcoin-core-dev
 202017-05-02T01:46:58  *** altoz has quit IRC
 212017-05-02T01:51:55  *** talmai has quit IRC
 222017-05-02T02:00:48  *** belcher has quit IRC
 232017-05-02T02:09:56  *** jcliff42 has joined #bitcoin-core-dev
 242017-05-02T02:13:35  *** tw2006 has joined #bitcoin-core-dev
 252017-05-02T02:18:10  *** tw2006 has quit IRC
 262017-05-02T02:30:04  *** chjj has quit IRC
 272017-05-02T02:30:22  *** goksinen has joined #bitcoin-core-dev
 282017-05-02T02:44:11  *** chjj has joined #bitcoin-core-dev
 292017-05-02T02:45:14  *** talmai has joined #bitcoin-core-dev
 302017-05-02T02:52:43  *** jcliff42 has quit IRC
 312017-05-02T03:08:37  *** goksinen has quit IRC
 322017-05-02T04:02:35  *** tw2006 has joined #bitcoin-core-dev
 332017-05-02T04:07:16  *** tw2006 has quit IRC
 342017-05-02T05:12:56  *** fanquake has joined #bitcoin-core-dev
 352017-05-02T05:14:05  <fanquake> cfields Can you take a look another the zeromq changes when you get a chance. I've played around with defining D_WIN32_WINNT=0x0600, but obviously I'm not doing something quite right. Will PR other depends changes today as well.
 362017-05-02T05:44:35  *** cryptapus_afk has quit IRC
 372017-05-02T05:48:46  *** Ylbam has joined #bitcoin-core-dev
 382017-05-02T05:51:32  *** tw2006 has joined #bitcoin-core-dev
 392017-05-02T05:53:41  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e4bbd3d230f2...0e8499c53f9e
 402017-05-02T05:53:41  <bitcoin-git> bitcoin/master c26655e Wladimir J. van der Laan: doc: Add RPC interface guidelines
 412017-05-02T05:53:42  <bitcoin-git> bitcoin/master 0e8499c Wladimir J. van der Laan: Merge #10281: doc: Add RPC interface guidelines...
 422017-05-02T05:54:05  <bitcoin-git> [bitcoin] laanwj closed pull request #10281: doc: Add RPC interface guidelines (master...2017_04_rpc_if_guidelines) https://github.com/bitcoin/bitcoin/pull/10281
 432017-05-02T05:56:05  *** tw2006 has quit IRC
 442017-05-02T06:02:01  *** cryptapus_afk has joined #bitcoin-core-dev
 452017-05-02T06:02:01  *** cryptapus_afk has joined #bitcoin-core-dev
 462017-05-02T06:03:51  *** RubenSomsen has joined #bitcoin-core-dev
 472017-05-02T06:09:52  *** talmai has quit IRC
 482017-05-02T06:20:03  *** SopaXorzTaker has joined #bitcoin-core-dev
 492017-05-02T06:43:21  *** jannes has joined #bitcoin-core-dev
 502017-05-02T06:47:11  *** cryptapus_afk has quit IRC
 512017-05-02T07:25:40  *** cryptapus_afk has joined #bitcoin-core-dev
 522017-05-02T07:25:58  *** cryptapus_afk has joined #bitcoin-core-dev
 532017-05-02T07:31:12  *** EagleTM has joined #bitcoin-core-dev
 542017-05-02T07:40:24  *** tw2006 has joined #bitcoin-core-dev
 552017-05-02T07:44:15  <bitcoin-git> [bitcoin] btcdrak opened pull request #10311: Remove unused args from GetFetchFlags() (master...getflags) https://github.com/bitcoin/bitcoin/pull/10311
 562017-05-02T07:45:04  *** tw2006 has quit IRC
 572017-05-02T08:01:06  *** Eliel_ has joined #bitcoin-core-dev
 582017-05-02T08:01:24  *** Eliel has quit IRC
 592017-05-02T08:04:29  *** sturles has quit IRC
 602017-05-02T08:06:25  *** wumpus has quit IRC
 612017-05-02T08:08:36  *** wumpus has joined #bitcoin-core-dev
 622017-05-02T08:11:19  *** timothy has joined #bitcoin-core-dev
 632017-05-02T08:15:02  *** Guest47977 is now known as array
 642017-05-02T09:29:26  *** tw2006 has joined #bitcoin-core-dev
 652017-05-02T09:32:05  *** decoy has quit IRC
 662017-05-02T09:34:20  *** tw2006 has quit IRC
 672017-05-02T09:43:46  *** CodeShark has quit IRC
 682017-05-02T09:43:46  *** mariorz has quit IRC
 692017-05-02T09:43:47  *** NicolasDorier has quit IRC
 702017-05-02T09:43:47  *** trotski2000 has quit IRC
 712017-05-02T09:43:47  *** ibrightly has quit IRC
 722017-05-02T09:43:49  *** wbnns has quit IRC
 732017-05-02T09:45:15  *** mariorz has joined #bitcoin-core-dev
 742017-05-02T09:49:23  *** wbnns has joined #bitcoin-core-dev
 752017-05-02T09:50:23  *** ibrightly has joined #bitcoin-core-dev
 762017-05-02T09:50:44  *** Lauda has quit IRC
 772017-05-02T09:50:58  *** Lauda has joined #bitcoin-core-dev
 782017-05-02T09:55:57  *** dermoth has quit IRC
 792017-05-02T10:15:57  *** CodeShark has joined #bitcoin-core-dev
 802017-05-02T10:16:07  *** NicolasDorier has joined #bitcoin-core-dev
 812017-05-02T10:17:48  *** trotski2000 has joined #bitcoin-core-dev
 822017-05-02T10:20:26  *** goksinen has joined #bitcoin-core-dev
 832017-05-02T10:24:42  *** Ylbam has quit IRC
 842017-05-02T10:27:42  *** BashCo has joined #bitcoin-core-dev
 852017-05-02T10:44:35  *** sturles has joined #bitcoin-core-dev
 862017-05-02T11:07:40  *** BashCo has quit IRC
 872017-05-02T11:08:56  *** BashCo has joined #bitcoin-core-dev
 882017-05-02T11:17:47  *** chartractegg has joined #bitcoin-core-dev
 892017-05-02T11:18:19  *** tw2006 has joined #bitcoin-core-dev
 902017-05-02T11:19:42  *** NewLiberty has joined #bitcoin-core-dev
 912017-05-02T11:23:19  *** tw2006 has quit IRC
 922017-05-02T11:24:23  *** tw2006 has joined #bitcoin-core-dev
 932017-05-02T11:29:35  *** chartractegg has quit IRC
 942017-05-02T11:31:17  *** mol has joined #bitcoin-core-dev
 952017-05-02T11:33:56  *** molz_ has joined #bitcoin-core-dev
 962017-05-02T11:34:40  *** moli_ has quit IRC
 972017-05-02T11:36:35  *** mol has quit IRC
 982017-05-02T11:50:05  *** laurentmt has joined #bitcoin-core-dev
 992017-05-02T11:54:08  *** laurentmt has quit IRC
1002017-05-02T12:38:09  *** Ylbam has joined #bitcoin-core-dev
1012017-05-02T12:49:21  *** afk11 has quit IRC
1022017-05-02T12:49:31  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0e8499c53f9e...e7b90f78f554
1032017-05-02T12:49:31  <bitcoin-git> bitcoin/master 66082e0 practicalswift: [trivial] Fix typos (tempoarily → temporarily, inadvertantly → inadvertently)
1042017-05-02T12:49:32  <bitcoin-git> bitcoin/master e7b90f7 Wladimir J. van der Laan: Merge #10180: [trivial] Fix typos (tempoarily → temporarily, inadvertantly → inadvertently)...
1052017-05-02T12:49:54  <bitcoin-git> [bitcoin] laanwj closed pull request #10180: [trivial] Fix typos (tempoarily → temporarily, inadvertantly → inadvertently) (master...inadvertantly-and-tempoarily) https://github.com/bitcoin/bitcoin/pull/10180
1062017-05-02T12:50:49  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e7b90f78f554...f4a6180607a8
1072017-05-02T12:50:50  <bitcoin-git> bitcoin/master 9a763d4 practicalswift: Remove excess logic.
1082017-05-02T12:50:50  <bitcoin-git> bitcoin/master f4a6180 Wladimir J. van der Laan: Merge #10175: Remove excess logic....
1092017-05-02T12:51:12  <bitcoin-git> [bitcoin] laanwj closed pull request #10175: Remove excess logic. (master...excess-logic) https://github.com/bitcoin/bitcoin/pull/10175
1102017-05-02T12:52:25  *** afk11 has joined #bitcoin-core-dev
1112017-05-02T12:55:31  *** fanquake has quit IRC
1122017-05-02T12:57:46  <bitcoin-git> [bitcoin] jonasschnelli pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f4a6180607a8...a3e756b7d610
1132017-05-02T12:57:46  <bitcoin-git> bitcoin/master 7278537 Jonas Schnelli: [Qt] Don't add arguments of sensitive command to console window
1142017-05-02T12:57:47  <bitcoin-git> bitcoin/master a3e756b Jonas Schnelli: Merge #10093: [Qt] Don't add arguments of sensitive command to console window...
1152017-05-02T12:58:04  <bitcoin-git> [bitcoin] jonasschnelli closed pull request #10093: [Qt] Don't add arguments of sensitive command to console window (master...2017/03/qt_console) https://github.com/bitcoin/bitcoin/pull/10093
1162017-05-02T13:14:02  *** dermoth has joined #bitcoin-core-dev
1172017-05-02T13:24:20  *** talmai has joined #bitcoin-core-dev
1182017-05-02T13:35:57  <jonasschnelli> For the HD restore keypool topup during SyncTransaction: I can I can't block the thread in the middle of SyncTransaction. But halting after the block has been stream through SyncTransaction may results in missing funds.
1192017-05-02T13:36:25  *** shockoo has joined #bitcoin-core-dev
1202017-05-02T13:42:28  *** talmai has quit IRC
1212017-05-02T13:43:50  *** Guyver2 has joined #bitcoin-core-dev
1222017-05-02T13:50:26  *** d_t has joined #bitcoin-core-dev
1232017-05-02T13:50:32  *** laurentmt has joined #bitcoin-core-dev
1242017-05-02T13:52:04  *** RubenSomsen has quit IRC
1252017-05-02T13:55:01  *** d_t has quit IRC
1262017-05-02T13:56:12  <jonasschnelli> Should the wallet really freeze the validation/net process because we can't extend the keypool?
1272017-05-02T13:56:58  <jonasschnelli> What if the wallet just refuses to sync/write best block if it can't topup the keypool and wait until the wallet is unlocked, then topup and call ScanForWalletTransactions?
1282017-05-02T13:57:21  <wumpus> I think only in case of pruning, when there is the risk the blocks will be lost
1292017-05-02T13:57:37  <jonasschnelli> wumpus: Good point.
1302017-05-02T13:58:54  <jonasschnelli> Should the BlockConnected() (it's now the main signal that calls SyncTransaction) allows to halt the validation process? Or do we need another signal into the other direction?
1312017-05-02T13:59:13  <jonasschnelli> I think setting a validation flag directly from wallet is not ideal
1322017-05-02T14:08:29  *** OfficialJustan has quit IRC
1332017-05-02T14:43:02  *** goksinen has quit IRC
1342017-05-02T14:43:05  *** nanotube has quit IRC
1352017-05-02T14:43:34  *** mr_burdell has quit IRC
1362017-05-02T14:43:44  *** NicolasDorier has quit IRC
1372017-05-02T14:43:45  *** ibrightly has quit IRC
1382017-05-02T14:43:56  *** thrasher` has quit IRC
1392017-05-02T14:44:23  *** mariorz has quit IRC
1402017-05-02T14:44:33  *** arubi has quit IRC
1412017-05-02T14:44:35  *** grubles has quit IRC
1422017-05-02T14:44:46  *** helo has quit IRC
1432017-05-02T14:44:51  *** adiabat has quit IRC
1442017-05-02T14:44:56  *** wallet42 has quit IRC
1452017-05-02T14:45:02  *** NicolasDorier has joined #bitcoin-core-dev
1462017-05-02T14:45:22  *** lejitz has quit IRC
1472017-05-02T14:45:22  *** profall has quit IRC
1482017-05-02T14:45:23  *** aceat64 has quit IRC
1492017-05-02T14:45:41  *** [b__b] has quit IRC
1502017-05-02T14:45:45  *** baldur has quit IRC
1512017-05-02T14:46:39  *** ibrightly has joined #bitcoin-core-dev
1522017-05-02T14:46:59  *** mariorz has joined #bitcoin-core-dev
1532017-05-02T14:47:05  *** luke-jr has quit IRC
1542017-05-02T14:47:56  *** twistedline_ has joined #bitcoin-core-dev
1552017-05-02T14:48:04  *** luke-jr has joined #bitcoin-core-dev
1562017-05-02T14:48:05  *** twistedline has quit IRC
1572017-05-02T14:48:08  *** trotski2000 has quit IRC
1582017-05-02T14:49:48  *** arubi has joined #bitcoin-core-dev
1592017-05-02T14:49:54  *** thrasher` has joined #bitcoin-core-dev
1602017-05-02T14:50:09  *** trotski2000 has joined #bitcoin-core-dev
1612017-05-02T14:50:43  *** wallet42 has joined #bitcoin-core-dev
1622017-05-02T14:51:24  *** grubles has joined #bitcoin-core-dev
1632017-05-02T14:51:25  *** grubles has joined #bitcoin-core-dev
1642017-05-02T14:52:04  *** bad_duck has quit IRC
1652017-05-02T14:52:11  *** mr_burdell has joined #bitcoin-core-dev
1662017-05-02T14:53:33  *** helo has joined #bitcoin-core-dev
1672017-05-02T14:54:32  *** QBcrusher_ has joined #bitcoin-core-dev
1682017-05-02T14:55:38  *** nanotube has joined #bitcoin-core-dev
1692017-05-02T14:56:03  *** bad_duck has joined #bitcoin-core-dev
1702017-05-02T14:56:09  *** adiabat has joined #bitcoin-core-dev
1712017-05-02T14:56:40  *** aceat64 has joined #bitcoin-core-dev
1722017-05-02T14:56:44  *** baldur has joined #bitcoin-core-dev
1732017-05-02T14:56:49  *** QBcrusher has quit IRC
1742017-05-02T15:00:31  <bitcoin-git> [bitcoin] instagibbs opened pull request #10313: [Consensus] Add constant for maximum stack size (master...stackconst) https://github.com/bitcoin/bitcoin/pull/10313
1752017-05-02T15:13:37  *** [b__b] has joined #bitcoin-core-dev
1762017-05-02T15:19:25  <bitcoin-git> [bitcoin] practicalswift opened pull request #10314: Remove unused forward declaration for non-existent ScriptPubKeyToJSON(...) (master...remove-unused-function) https://github.com/bitcoin/bitcoin/pull/10314
1772017-05-02T15:44:11  *** BashCo has quit IRC
1782017-05-02T15:53:56  *** abpa has joined #bitcoin-core-dev
1792017-05-02T16:42:39  <bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/a3e756b7d610...faf2dea5ea81
1802017-05-02T16:42:40  <bitcoin-git> bitcoin/master 77c54b2 John Newbery: [net] listbanned RPC and QT should show correct banned subnets
1812017-05-02T16:42:40  <bitcoin-git> bitcoin/master d6732d8 John Newbery: [tests] update disconnect_ban.py test case to work with listbanned
1822017-05-02T16:42:41  <bitcoin-git> bitcoin/master faf2dea Wladimir J. van der Laan: Merge #10234: [net] listbanned RPC and QT should show correct banned subnets...
1832017-05-02T16:43:03  <bitcoin-git> [bitcoin] laanwj closed pull request #10234: [net] listbanned RPC and QT should show correct banned subnets (master...list_banned_correctly) https://github.com/bitcoin/bitcoin/pull/10234
1842017-05-02T16:43:55  *** lejitz has joined #bitcoin-core-dev
1852017-05-02T16:47:23  *** paveljanik has joined #bitcoin-core-dev
1862017-05-02T16:47:23  *** paveljanik has joined #bitcoin-core-dev
1872017-05-02T16:49:51  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/faf2dea5ea81...1d75597ba6e8
1882017-05-02T16:49:51  <bitcoin-git> bitcoin/master 661caf8 fanquake: [doc] Minor corrections to osx dependencies
1892017-05-02T16:49:52  <bitcoin-git> bitcoin/master 1d75597 Wladimir J. van der Laan: Merge #10260: [doc] Minor corrections to osx dependencies...
1902017-05-02T16:50:11  *** laurentmt has quit IRC
1912017-05-02T16:50:19  <bitcoin-git> [bitcoin] laanwj closed pull request #10260: [doc] Minor corrections to osx dependencies (master...osx-doc-update) https://github.com/bitcoin/bitcoin/pull/10260
1922017-05-02T16:54:58  *** profall has joined #bitcoin-core-dev
1932017-05-02T16:57:23  *** Sosumi has joined #bitcoin-core-dev
1942017-05-02T17:00:49  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1d75597ba6e8...2580ff81f442
1952017-05-02T17:00:50  <bitcoin-git> bitcoin/master 3fe8a83 practicalswift: Remove unused forward declaration for non-existent ScriptPubKeyToJSON(...)
1962017-05-02T17:00:50  *** timothy has quit IRC
1972017-05-02T17:00:50  <bitcoin-git> bitcoin/master 2580ff8 Wladimir J. van der Laan: Merge #10314: Remove unused forward declaration for non-existent ScriptPubKeyToJSON(...)...
1982017-05-02T17:01:21  <bitcoin-git> [bitcoin] laanwj closed pull request #10314: Remove unused forward declaration for non-existent ScriptPubKeyToJSON(...) (master...remove-unused-function) https://github.com/bitcoin/bitcoin/pull/10314
1992017-05-02T17:02:01  *** fronti has left #bitcoin-core-dev
2002017-05-02T17:02:57  *** laurentmt has joined #bitcoin-core-dev
2012017-05-02T17:04:06  *** iroot-servers has joined #bitcoin-core-dev
2022017-05-02T17:12:36  <bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/2580ff81f442...8f3e38477e30
2032017-05-02T17:12:37  <bitcoin-git> bitcoin/master 52e15aa John Newbery: Adds helper functions to NodeConnCB...
2042017-05-02T17:12:38  <bitcoin-git> bitcoin/master 2a52ae6 John Newbery: Remove duplicate method definitions in NodeConnCB subclasses...
2052017-05-02T17:12:38  <bitcoin-git> bitcoin/master 8f3e384 Wladimir J. van der Laan: Merge #10169: [tests] Remove func test code duplication...
2062017-05-02T17:15:16  *** altoz_ is now known as altoz
2072017-05-02T17:20:38  *** jannes has quit IRC
2082017-05-02T17:25:40  *** laurentmt has quit IRC
2092017-05-02T17:38:14  *** laurentmt has joined #bitcoin-core-dev
2102017-05-02T17:39:53  *** SopaXorzTaker has quit IRC
2112017-05-02T17:41:17  *** SopaXorzTaker has joined #bitcoin-core-dev
2122017-05-02T17:47:07  *** goksinen has joined #bitcoin-core-dev
2132017-05-02T17:48:32  *** goksinen has quit IRC
2142017-05-02T17:57:24  *** hashmynonce has joined #bitcoin-core-dev
2152017-05-02T17:59:09  *** BashCo has joined #bitcoin-core-dev
2162017-05-02T17:59:11  *** SopaXorzTaker has quit IRC
2172017-05-02T18:07:08  <bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8f3e38477e30...dc8fc0c73beb
2182017-05-02T18:07:09  <bitcoin-git> bitcoin/master c9cc76d John Newbery: [tests] Remove is_network_split from funtional test cases
2192017-05-02T18:07:09  <bitcoin-git> bitcoin/master dc8fc0c MarcoFalke: Merge #10198: [tests] Remove is_network_split from functional test framework...
2202017-05-02T18:07:30  <bitcoin-git> [bitcoin] MarcoFalke closed pull request #10198: [tests] Remove is_network_split from functional test framework (master...remove_is_network_split) https://github.com/bitcoin/bitcoin/pull/10198
2212017-05-02T18:15:44  <bitcoin-git> [bitcoin] parazyd opened pull request #10316: fix contrib/debian builds; prefer qt5 (master...debian-packaging) https://github.com/bitcoin/bitcoin/pull/10316
2222017-05-02T18:15:58  *** iroot-servers has quit IRC
2232017-05-02T18:20:45  *** hashmynonce has quit IRC
2242017-05-02T18:21:05  <bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/dc8fc0c73beb...3c5e6c94caf4
2252017-05-02T18:21:05  <bitcoin-git> bitcoin/master dadfee3 Jimmy Song: [test] Add test for listaddressgroupings...
2262017-05-02T18:21:06  <bitcoin-git> bitcoin/master 3c5e6c9 MarcoFalke: Merge #10255: [test] Add test for listaddressgroupings...
2272017-05-02T18:21:30  <bitcoin-git> [bitcoin] MarcoFalke closed pull request #10255: [test] Add test for listaddressgroupings (master...test_listaddressgroupings) https://github.com/bitcoin/bitcoin/pull/10255
2282017-05-02T18:23:05  *** bsm1175321 has joined #bitcoin-core-dev
2292017-05-02T18:25:51  <BlueMatt> cfields: yo
2302017-05-02T18:35:36  *** EagleTM has left #bitcoin-core-dev
2312017-05-02T18:40:33  *** laurentmt has quit IRC
2322017-05-02T18:44:17  <cfields> BlueMatt: pong
2332017-05-02T18:53:47  <bitcoin-git> [bitcoin] MarcoFalke pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/3c5e6c94caf4...75171f099e82
2342017-05-02T18:53:48  <bitcoin-git> bitcoin/master b8d9a86 practicalswift: Remove unused import
2352017-05-02T18:53:48  <bitcoin-git> bitcoin/master 73b3721 practicalswift: Remove accidental trailing semicolons
2362017-05-02T18:53:49  <bitcoin-git> bitcoin/master 75171f0 MarcoFalke: Merge #10137: Remove unused import. Remove accidental trailing semicolons....
2372017-05-02T18:54:12  <bitcoin-git> [bitcoin] MarcoFalke closed pull request #10137: Remove unused import. Remove accidental trailing semicolons. (master...python-20170402) https://github.com/bitcoin/bitcoin/pull/10137
2382017-05-02T18:54:54  *** [Author] has quit IRC
2392017-05-02T18:58:36  *** [Author] has joined #bitcoin-core-dev
2402017-05-02T18:59:32  *** tw2006 has quit IRC
2412017-05-02T19:23:04  <bitcoin-git> [bitcoin] practicalswift opened pull request #10317: Remove unused Python imports (master...remove-unused-python-imports-ii) https://github.com/bitcoin/bitcoin/pull/10317
2422017-05-02T19:25:50  *** shockoo has quit IRC
2432017-05-02T19:26:14  *** Madars has joined #bitcoin-core-dev
2442017-05-02T19:26:29  *** nemgun has joined #bitcoin-core-dev
2452017-05-02T19:32:36  *** tw2006 has joined #bitcoin-core-dev
2462017-05-02T19:33:36  *** dermoth has quit IRC
2472017-05-02T19:43:26  *** nemgun has quit IRC
2482017-05-02T19:54:12  <BlueMatt> cfields: nvm, see review on #10285
2492017-05-02T19:54:14  <gribble> https://github.com/bitcoin/bitcoin/issues/10285 | net: refactor the connection process. moving towards async connections. by theuni · Pull Request #10285 · bitcoin/bitcoin · GitHub
2502017-05-02T20:07:20  *** anthonyjd has joined #bitcoin-core-dev
2512017-05-02T20:14:26  <BlueMatt> cfields: yea, #10285 is super annoying because it seems to be a step in the wrong direction by itself
2522017-05-02T20:14:27  <BlueMatt> :/
2532017-05-02T20:14:27  <gribble> https://github.com/bitcoin/bitcoin/issues/10285 | net: refactor the connection process. moving towards async connections. by theuni · Pull Request #10285 · bitcoin/bitcoin · GitHub
2542017-05-02T20:15:10  <cfields> BlueMatt: yea, understood. I kinda expected that type of feedback :(
2552017-05-02T20:15:38  <cfields> BlueMatt: the next commits make things happen in callbacks, so the flow needs to be broken up. But I'm afraid that's just too much for one PR.
2562017-05-02T20:15:40  <BlueMatt> yea, i mean im not gonna nack it because i know you're going somewhere, but maybe open the next pr to get concept acks before this one gets merged?
2572017-05-02T20:15:57  <BlueMatt> (which is apparently what we're all doing now)
2582017-05-02T20:16:26  <cfields> heh, yea
2592017-05-02T20:16:38  <cfields> ok, I can do that.
2602017-05-02T20:18:30  *** Sosumi has quit IRC
2612017-05-02T20:37:04  <bitcoin-git> [bitcoin] jnewbery opened pull request #10318: [tests] fix wait_for_inv() (master...fix_wait_for_inv) https://github.com/bitcoin/bitcoin/pull/10318
2622017-05-02T20:43:27  *** marcoagner has quit IRC
2632017-05-02T20:57:01  *** marcoagner has joined #bitcoin-core-dev
2642017-05-02T20:58:22  *** Lauda has quit IRC
2652017-05-02T20:59:45  *** Lauda has joined #bitcoin-core-dev
2662017-05-02T21:02:25  <bitcoin-git> [bitcoin] practicalswift opened pull request #10319: Remove unused argument from MarkBlockAsInFlight(...) (master...remove-unused-argument) https://github.com/bitcoin/bitcoin/pull/10319
2672017-05-02T21:09:41  *** instagibbs has quit IRC
2682017-05-02T21:17:24  *** instagibbs has joined #bitcoin-core-dev
2692017-05-02T21:40:00  *** cryptapus_afk has quit IRC
2702017-05-02T21:52:17  *** Guyver2 has quit IRC
2712017-05-02T21:52:59  *** cryptapus has joined #bitcoin-core-dev
2722017-05-02T21:52:59  *** cryptapus has joined #bitcoin-core-dev
2732017-05-02T21:53:03  *** cryptapus is now known as cryptapus_afk
2742017-05-02T21:53:33  *** questioner2 has joined #bitcoin-core-dev
2752017-05-02T21:53:42  <questioner2> <questioner> ok guys. imagine its december 2017. segwit has activated. blocks are being made to segwit specification. all the major nodes are relaying and accepting to the consensus and policy of 0.13.2+.. now then. we have a legacy user making a legacy transaction. is the network and pool nodes only allowing 4k sigops (20k/5) or 16k (80k/5) for the legacy tx. can someone show me where the (20k/5) still exists as a rule
2762017-05-02T21:55:17  <luke-jr> questioner2: sigops in legacy scripts count 4x toward the 80k limit
2772017-05-02T21:56:21  <luke-jr> this is in validation.cpp:GetTransactionSigOpCost
2782017-05-02T21:56:39  <questioner2> so a legacy tx can make 4 tx of 20k each
2792017-05-02T22:01:11  *** Naphex has quit IRC
2802017-05-02T22:01:24  *** Naphex has joined #bitcoin-core-dev
2812017-05-02T22:10:05  <questioner2> ok after trying to read what you say. the consensus/policy is 80k/5 for tx sigops = 16k. but legacy tx has 4x of the 16k = 4k
2822017-05-02T22:11:09  <questioner2> so just 20 tx of 4k legacy can use up all the blocks sigops
2832017-05-02T22:11:14  <sipa> questioner2: the easiest way to see it is that from the point of view of an old wallet, nothing changed
2842017-05-02T22:11:27  <sipa> 20k legacy sigops were allowed before, 20k legacy sigops are allowed after
2852017-05-02T22:12:45  *** NewLiberty has quit IRC
2862017-05-02T22:14:21  <questioner2> is using up all 20k legacy treated as using up all 80k block sigops. or treated as legacy tx only use 25% of blocksigop limits allowing segwit transactions to still have 60x sigops spare to use in the base block
2872017-05-02T22:14:42  <questioner2> 60k*
2882017-05-02T22:14:54  <sipa> a legacy sigop counts as 4 segwit sigops
2892017-05-02T22:15:03  <sipa> so 20k legacy sigops would fill a block
2902017-05-02T22:15:56  <questioner2> so 5 legacy tx of 4k sigops would fill the block not allowing segwit txdata to sit in base block
2912017-05-02T22:16:48  <questioner2> 4k sigops each* (=20k legacy)
2922017-05-02T22:17:13  <sipa> just as 1M of non-segwit transactions would fill the whole block, not leaving space for segwit transactions
2932017-05-02T22:17:19  <sipa> *1M bytes
2942017-05-02T22:17:24  *** vicenteH has quit IRC
2952017-05-02T22:20:04  *** NewLiberty has joined #bitcoin-core-dev
2962017-05-02T22:20:56  <questioner2> i understand data spam of 1m ;egacy in base block can stop segwit tx's as one spam attack vector. i am just looking at other attack vectors using sigops where it doesnt need 1m of data, just a certain number of sigops to hold up possible capcity utility
2972017-05-02T22:33:35  *** cryptapus_afk has quit IRC
2982017-05-02T22:35:58  *** cryptapus has joined #bitcoin-core-dev
2992017-05-02T22:38:08  *** justanotheruser has joined #bitcoin-core-dev
3002017-05-02T22:39:56  *** cryptapus is now known as cryptapus_afk
3012017-05-02T22:48:14  *** marcoagner has quit IRC
3022017-05-02T22:51:35  *** _anthony_ has joined #bitcoin-core-dev
3032017-05-02T22:51:38  *** anthonyjd has quit IRC
3042017-05-02T22:55:04  *** marcoagner has joined #bitcoin-core-dev
3052017-05-02T23:01:52  <bitcoin-git> [bitcoin] sipa closed pull request #10299: Remove OpenSSL (master...simplerandom) https://github.com/bitcoin/bitcoin/pull/10299
3062017-05-02T23:03:24  <bitcoin-git> [bitcoin] sipa opened pull request #10321: Use FastRandomContext for all tests (master...fast_rand_tests) https://github.com/bitcoin/bitcoin/pull/10321
3072017-05-02T23:35:07  *** PRab has quit IRC
3082017-05-02T23:45:57  *** owowo has quit IRC
3092017-05-02T23:51:57  *** owowo has joined #bitcoin-core-dev
3102017-05-02T23:58:00  *** abpa has quit IRC
3112017-05-02T23:58:40  *** owowo has quit IRC