Differential D9171

net: improve encapsulation of CNetAddr
ClosedPublic
Actions

Authored by Fabien on Feb 4 2021, 16:59.

Details

Reviewers

majcosta

Group Reviewers

Restricted Project

Commits

rABC2184cc687dab: net: improve encapsulation of CNetAddr

Summary

Do not access `CNetAddr::ip` directly from `CService` methods.

This improvement will help later when we change the type of
`CNetAddr::ip` (in the BIP155 implementation).

Backport of core PR19360.

Depends on D9169.

Test Plan

ninja all check-all

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Feb 4 2021, 16:59

Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 4 2021, 16:59

Fabien requested review of this revision.Feb 4 2021, 16:59

teamcity edited the summary of this revision. (Show Details)Feb 4 2021, 16:59

Fabien added a child revision: D9172: net: save the network type explicitly in CNetAddr.Feb 4 2021, 17:05

Harbormaster completed remote builds in B15125: Diff 27558.Feb 4 2021, 17:17

majcosta accepted this revision.Feb 4 2021, 22:16

This revision is now accepted and ready to land.Feb 4 2021, 22:16

Closed by commit rABC2184cc687dab: net: improve encapsulation of CNetAddr (authored by Vasil Dimov <vd@FreeBSD.org>, committed by Fabien). · Explain WhyFeb 5 2021, 17:33

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC2184cc687dab: net: improve encapsulation of CNetAddr.

Revision Contents
Changeset List

Path

Size

src/

netaddress.h

3 lines

netaddress.cpp

10 lines

Diff 27582

View Options

src/netaddress.h