1 2017-02-01T00:12:18  *** aguycalled has quit IRC
  2 2017-02-01T00:12:52  *** aguycalled has joined #bitcoin-core-dev
  3 2017-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
  4 2017-02-01T00:39:23  *** Chris_Stewart_5 has quit IRC
  5 2017-02-01T00:43:04  *** wasi has quit IRC
  6 2017-02-01T00:44:04  *** luke-jr has joined #bitcoin-core-dev
  7 2017-02-01T00:51:30  *** Chris_Stewart_5 has joined #bitcoin-core-dev
  8 2017-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?
  9 2017-02-01T00:55:53  *** wasi has joined #bitcoin-core-dev
 10 2017-02-01T00:56:38  <sipa> the segwit rules require 1) no witness is present if the spent output is not a witness program
 11 2017-02-01T00:57:01  <sipa> 2) witness passes witness validation if the spent output is a witness program
 12 2017-02-01T01:00:20  *** abpa has quit IRC
 13 2017-02-01T01:03:59  *** MarcoFalke has joined #bitcoin-core-dev
 14 2017-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
 15 2017-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?
 16 2017-02-01T01:22:30  *** chjj has quit IRC
 17 2017-02-01T01:28:02  *** MarcoFalke has quit IRC
 18 2017-02-01T01:36:30  *** chjj has joined #bitcoin-core-dev
 19 2017-02-01T02:09:40  *** Giszmo has quit IRC
 20 2017-02-01T02:19:56  *** Giszmo has joined #bitcoin-core-dev
 21 2017-02-01T02:28:42  *** harrymm has joined #bitcoin-core-dev
 22 2017-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.
 23 2017-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.
 24 2017-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*
 25 2017-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.
 26 2017-02-01T02:35:15  <luke-jr> "valid" isn't in that?
 27 2017-02-01T02:35:56  <Chris_Stewart_5> no
 28 2017-02-01T02:36:27  * luke-jr is now confused as to the topic of discussion
 29 2017-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.
 30 2017-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
 31 2017-02-01T02:38:39  <Chris_Stewart_5> Yeah, my text modification doesn't contain the word 'valid' at all.
 32 2017-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
 33 2017-02-01T02:40:39  <Chris_Stewart_5> ehhh actually the more I think about it I can see your point.
 34 2017-02-01T02:42:59  <Chris_Stewart_5> wrt not using the word 'valid'
 35 2017-02-01T02:55:51  *** aguycalled has quit IRC
 36 2017-02-01T03:04:10  *** Giszmo has quit IRC
 37 2017-02-01T03:19:07  *** Giszmo has joined #bitcoin-core-dev
 38 2017-02-01T04:54:36  *** chris200_ has joined #bitcoin-core-dev
 39 2017-02-01T04:57:41  *** chris2000 has quit IRC
 40 2017-02-01T06:07:31  *** Ylbam has quit IRC
 41 2017-02-01T06:16:48  *** chjj has quit IRC
 42 2017-02-01T06:50:03  *** lclc has joined #bitcoin-core-dev
 43 2017-02-01T07:18:48  *** Giszmo has quit IRC
 44 2017-02-01T07:23:12  *** chjj has joined #bitcoin-core-dev
 45 2017-02-01T07:25:51  *** Ylbam has joined #bitcoin-core-dev
 46 2017-02-01T07:38:50  *** Giszmo has joined #bitcoin-core-dev
 47 2017-02-01T07:43:35  <bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/e99f0d7ad443...7bfb77045c4b
 48 2017-02-01T07:43:36  <bitcoin-git> bitcoin/master d625b90 Suhas Daftuar: wallet: Refactor dummy signature signing for reusability
 49 2017-02-01T07:43:36  <bitcoin-git> bitcoin/master f626594 Suhas Daftuar: rpc: bumpfee: use correct maximum signed tx size for fee calculation...
 50 2017-02-01T07:43:36  <bitcoin-git> bitcoin/master 9522b53 Suhas Daftuar: rpc: bumpfee: handle errors more gracefully
 51 2017-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
 52 2017-02-01T08:08:03  *** BashCo has quit IRC
 53 2017-02-01T08:08:40  *** BashCo has joined #bitcoin-core-dev
 54 2017-02-01T08:13:28  *** BashCo has quit IRC
 55 2017-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.
 56 2017-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
 57 2017-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)
 58 2017-02-01T08:21:43  <sipa> well don't mix them into the same wallet then
 59 2017-02-01T08:21:56  <sipa> that's asking for trouble regardless i think
 60 2017-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.
 61 2017-02-01T08:23:03  <jonasschnelli> IMO it would be sane if fundraw had an option to say "only use watchonly".
 62 2017-02-01T08:23:16  <jonasschnelli> Just in case you where dump enought to receive coins into your hot wallet
 63 2017-02-01T08:23:19  <jonasschnelli> dumb
 64 2017-02-01T08:23:58  <sipa> that's equally dumb to otherwise mixing two other wallets
 65 2017-02-01T08:24:02  <bitcoin-git> [bitcoin] laanwj pushed 1 new commit to 0.13: https://github.com/bitcoin/bitcoin/commit/59c37ae55a476d3cf84c9bafbc083904472fe4db
 66 2017-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....
 67 2017-02-01T08:24:17  <sipa> why not provide an option for separating an accidentally-mixed wallet out
 68 2017-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
 69 2017-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.
 70 2017-02-01T08:29:13  <jonasschnelli> I'm almost certain they will screw up.
 71 2017-02-01T08:29:36  <jonasschnelli> Maybe an option to disables the hot/private-keys mode in a core wallet could be something... but meh
 72 2017-02-01T08:30:18  *** BashCo has joined #bitcoin-core-dev
 73 2017-02-01T08:51:21  *** Guyver2 has joined #bitcoin-core-dev
 74 2017-02-01T09:07:40  *** isle2983 has quit IRC
 75 2017-02-01T09:09:51  *** Chris_Stewart_5 has quit IRC
 76 2017-02-01T09:23:26  *** jannes has joined #bitcoin-core-dev
 77 2017-02-01T10:18:26  *** MarcoFalke has joined #bitcoin-core-dev
 78 2017-02-01T10:23:12  *** paveljanik has quit IRC
 79 2017-02-01T10:33:38  *** aguycalled has joined #bitcoin-core-dev
 80 2017-02-01T10:41:28  <bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7bfb77045c4b...77bd8c4cab67
 81 2017-02-01T10:41:28  <bitcoin-git> bitcoin/master 29fb311 Alex Morcos: Increase minimum debug.log size to 10MB after shrink.
 82 2017-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....
 83 2017-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
 84 2017-02-01T10:45:37  *** aguycalled has quit IRC
 85 2017-02-01T10:46:14  *** aguycalled has joined #bitcoin-core-dev
 86 2017-02-01T10:48:10  *** lclc has quit IRC
 87 2017-02-01T11:13:14  *** waxwing has joined #bitcoin-core-dev
 88 2017-02-01T11:13:28  *** MarcoFalke has quit IRC
 89 2017-02-01T11:32:38  *** arubi has quit IRC
 90 2017-02-01T11:33:00  *** arubi has joined #bitcoin-core-dev
 91 2017-02-01T11:41:56  *** lclc has joined #bitcoin-core-dev
 92 2017-02-01T12:20:12  *** CubicEarth has quit IRC
 93 2017-02-01T12:21:36  *** dm67x has joined #bitcoin-core-dev
 94 2017-02-01T12:24:38  *** justan0theruser has quit IRC
 95 2017-02-01T12:36:14  *** lclc has quit IRC
 96 2017-02-01T12:45:32  *** lclc has joined #bitcoin-core-dev
 97 2017-02-01T13:02:25  *** lclc has quit IRC
 98 2017-02-01T13:03:44  *** Cory has quit IRC
 99 2017-02-01T13:04:56  *** lclc has joined #bitcoin-core-dev
