12017-02-01T00:12:18  *** aguycalled has quit IRC
  22017-02-01T00:12:52  *** aguycalled has joined #bitcoin-core-dev
  32017-02-01T00:20:57  <bitcoin-git> [bitcoin] droark opened pull request #9660: Backport “Fix: OSX QT compile: use built-in swap if available, or defer” (0.13...macbackport) https://github.com/bitcoin/bitcoin/pull/9660
  42017-02-01T00:39:23  *** Chris_Stewart_5 has quit IRC
  52017-02-01T00:43:04  *** wasi has quit IRC
  62017-02-01T00:44:04  *** luke-jr has joined #bitcoin-core-dev
  72017-02-01T00:51:30  *** Chris_Stewart_5 has joined #bitcoin-core-dev
  82017-02-01T00:55:21  <Chris_Stewart_5> luke-jr: Do you mean that in the sense that the script will pass since segwit is a soft fork? No segwit script logic will be run?
  92017-02-01T00:55:53  *** wasi has joined #bitcoin-core-dev
 102017-02-01T00:56:38  <sipa> the segwit rules require 1) no witness is present if the spent output is not a witness program
 112017-02-01T00:57:01  <sipa> 2) witness passes witness validation if the spent output is a witness program
 122017-02-01T01:00:20  *** abpa has quit IRC
 132017-02-01T01:03:59  *** MarcoFalke has joined #bitcoin-core-dev
 142017-02-01T01:18:29  <Chris_Stewart_5> sipa: Hmm, so if I give a witness and an OP_1NEGATE as a the wit version I would fail here: https://github.com/Christewart/bitcoin/blob/segwit_invalid_op1negate/src/script/interpreter.cpp#L1511
 152017-02-01T01:18:47  <Chris_Stewart_5> However, if I provided that same program w/ no witness, it would be treated as any old scriptPubKey, correct?
 162017-02-01T01:22:30  *** chjj has quit IRC
 172017-02-01T01:28:02  *** MarcoFalke has quit IRC
 182017-02-01T01:36:30  *** chjj has joined #bitcoin-core-dev
 192017-02-01T02:09:40  *** Giszmo has quit IRC
 202017-02-01T02:19:56  *** Giszmo has joined #bitcoin-core-dev
 212017-02-01T02:28:42  *** harrymm has joined #bitcoin-core-dev
 222017-02-01T02:31:41  <luke-jr> Chris_Stewart_5: I mean it's a valid opcode and even performs a well-defined task. Just don't include any witness data in the input spending it, and it is valid in a block.
 232017-02-01T02:33:25  <Chris_Stewart_5> luke-jr: But in the context that we are talking about it is *not* a valid witness program version. So I think my modified text is correct.
 242017-02-01T02:34:07  <luke-jr> Chris_Stewart_5: I can't view the text right now, but the way it was worded was in the context of a valid *opcode*
 252017-02-01T02:34:55  <Chris_Stewart_5> ... that consists of a select subset of opcodes (<code>OP_0,OP_1,OP_2,...,OP_16</code>) followed by a data push between 2 and 40 bytes gets a new special meaning.
 262017-02-01T02:35:15  <luke-jr> "valid" isn't in that?
 272017-02-01T02:35:56  <Chris_Stewart_5> no
 282017-02-01T02:36:27  * luke-jr is now confused as to the topic of discussion
 292017-02-01T02:36:51  <Chris_Stewart_5> are you talking about about valid op codes or valid witness program version's? I'm talking about the latter.
 302017-02-01T02:37:21  <luke-jr> my objection was the misuse of the term "valid" in the description. But your paste doesn't use "valid" at all, so I don't see the problem
 312017-02-01T02:38:39  <Chris_Stewart_5> Yeah, my text modification doesn't contain the word 'valid' at all.
 322017-02-01T02:39:08  <Chris_Stewart_5> even though I think 'select subset of opcodes' and 'valid opcodes' are semantically the same.. anyway i digress
 332017-02-01T02:40:39  <Chris_Stewart_5> ehhh actually the more I think about it I can see your point.
 342017-02-01T02:42:59  <Chris_Stewart_5> wrt not using the word 'valid'
 352017-02-01T02:55:51  *** aguycalled has quit IRC
 362017-02-01T03:04:10  *** Giszmo has quit IRC
 372017-02-01T03:19:07  *** Giszmo has joined #bitcoin-core-dev
 382017-02-01T04:54:36  *** chris200_ has joined #bitcoin-core-dev
 392017-02-01T04:57:41  *** chris2000 has quit IRC
 402017-02-01T06:07:31  *** Ylbam has quit IRC
 412017-02-01T06:16:48  *** chjj has quit IRC
 422017-02-01T06:50:03  *** lclc has joined #bitcoin-core-dev
 432017-02-01T07:18:48  *** Giszmo has quit IRC
 442017-02-01T07:23:12  *** chjj has joined #bitcoin-core-dev
 452017-02-01T07:25:51  *** Ylbam has joined #bitcoin-core-dev
 462017-02-01T07:38:50  *** Giszmo has joined #bitcoin-core-dev
 472017-02-01T07:43:35  <bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/e99f0d7ad443...7bfb77045c4b
 482017-02-01T07:43:36  <bitcoin-git> bitcoin/master d625b90 Suhas Daftuar: wallet: Refactor dummy signature signing for reusability
 492017-02-01T07:43:36  <bitcoin-git> bitcoin/master f626594 Suhas Daftuar: rpc: bumpfee: use correct maximum signed tx size for fee calculation...
 502017-02-01T07:43:36  <bitcoin-git> bitcoin/master 9522b53 Suhas Daftuar: rpc: bumpfee: handle errors more gracefully
 512017-02-01T07:43:50  <bitcoin-git> [bitcoin] laanwj closed pull request #9640: Bumpfee: bugfixes for error handling and feerate calculation (master...2017-01-bumpfee-error-cleanup) https://github.com/bitcoin/bitcoin/pull/9640
 522017-02-01T08:08:03  *** BashCo has quit IRC
 532017-02-01T08:08:40  *** BashCo has joined #bitcoin-core-dev
 542017-02-01T08:13:28  *** BashCo has quit IRC
 552017-02-01T08:17:53  <jonasschnelli> Fundrawtransaction has the option "includeWatching". Does it make sense to "include" watch-onlys? IMO it should be "onlyUseWatching" (or similar) or at least we should offer this function.
 562017-02-01T08:18:39  <jonasschnelli> Otherwise, if you run a cold-storage wallet via importing addresses/pubkeys, you run into the risk of mixing the cold-storage with hot-keys from the wallet
 572017-02-01T08:19:27  <jonasschnelli> Well, if you don't receive coins over the hot wallet, then you won't mix inputs.. but that's a bad sanity check (via users-behavior)
 582017-02-01T08:21:43  <sipa> well don't mix them into the same wallet then
 592017-02-01T08:21:56  <sipa> that's asking for trouble regardless i think
 602017-02-01T08:22:45  <jonasschnelli> sipa: Yes. But assume you want to use a hardware wallet with a core wallet... you import plenty of addresses, ... but there is no sanity that you "could" mix inputs.
 612017-02-01T08:23:03  <jonasschnelli> IMO it would be sane if fundraw had an option to say "only use watchonly".
 622017-02-01T08:23:16  <jonasschnelli> Just in case you where dump enought to receive coins into your hot wallet
 632017-02-01T08:23:19  <jonasschnelli> dumb
 642017-02-01T08:23:58  <sipa> that's equally dumb to otherwise mixing two other wallets
 652017-02-01T08:24:02  <bitcoin-git> [bitcoin] laanwj pushed 1 new commit to 0.13: https://github.com/bitcoin/bitcoin/commit/59c37ae55a476d3cf84c9bafbc083904472fe4db
 662017-02-01T08:24:03  <bitcoin-git> bitcoin/0.13 59c37ae Karl-Johan Alm: Uses built-in byte swap if available (Apple) and if bswap_XX is undefined....
 672017-02-01T08:24:17  <sipa> why not provide an option for separating an accidentally-mixed wallet out
 682017-02-01T08:24:32  <bitcoin-git> [bitcoin] laanwj closed pull request #9660: Backport “Fix: OSX QT compile: use built-in swap if available, or defer” (0.13...macbackport) https://github.com/bitcoin/bitcoin/pull/9660
 692017-02-01T08:29:02  <jonasschnelli> sipa: Yes. Assume I tell some hardware-wallet users to use watchonlys in order to use core with the HWW.
 702017-02-01T08:29:13  <jonasschnelli> I'm almost certain they will screw up.
 712017-02-01T08:29:36  <jonasschnelli> Maybe an option to disables the hot/private-keys mode in a core wallet could be something... but meh
 722017-02-01T08:30:18  *** BashCo has joined #bitcoin-core-dev
 732017-02-01T08:51:21  *** Guyver2 has joined #bitcoin-core-dev
 742017-02-01T09:07:40  *** isle2983 has quit IRC
 752017-02-01T09:09:51  *** Chris_Stewart_5 has quit IRC
 762017-02-01T09:23:26  *** jannes has joined #bitcoin-core-dev
 772017-02-01T10:18:26  *** MarcoFalke has joined #bitcoin-core-dev
 782017-02-01T10:23:12  *** paveljanik has quit IRC
 792017-02-01T10:33:38  *** aguycalled has joined #bitcoin-core-dev
 802017-02-01T10:41:28  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7bfb77045c4b...77bd8c4cab67
 812017-02-01T10:41:28  <bitcoin-git> bitcoin/master 29fb311 Alex Morcos: Increase minimum debug.log size to 10MB after shrink.
 822017-02-01T10:41:29  <bitcoin-git> bitcoin/master 77bd8c4 Wladimir J. van der Laan: Merge #9625: Increase minimum debug.log size to 10MB after shrink....
 832017-02-01T10:41:43  <bitcoin-git> [bitcoin] laanwj closed pull request #9625: Increase minimum debug.log size to 10MB after shrink. (master...shrinkless) https://github.com/bitcoin/bitcoin/pull/9625
 842017-02-01T10:45:37  *** aguycalled has quit IRC
 852017-02-01T10:46:14  *** aguycalled has joined #bitcoin-core-dev
 862017-02-01T10:48:10  *** lclc has quit IRC
 872017-02-01T11:13:14  *** waxwing has joined #bitcoin-core-dev
 882017-02-01T11:13:28  *** MarcoFalke has quit IRC
 892017-02-01T11:32:38  *** arubi has quit IRC
 902017-02-01T11:33:00  *** arubi has joined #bitcoin-core-dev
 912017-02-01T11:41:56  *** lclc has joined #bitcoin-core-dev
 922017-02-01T12:20:12  *** CubicEarth has quit IRC
 932017-02-01T12:21:36  *** dm67x has joined #bitcoin-core-dev
 942017-02-01T12:24:38  *** justan0theruser has quit IRC
 952017-02-01T12:36:14  *** lclc has quit IRC
 962017-02-01T12:45:32  *** lclc has joined #bitcoin-core-dev
 972017-02-01T13:02:25  *** lclc has quit IRC
 982017-02-01T13:03:44  *** Cory has quit IRC
 992017-02-01T13:04:56  *** lclc has joined #bitcoin-core-dev
