1 2017-02-01 00:20:59	0|bitcoin-git|[13bitcoin] 15droark opened pull request #9660: Backport “Fix: OSX QT compile: use built-in swap if available, or defer” (060.13...06macbackport) 02https://github.com/bitcoin/bitcoin/pull/9660
 2 2017-02-01 00:55:22	0|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?
 3 2017-02-01 00:56:39	0|sipa|the segwit rules require 1) no witness is present if the spent output is not a witness program
 4 2017-02-01 00:57:02	0|sipa|2) witness passes witness validation if the spent output is a witness program
 5 2017-02-01 01:18:31	0|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
 6 2017-02-01 01:18:48	0|Chris_Stewart_5|However, if I provided that same program w/ no witness, it would be treated as any old scriptPubKey, correct?
 7 2017-02-01 02:31:43	0|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.
 8 2017-02-01 02:33:27	0|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.
 9 2017-02-01 02:34:08	0|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*
10 2017-02-01 02:34:56	0|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.
11 2017-02-01 02:35:16	0|luke-jr|"valid" isn't in that?
12 2017-02-01 02:35:58	0|Chris_Stewart_5|no
13 2017-02-01 02:36:52	0|Chris_Stewart_5|are you talking about about valid op codes or valid witness program version's? I'm talking about the latter.
14 2017-02-01 02:37:22	0|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
15 2017-02-01 02:38:40	0|Chris_Stewart_5|Yeah, my text modification doesn't contain the word 'valid' at all.
16 2017-02-01 02:39:10	0|Chris_Stewart_5|even though I think 'select subset of opcodes' and 'valid opcodes' are semantically the same.. anyway i digress
17 2017-02-01 02:40:40	0|Chris_Stewart_5|ehhh actually the more I think about it I can see your point.
18 2017-02-01 02:43:01	0|Chris_Stewart_5|wrt not using the word 'valid'
19 2017-02-01 07:43:36	0|bitcoin-git|[13bitcoin] 15laanwj pushed 4 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/e99f0d7ad443...7bfb77045c4b
20 2017-02-01 07:43:37	0|bitcoin-git|13bitcoin/06master 14d625b90 15Suhas Daftuar: wallet: Refactor dummy signature signing for reusability
21 2017-02-01 07:43:37	0|bitcoin-git|13bitcoin/06master 14f626594 15Suhas Daftuar: rpc: bumpfee: use correct maximum signed tx size for fee calculation...
22 2017-02-01 07:43:38	0|bitcoin-git|13bitcoin/06master 149522b53 15Suhas Daftuar: rpc: bumpfee: handle errors more gracefully
23 2017-02-01 07:43:51	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #9640: Bumpfee: bugfixes for error handling and feerate calculation (06master...062017-01-bumpfee-error-cleanup) 02https://github.com/bitcoin/bitcoin/pull/9640
24 2017-02-01 08:17:54	0|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.
25 2017-02-01 08:18:41	0|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
26 2017-02-01 08:19:29	0|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)
27 2017-02-01 08:21:44	0|sipa|well don't mix them into the same wallet then
28 2017-02-01 08:21:57	0|sipa|that's asking for trouble regardless i think
29 2017-02-01 08:22:46	0|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.
30 2017-02-01 08:23:05	0|jonasschnelli|IMO it would be sane if fundraw had an option to say "only use watchonly".
31 2017-02-01 08:23:18	0|jonasschnelli|Just in case you where dump enought to receive coins into your hot wallet
32 2017-02-01 08:23:21	0|jonasschnelli|dumb
33 2017-02-01 08:24:00	0|sipa|that's equally dumb to otherwise mixing two other wallets
34 2017-02-01 08:24:04	0|bitcoin-git|13bitcoin/060.13 1459c37ae 15Karl-Johan Alm: Uses built-in byte swap if available (Apple) and if bswap_XX is undefined....
35 2017-02-01 08:24:04	0|bitcoin-git|[13bitcoin] 15laanwj pushed 1 new commit to 060.13: 02https://github.com/bitcoin/bitcoin/commit/59c37ae55a476d3cf84c9bafbc083904472fe4db
36 2017-02-01 08:24:18	0|sipa|why not provide an option for separating an accidentally-mixed wallet out
37 2017-02-01 08:24:34	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #9660: Backport “Fix: OSX QT compile: use built-in swap if available, or defer” (060.13...06macbackport) 02https://github.com/bitcoin/bitcoin/pull/9660
38 2017-02-01 08:29:03	0|jonasschnelli|sipa: Yes. Assume I tell some hardware-wallet users to use watchonlys in order to use core with the HWW.
39 2017-02-01 08:29:14	0|jonasschnelli|I'm almost certain they will screw up.
40 2017-02-01 08:29:37	0|jonasschnelli|Maybe an option to disables the hot/private-keys mode in a core wallet could be something... but meh
41 2017-02-01 10:41:30	0|bitcoin-git|13bitcoin/06master 1429fb311 15Alex Morcos: Increase minimum debug.log size to 10MB after shrink.
42 2017-02-01 10:41:30	0|bitcoin-git|[13bitcoin] 15laanwj pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/7bfb77045c4b...77bd8c4cab67
43 2017-02-01 10:41:31	0|bitcoin-git|13bitcoin/06master 1477bd8c4 15Wladimir J. van der Laan: Merge #9625: Increase minimum debug.log size to 10MB after shrink....
44 2017-02-01 10:41:45	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #9625: Increase minimum debug.log size to 10MB after shrink. (06master...06shrinkless) 02https://github.com/bitcoin/bitcoin/pull/9625
45 2017-02-01 13:16:33	0|bitcoin-git|[13bitcoin] 15jonasschnelli opened pull request #9662: Add `-disablehot` mode: a sane mode for watchonly-wallets (06master...062017/02/disable_hot) 02https://github.com/bitcoin/bitcoin/pull/9662
46 2017-02-01 17:22:12	0|bitcoin-git|[13bitcoin] 15instagibbs opened pull request #9663: [RPC] Trivial: clarify listunspent amount description (06master...06listoutput) 02https://github.com/bitcoin/bitcoin/pull/9663
47 2017-02-01 20:45:38	0|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?
48 2017-02-01 20:45:54	0|Chris_Stewart_5|Similar to a higher order function.. this might be a more wizardy question, thought i'd ask here first though
49 2017-02-01 20:46:05	0|sipa|simplicity
50 2017-02-01 20:49:00	0|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..
51 2017-02-01 20:49:24	0|sipa|what good would it be?
52 2017-02-01 20:49:34	0|sipa|what could you do with it that you can't do now?
53 2017-02-01 20:50:03	0|Chris_Stewart_5|support for doubles in script? That isn't supported right now is it?
54 2017-02-01 20:50:11	0|sipa|die.
55 2017-02-01 20:50:48	0|gmaxwell|Chris_Stewart_5: you're asking about defining functions then?
56 2017-02-01 20:51:13	0|gmaxwell|it's very difficult to prove that anything which can do that doesn't immediately result in unbounded memory or computational usage.
57 2017-02-01 20:52:55	0|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
58 2017-02-01 20:53:07	0|sipa|Chris_Stewart_5: floating point numbers in consensus code?
59 2017-02-01 20:53:24	0|sipa|you'd cause a fork between different hardware that even just implements rounding slightly differently
60 2017-02-01 21:09:28	0|Chris_Stewart_5|I think that did deserve a 'die' response. Thanks sipa.
61 2017-02-01 21:16:21	0|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
62 2017-02-01 21:17:12	0|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 :/
63 2017-02-01 21:19:18	0|bitcoin-git|[13bitcoin] 15TheBlueMatt opened pull request #9665: Use cached [compact] blocks to respond to getdata messages (06master...062017-02-processgetdata-cache) 02https://github.com/bitcoin/bitcoin/pull/9665
64 2017-02-01 21:30:50	0|BlueMatt|#9650 (and, by extension, #9634) are probably 0.14
65 2017-02-01 21:30:51	0|gribble|https://github.com/bitcoin/bitcoin/issues/9650 | Better handle invalid parameters to signrawtransaction by TheBlueMatt · Pull Request #9650 · bitcoin/bitcoin · GitHub
66 2017-02-01 21:30:53	0|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
67 2017-02-01 23:16:45	0|bitcoin-git|[13bitcoin] 15luke-jr closed pull request #9621: Define, check, and use MIN_TRANSACTION_SIZE as a const (06master...06cleanup_mintxsize) 02https://github.com/bitcoin/bitcoin/pull/9621