Paths

Table of Contentst

Differential D14426

[Chronik] Add `compress_script_variant` to compress script variants
ClosedPublic
Actions

Authored by tobias_ruck on Aug 29 2023, 14:12.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCaf6089b30a23: [Chronik] Add `compress_script_variant` to compress script variants

Summary

Mirrors ScriptCompression in /src/compressor.h.

Having a Rust variant of this function allows us to avoid costly FFI bridging, which gets relevant in hot loops.

Test Plan

ninja check-crates

Diff Detail

Repository

rABC Bitcoin ABC

Branch

chronik-compress_script_variant

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 24887
Build 49362: Build Diff	build-chronik
Build 49361: arc lint + arc unit

Event Timeline

tobias_ruck created this revision.Aug 29 2023, 14:12

Herald added a reviewer: Restricted Project. · View Herald TranscriptAug 29 2023, 14:12

tobias_ruck requested review of this revision.Aug 29 2023, 14:12

Harbormaster completed remote builds in B24887: Diff 41987.Aug 29 2023, 14:31

Fabien accepted this revision.Aug 29 2023, 18:11

This revision is now accepted and ready to land.Aug 29 2023, 18:11

Closed by commit rABCaf6089b30a23: [Chronik] Add `compress_script_variant` to compress script variants (authored by tobias_ruck). · Explain WhyAug 29 2023, 22:01

This revision was automatically updated to reflect the committed changes.

tobias_ruck added a commit: rABCaf6089b30a23: [Chronik] Add `compress_script_variant` to compress script variants.

Revision Contents
Changeset List

Path

Size

chronik/

bitcoinsuite-core/

src/

script/

compress.rs

181 lines

mod.rs

2 lines

Diff 41987

View Options

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