Merge pull request #47 from bitcoin-sv/fix-op-return

script interpreter update

Author: rohenaz

RELEASE.md

@@ -1,6 +1,6 @@
 # Steps for creating a new release
 
-1. `git tag v1.0.0`
-2. `git push origin v1.0.0`
-3. `marge tagged branch to master branch`
+1. `merge tagged branch to master branch`
+2. `git tag v1.0.0`
+3. `git push origin v1.0.0`
 4. `goreleaser release`

script/interpreter/example_test.go

@@ -155,6 +155,10 @@ func (p *DefaultOpcodeParser) Parse(s *script.Script) (ParsedScript, error) {
 			// If we are not in a conditional block, we end script evaluation.
 			// This must be the final evaluated opcode, everything after is ignored.
 			if conditionalBlock == 0 {
+				if i+1 < len(scr) {
+					parsedOp.Data = scr[i+1:]
+					parsedOp.op.length = 1 + len(parsedOp.Data)
+				}
 				parsedOps = append(parsedOps, parsedOp)
 				return parsedOps, nil
 			}

script/interpreter/opcodeparser.go

@@ -65,10 +65,10 @@ func TestParse(t *testing.T) {
 					op: opcode{
 						val:    script.OpRETURN,
 						name:   "OP_RETURN",
-						length: 1,
+						length: 4,
 						exec:   opcodeReturn,
 					},
-					Data: nil,
+					Data: []byte{0x00, 0x24, 0xDC},
 				},
 			},
 		},