Page MenuHomePhabricator
Differential D13539

[Chronik] Add `Group::ser_member`, remove `Into<Vec<u8>>` bound on `Group::Member`
ClosedPublic
Actions

Authored by tobias_ruck on Apr 3 2023, 13:23.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC201a7ecbbbe2: [Chronik] Add `Group::ser_member`, remove `Into<Vec<u8>>` bound on `Group…
Summary

This makes the code cleaner (no newtyping necessary to get custom serialization), and more flexible (groups can use &self) when serializing.

Test Plan

ninja check-crates

Diff Detail

Repository
rABC Bitcoin ABC
Branch
chronik-group-ser_member
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 22906
Build 45432: Build Diffbuild-chronik
Build 45431: arc lint + arc unit

Revision Contents
Changeset List

PathSize
chronik/
chronik-db/
src/
5 lines
io/
4 lines
mem/
4 lines
test/
12 lines

Diff 39152

View Options

chronik/chronik-db/src/group.rs

Loading...
View Options

chronik/chronik-db/src/io/group_history.rs

Loading...
View Options

chronik/chronik-db/src/mem/group_history.rs

Loading...
View Options

chronik/chronik-db/src/test/value_group.rs

Loading...