[Chronik] Make `u8` in `Opcode` pub
ClosedPublic
Actions

Authored by tobias_ruck on Apr 26 2023, 10:02.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC94e1e0c9ddfc: [Chronik] Make `u8` in `Opcode` pub

Summary

This allows nice pattern matching.

Depends on D13784.

Test Plan

ninja check-crates

Diff Detail

Repository

rABC Bitcoin ABC

Branch

chronik-opcode-pub

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 23468
Build 46555: Build Diff	build-chronik
Build 46554: arc lint + arc unit

Event Timeline

tobias_ruck created this revision.Apr 26 2023, 10:02

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 26 2023, 10:02

tobias_ruck requested review of this revision.Apr 26 2023, 10:02

Harbormaster completed remote builds in B23468: Diff 39953.Apr 26 2023, 10:16

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

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

Closed by commit rABC94e1e0c9ddfc: [Chronik] Make `u8` in `Opcode` pub (authored by tobias_ruck). · Explain WhyApr 26 2023, 13:37

This revision was automatically updated to reflect the committed changes.

tobias_ruck added a commit: rABC94e1e0c9ddfc: [Chronik] Make `u8` in `Opcode` pub.

Revision Contents
Changeset List

Path

Size

chronik/

bitcoinsuite-core/

src/

script/

opcode.rs

9 lines

Diff 39953

View Options

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

[Chronik] Make `u8` in `Opcode` pubClosedPublicActions