100 2017-02-01T13:05:45  *** nanotube has quit IRC
101 2017-02-01T13:06:53  *** Cory has joined #bitcoin-core-dev
102 2017-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
103 2017-02-01T13:20:10  *** nanotube has joined #bitcoin-core-dev
104 2017-02-01T13:20:41  *** CubicEarth has joined #bitcoin-core-dev
105 2017-02-01T13:23:03  *** Chris_Stewart_5 has joined #bitcoin-core-dev
106 2017-02-01T13:25:31  *** CubicEarth has quit IRC
107 2017-02-01T13:31:08  *** lclc has quit IRC
108 2017-02-01T13:50:16  *** lclc has joined #bitcoin-core-dev
109 2017-02-01T13:55:13  *** lclc has quit IRC
110 2017-02-01T14:05:27  *** dermoth_ has joined #bitcoin-core-dev
111 2017-02-01T14:05:47  *** dermoth has quit IRC
112 2017-02-01T14:05:57  *** dermoth_ is now known as dermoth
113 2017-02-01T14:21:02  *** isle2983 has joined #bitcoin-core-dev
114 2017-02-01T14:21:48  *** CubicEarth has joined #bitcoin-core-dev
115 2017-02-01T14:25:26  *** goregrind has quit IRC
116 2017-02-01T14:25:31  *** goregrin1 has joined #bitcoin-core-dev
117 2017-02-01T14:26:47  *** CubicEarth has quit IRC
118 2017-02-01T14:52:00  *** lclc has joined #bitcoin-core-dev
119 2017-02-01T14:54:53  *** waxwing has quit IRC
120 2017-02-01T15:17:38  *** aguycall_ has joined #bitcoin-core-dev
121 2017-02-01T15:17:38  *** aguycalled has quit IRC
122 2017-02-01T15:22:32  *** CubicEarth has joined #bitcoin-core-dev
123 2017-02-01T15:24:20  *** RoyceX has quit IRC
124 2017-02-01T15:26:07  *** Giszmo has quit IRC
125 2017-02-01T15:27:26  *** CubicEarth has quit IRC
126 2017-02-01T15:41:28  *** Giszmo has joined #bitcoin-core-dev
127 2017-02-01T15:47:47  *** BashCo_ has joined #bitcoin-core-dev
128 2017-02-01T15:50:47  *** BashCo has quit IRC
129 2017-02-01T15:56:16  *** sevenqueue has joined #bitcoin-core-dev
130 2017-02-01T16:04:35  *** Cheeseo has joined #bitcoin-core-dev
131 2017-02-01T16:04:35  *** Cheeseo has joined #bitcoin-core-dev
132 2017-02-01T16:11:46  *** lclc has quit IRC
133 2017-02-01T16:23:19  *** CubicEarth has joined #bitcoin-core-dev
134 2017-02-01T16:26:47  *** rafalcpp has quit IRC
135 2017-02-01T16:28:18  *** rafalcpp has joined #bitcoin-core-dev
136 2017-02-01T16:28:23  *** CubicEarth has quit IRC
137 2017-02-01T16:34:20  *** paveljanik has joined #bitcoin-core-dev
138 2017-02-01T16:34:20  *** paveljanik has joined #bitcoin-core-dev
139 2017-02-01T16:34:54  *** aguycall_ has quit IRC
140 2017-02-01T16:42:59  *** rafalcpp has quit IRC
141 2017-02-01T16:54:40  *** jtimon has quit IRC
142 2017-02-01T17:18:19  *** rafalcpp has joined #bitcoin-core-dev
143 2017-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
144 2017-02-01T17:24:03  *** CubicEarth has joined #bitcoin-core-dev
145 2017-02-01T17:24:56  *** isle2983 has quit IRC
146 2017-02-01T17:25:32  *** isle2983 has joined #bitcoin-core-dev
147 2017-02-01T17:30:27  *** CubicEarth has quit IRC
148 2017-02-01T17:51:06  *** abpa has joined #bitcoin-core-dev
149 2017-02-01T18:12:53  *** laurentmt has joined #bitcoin-core-dev
150 2017-02-01T18:13:53  *** laurentmt has quit IRC
151 2017-02-01T18:17:24  *** BashCo_ has quit IRC
152 2017-02-01T18:19:42  *** luke-jr has quit IRC
153 2017-02-01T18:33:00  *** CubicEarth has joined #bitcoin-core-dev
154 2017-02-01T18:37:17  *** BashCo has joined #bitcoin-core-dev
155 2017-02-01T19:00:24  *** lclc has joined #bitcoin-core-dev
156 2017-02-01T19:07:58  *** luke-jr has joined #bitcoin-core-dev
157 2017-02-01T19:08:48  *** sevenqueue has left #bitcoin-core-dev
158 2017-02-01T19:24:52  *** luke-jr has quit IRC
159 2017-02-01T20:37:24  *** chjj has quit IRC
160 2017-02-01T20:45:05  *** lclc has quit IRC
161 2017-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?
162 2017-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
163 2017-02-01T20:46:04  <sipa> simplicity
164 2017-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..
165 2017-02-01T20:49:22  <sipa> what good would it be?
166 2017-02-01T20:49:33  <sipa> what could you do with it that you can't do now?
167 2017-02-01T20:50:01  <Chris_Stewart_5> support for doubles in script? That isn't supported right now is it?
168 2017-02-01T20:50:09  <sipa> die.
169 2017-02-01T20:50:47  <gmaxwell> Chris_Stewart_5: you're asking about defining functions then?
170 2017-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.
171 2017-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
172 2017-02-01T20:52:54  *** reginaldo has joined #bitcoin-core-dev
173 2017-02-01T20:53:06  <sipa> Chris_Stewart_5: floating point numbers in consensus code?
174 2017-02-01T20:53:23  <sipa> you'd cause a fork between different hardware that even just implements rounding slightly differently
175 2017-02-01T20:55:03  *** luke-jr has joined #bitcoin-core-dev
176 2017-02-01T21:09:26  <Chris_Stewart_5> I think that did deserve a 'die' response. Thanks sipa.
177 2017-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
178 2017-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 :/
179 2017-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
180 2017-02-01T21:23:02  *** waxwing has joined #bitcoin-core-dev
181 2017-02-01T21:23:05  *** reginaldo has quit IRC
182 2017-02-01T21:26:03  *** chjj has joined #bitcoin-core-dev
183 2017-02-01T21:30:48  <BlueMatt> #9650 (and, by extension, #9634) are probably 0.14
184 2017-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
185 2017-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
186 2017-02-01T21:43:39  *** chjj has quit IRC
187 2017-02-01T21:43:57  *** reginaldo has joined #bitcoin-core-dev
188 2017-02-01T21:43:59  *** chjj has joined #bitcoin-core-dev
189 2017-02-01T22:05:54  *** Evel-Knievel has quit IRC
190 2017-02-01T22:16:51  *** reginaldo has quit IRC
191 2017-02-01T22:31:45  *** jannes has quit IRC
192 2017-02-01T22:32:44  *** Evel-Knievel has joined #bitcoin-core-dev
193 2017-02-01T22:34:07  *** MarcoFalke has joined #bitcoin-core-dev
194 2017-02-01T22:51:27  *** jtimon has joined #bitcoin-core-dev
195 2017-02-01T22:52:50  *** Giszmo has quit IRC
196 2017-02-01T22:55:02  *** Giszmo has joined #bitcoin-core-dev
197 2017-02-01T22:59:17  *** Evel-Knievel has quit IRC
198 2017-02-01T23:00:45  *** Evel-Knievel has joined #bitcoin-core-dev
199 2017-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
200 2017-02-01T23:21:58  *** waxwing has quit IRC
201 2017-02-01T23:22:54  *** Guyver2 has quit IRC
202 2017-02-01T23:39:35  *** MarcoFalke has quit IRC
203 2017-02-01T23:54:03  *** Squidicc has joined #bitcoin-core-dev
204 2017-02-01T23:54:41  *** justanotheruser has joined #bitcoin-core-dev
205 2017-02-01T23:56:09  *** Squidicuz has quit IRC