Differential D8743

Add custom vector-element formatter
ClosedPublic
Actions

Authored by PiRK on Dec 22 2020, 16:18.

Details

Reviewers

majcosta

Group Reviewers

Restricted Project

Commits

rABC4ce8d58f5c6a: Add custom vector-element formatter

Summary

This allows a very compact notation for serialization of vectors whose
elements are not serialized using their default encoding.

This is a partial backport of Core PR18021 [2/4]
https://github.com/bitcoin/bitcoin/pull/18021/commits/abf86243568af380c1384ac4e0bfcdcfd4dab085

Depends on D8742

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

PiRK created this revision.Dec 22 2020, 16:18

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 22 2020, 16:18

PiRK requested review of this revision.Dec 22 2020, 16:18

teamcity edited the summary of this revision. (Show Details)Dec 22 2020, 16:19

PiRK edited the summary of this revision. (Show Details)Dec 22 2020, 16:21

PiRK added a parent revision: D8742: Add a constant for the maximum vector allocation (5 Mbyte).

PiRK added a child revision: D8744: Make std::vector and prevector reuse the VectorFormatter logic.Dec 22 2020, 16:21

Harbormaster completed remote builds in B14507: Diff 26530.Dec 22 2020, 16:39

majcosta accepted this revision.Dec 22 2020, 17:55

This revision is now accepted and ready to land.Dec 22 2020, 17:55

Closed by commit rABC4ce8d58f5c6a: Add custom vector-element formatter (authored by Pieter Wuille <pieter.wuille@gmail.com>, committed by PiRK). · Explain WhyDec 23 2020, 14:05

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC4ce8d58f5c6a: Add custom vector-element formatter.

Revision Contents
Changeset List

Path

Size

src/

serialize.h

48 lines

Diff 26541

View Options

Add custom vector-element formatterClosedPublicActions