[Chronik] Add all push opcodes to opcode.rs
ClosedPublic
Actions

Authored by tobias_ruck on Apr 26 2023, 09:54.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC5c7a3b948c0e: [Chronik] Add all push opcodes to opcode.rs

Summary

Also add the aliases OP_FALSE and OP_TRUE as the node does it.

The first definition of an opcode number defines the name of the opcode (hence the allow(unreachable_patterns); the second opcode is always ignored).

Test Plan

ninja check-crates

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tobias_ruck created this revision.Apr 26 2023, 09:54

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 26 2023, 09:54

tobias_ruck requested review of this revision.Apr 26 2023, 09:54

tobias_ruck added a child revision: D13785: [Chronik] Make `u8` in `Opcode` pub.Apr 26 2023, 10:02

Harbormaster completed remote builds in B23467: Diff 39952.Apr 26 2023, 10:09

Fabien accepted this revision.Apr 26 2023, 13:13

This revision is now accepted and ready to land.Apr 26 2023, 13:13

Closed by commit rABC5c7a3b948c0e: [Chronik] Add all push opcodes to opcode.rs (authored by tobias_ruck). · Explain WhyApr 26 2023, 13:35

This revision was automatically updated to reflect the committed changes.

tobias_ruck added a commit: rABC5c7a3b948c0e: [Chronik] Add all push opcodes to opcode.rs.

Revision Contents
Changeset List

Path

Size

chronik/

bitcoinsuite-core/

src/

script/

opcode.rs

53 lines

Diff 39965

View Options

chronik/bitcoinsuite-core/src/script/opcode.rs

[Chronik] Add all push opcodes to opcode.rsClosedPublicActions