1002017-02-01T13:05:45  *** nanotube has quit IRC
1012017-02-01T13:06:53  *** Cory has joined #bitcoin-core-dev
1022017-02-01T13:16:31  <bitcoin-git> [bitcoin] jonasschnelli opened pull request #9662: Add `-disablehot` mode: a sane mode for watchonly-wallets (master...2017/02/disable_hot) https://github.com/bitcoin/bitcoin/pull/9662
1032017-02-01T13:20:10  *** nanotube has joined #bitcoin-core-dev
1042017-02-01T13:20:41  *** CubicEarth has joined #bitcoin-core-dev
1052017-02-01T13:23:03  *** Chris_Stewart_5 has joined #bitcoin-core-dev
1062017-02-01T13:25:31  *** CubicEarth has quit IRC
1072017-02-01T13:31:08  *** lclc has quit IRC
1082017-02-01T13:50:16  *** lclc has joined #bitcoin-core-dev
1092017-02-01T13:55:13  *** lclc has quit IRC
1102017-02-01T14:05:27  *** dermoth_ has joined #bitcoin-core-dev
1112017-02-01T14:05:47  *** dermoth has quit IRC
1122017-02-01T14:05:57  *** dermoth_ is now known as dermoth
1132017-02-01T14:21:02  *** isle2983 has joined #bitcoin-core-dev
1142017-02-01T14:21:48  *** CubicEarth has joined #bitcoin-core-dev
1152017-02-01T14:25:26  *** goregrind has quit IRC
1162017-02-01T14:25:31  *** goregrin1 has joined #bitcoin-core-dev
1172017-02-01T14:26:47  *** CubicEarth has quit IRC
1182017-02-01T14:52:00  *** lclc has joined #bitcoin-core-dev
1192017-02-01T14:54:53  *** waxwing has quit IRC
1202017-02-01T15:17:38  *** aguycall_ has joined #bitcoin-core-dev
1212017-02-01T15:17:38  *** aguycalled has quit IRC
1222017-02-01T15:22:32  *** CubicEarth has joined #bitcoin-core-dev
1232017-02-01T15:24:20  *** RoyceX has quit IRC
1242017-02-01T15:26:07  *** Giszmo has quit IRC
1252017-02-01T15:27:26  *** CubicEarth has quit IRC
1262017-02-01T15:41:28  *** Giszmo has joined #bitcoin-core-dev
1272017-02-01T15:47:47  *** BashCo_ has joined #bitcoin-core-dev
1282017-02-01T15:50:47  *** BashCo has quit IRC
1292017-02-01T15:56:16  *** sevenqueue has joined #bitcoin-core-dev
1302017-02-01T16:04:35  *** Cheeseo has joined #bitcoin-core-dev
1312017-02-01T16:04:35  *** Cheeseo has joined #bitcoin-core-dev
1322017-02-01T16:11:46  *** lclc has quit IRC
1332017-02-01T16:23:19  *** CubicEarth has joined #bitcoin-core-dev
1342017-02-01T16:26:47  *** rafalcpp has quit IRC
1352017-02-01T16:28:18  *** rafalcpp has joined #bitcoin-core-dev
1362017-02-01T16:28:23  *** CubicEarth has quit IRC
1372017-02-01T16:34:20  *** paveljanik has joined #bitcoin-core-dev
1382017-02-01T16:34:20  *** paveljanik has joined #bitcoin-core-dev
1392017-02-01T16:34:54  *** aguycall_ has quit IRC
1402017-02-01T16:42:59  *** rafalcpp has quit IRC
1412017-02-01T16:54:40  *** jtimon has quit IRC
1422017-02-01T17:18:19  *** rafalcpp has joined #bitcoin-core-dev
1432017-02-01T17:22:11  <bitcoin-git> [bitcoin] instagibbs opened pull request #9663: [RPC] Trivial: clarify listunspent amount description (master...listoutput) https://github.com/bitcoin/bitcoin/pull/9663
1442017-02-01T17:24:03  *** CubicEarth has joined #bitcoin-core-dev
1452017-02-01T17:24:56  *** isle2983 has quit IRC
1462017-02-01T17:25:32  *** isle2983 has joined #bitcoin-core-dev
1472017-02-01T17:30:27  *** CubicEarth has quit IRC
1482017-02-01T17:51:06  *** abpa has joined #bitcoin-core-dev
1492017-02-01T18:12:53  *** laurentmt has joined #bitcoin-core-dev
1502017-02-01T18:13:53  *** laurentmt has quit IRC
1512017-02-01T18:17:24  *** BashCo_ has quit IRC
1522017-02-01T18:19:42  *** luke-jr has quit IRC
1532017-02-01T18:33:00  *** CubicEarth has joined #bitcoin-core-dev
1542017-02-01T18:37:17  *** BashCo has joined #bitcoin-core-dev
1552017-02-01T19:00:24  *** lclc has joined #bitcoin-core-dev
1562017-02-01T19:07:58  *** luke-jr has joined #bitcoin-core-dev
1572017-02-01T19:08:48  *** sevenqueue has left #bitcoin-core-dev
1582017-02-01T19:24:52  *** luke-jr has quit IRC
1592017-02-01T20:37:24  *** chjj has quit IRC
1602017-02-01T20:45:05  *** lclc has quit IRC
1612017-02-01T20:45:36  <Chris_Stewart_5> Is there any reason why we couldn't push certain op codes (i.e. predicates) onto the stack and use them for other op codes?
1622017-02-01T20:45:52  <Chris_Stewart_5> Similar to a higher order function.. this might be a more wizardy question, thought i'd ask here first though
1632017-02-01T20:46:04  <sipa> simplicity
1642017-02-01T20:48:58  <Chris_Stewart_5> and this can't be done with the current op codes we have rigtht? I can't think of one off of the top of my head that we can hack to achieve that..
1652017-02-01T20:49:22  <sipa> what good would it be?
1662017-02-01T20:49:33  <sipa> what could you do with it that you can't do now?
1672017-02-01T20:50:01  <Chris_Stewart_5> support for doubles in script? That isn't supported right now is it?
1682017-02-01T20:50:09  <sipa> die.
1692017-02-01T20:50:47  <gmaxwell> Chris_Stewart_5: you're asking about defining functions then?
1702017-02-01T20:51:11  <gmaxwell> it's very difficult to prove that anything which can do that doesn't immediately result in unbounded memory or computational usage.
1712017-02-01T20:52:54  <Chris_Stewart_5> sipa: Can you elaborate further than 'die'? Why is this such a bad idea? Not trying to 'stir the pot' so to speak, just want to understand the concerns
1722017-02-01T20:52:54  *** reginaldo has joined #bitcoin-core-dev
1732017-02-01T20:53:06  <sipa> Chris_Stewart_5: floating point numbers in consensus code?
1742017-02-01T20:53:23  <sipa> you'd cause a fork between different hardware that even just implements rounding slightly differently
1752017-02-01T20:55:03  *** luke-jr has joined #bitcoin-core-dev
1762017-02-01T21:09:26  <Chris_Stewart_5> I think that did deserve a 'die' response. Thanks sipa.
1772017-02-01T21:16:19  <Chris_Stewart_5> gmaxwell: I think it would be interesting to allow for predicates between two numbers in script, that was my hack to get around script not having floats until it was pointed out it is a bad idea
1782017-02-01T21:17:11  <Chris_Stewart_5> but i see your point about computational usage, and I don't know if it would be easy to limit their use similar to how we limit sigops :/
1792017-02-01T21:19:16  <bitcoin-git> [bitcoin] TheBlueMatt opened pull request #9665: Use cached [compact] blocks to respond to getdata messages (master...2017-02-processgetdata-cache) https://github.com/bitcoin/bitcoin/pull/9665
1802017-02-01T21:23:02  *** waxwing has joined #bitcoin-core-dev
1812017-02-01T21:23:05  *** reginaldo has quit IRC
1822017-02-01T21:26:03  *** chjj has joined #bitcoin-core-dev
1832017-02-01T21:30:48  <BlueMatt> #9650 (and, by extension, #9634) are probably 0.14
1842017-02-01T21:30:50  <gribble> https://github.com/bitcoin/bitcoin/issues/9650 | Better handle invalid parameters to signrawtransaction by TheBlueMatt · Pull Request #9650 · bitcoin/bitcoin · GitHub
1852017-02-01T21:30:52  <gribble> https://github.com/bitcoin/bitcoin/issues/9634 | Fail in DecodeHexTx if there is extra data at the end by jtimon · Pull Request #9634 · bitcoin/bitcoin · GitHub
1862017-02-01T21:43:39  *** chjj has quit IRC
1872017-02-01T21:43:57  *** reginaldo has joined #bitcoin-core-dev
1882017-02-01T21:43:59  *** chjj has joined #bitcoin-core-dev
1892017-02-01T22:05:54  *** Evel-Knievel has quit IRC
1902017-02-01T22:16:51  *** reginaldo has quit IRC
1912017-02-01T22:31:45  *** jannes has quit IRC
1922017-02-01T22:32:44  *** Evel-Knievel has joined #bitcoin-core-dev
1932017-02-01T22:34:07  *** MarcoFalke has joined #bitcoin-core-dev
1942017-02-01T22:51:27  *** jtimon has joined #bitcoin-core-dev
1952017-02-01T22:52:50  *** Giszmo has quit IRC
1962017-02-01T22:55:02  *** Giszmo has joined #bitcoin-core-dev
1972017-02-01T22:59:17  *** Evel-Knievel has quit IRC
1982017-02-01T23:00:45  *** Evel-Knievel has joined #bitcoin-core-dev
1992017-02-01T23:16:44  <bitcoin-git> [bitcoin] luke-jr closed pull request #9621: Define, check, and use MIN_TRANSACTION_SIZE as a const (master...cleanup_mintxsize) https://github.com/bitcoin/bitcoin/pull/9621
2002017-02-01T23:21:58  *** waxwing has quit IRC
2012017-02-01T23:22:54  *** Guyver2 has quit IRC
2022017-02-01T23:39:35  *** MarcoFalke has quit IRC
2032017-02-01T23:54:03  *** Squidicc has joined #bitcoin-core-dev
2042017-02-01T23:54:41  *** justanotheruser has joined #bitcoin-core-dev
2052017-02-01T23:56:09  *** Squidicuz has quit IRC