Preparing for Multi-signature Transactions#1
Conversation
| else{ | ||
| _isDecoded = true; | ||
| _chunks = bscript.decompile(output); | ||
| _temp['m'] = _chunks[0] - OPS['OP_RESERVED']; |
There was a problem hiding this comment.
Please use P2MSData, dynamic some cases cause dangerous
There was a problem hiding this comment.
Should I the type of _chunks from List? I am unsure where to use P2MSData types, aside from the _temp issue addressed later.
| if (data.input == null) {return;} | ||
| var list = bscript.decompile(data.input); | ||
| list.removeAt(0); | ||
| List<Uint8List> uintList = []; |
| P2MSData data; | ||
| NetworkType network; | ||
| List<dynamic> _chunks; | ||
| Map _temp; |
There was a problem hiding this comment.
Do you mean that _temp should not exist and that I should change the value of P2MSData data directly?
The following code:
_temp['m'] = _chunks[0] - OPS['OP_RESERVED'];
should be:
data.m = _chunks[0] - OPS['OP_RESERVED']; instead?
There was a problem hiding this comment.
I removed all _temp usage, but it has caused the code to regress and fail 3 test. I reference this regression in this commit. I am unsure of how to fix this. I don't think we need this test anymore, but it feels wrong to delete test, in response to code regression.
Update my branch for Circle CI
fails these 3 test starting at lines 146
|
I removed all _temp usage, but it has caused the code to regress and fail 3 test. I reference this regression in this commit. I am unsure of how to fix this. I don't think we need this test anymore, but it feels wrong to delete test, in response to code regression. |
2 participants
I would like to use this library to do multisig transactions in dart for bitcoin. This is the first port of libraries from bitcoinjs required to accomplish this.