Differential D7175

Document 2-complement assumption in assumption.h
ClosedPublic
Actions

Authored by deadalnix on Aug 9 2020, 16:41.

Details

Reviewers

majcosta

Group Reviewers

Restricted Project

Commits

rABCf7ddc14fa51c: Document 2-complement assumption in assumption.h

Summary

As per title.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Branch

twocompassumption

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 12359
Build 24905: Build Diff	build-clang-tidy · build-clang-10 · build-diff · build-without-wallet
Build 24904: arc lint + arc unit

Event Timeline

deadalnix created this revision.Aug 9 2020, 16:41

Herald added a reviewer: Restricted Project. · View Herald TranscriptAug 9 2020, 16:41

deadalnix requested review of this revision.Aug 9 2020, 16:41

Fix typos

majcosta accepted this revision.Aug 9 2020, 16:46

This revision is now accepted and ready to land.Aug 9 2020, 16:46

This revision was landed with ongoing or failed builds.Aug 9 2020, 16:48

Closed by commit rABCf7ddc14fa51c: Document 2-complement assumption in assumption.h (authored by deadalnix). · Explain Why

This revision was automatically updated to reflect the committed changes.

deadalnix added a commit: rABCf7ddc14fa51c: Document 2-complement assumption in assumption.h.

Harbormaster completed remote builds in B12359: Diff 22843.Aug 9 2020, 16:57

Harbormaster completed remote builds in B12358: Diff 22841.

Revision Contents
Changeset List

Path

Size

src/

compat/

assumptions.h

11 lines

Diff 22843

View Options

Document 2-complement assumption in assumption.hClosedPublicActions