Differential D11462

[avalanche] Don't assume shared_ptr for ProofRef
ClosedPublic
Actions

Authored by Fabien on May 13 2022, 23:10.

Details

Reviewers

deadalnix

Group Reviewers

Restricted Project

Commits

rABC1f440d3c6c59: [avalanche] Don't assume shared_ptr for ProofRef

Summary

This updates the places where the code assumes a shared pointer and assign nullptr. There is no change in behavior.

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.May 13 2022, 23:10

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 13 2022, 23:10

Fabien requested review of this revision.May 13 2022, 23:10

Harbormaster completed remote builds in B19023: Diff 33510.May 13 2022, 23:27

Fabien planned changes to this revision.May 13 2022, 23:32

deadalnix accepted this revision.May 13 2022, 23:33

Add missed cases

This revision is now accepted and ready to land.May 13 2022, 23:38

Harbormaster completed remote builds in B19027: Diff 33514.May 14 2022, 00:04

Closed by commit rABC1f440d3c6c59: [avalanche] Don't assume shared_ptr for ProofRef (authored by Fabien). · Explain WhyMay 14 2022, 00:13

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC1f440d3c6c59: [avalanche] Don't assume shared_ptr for ProofRef.

Revision Contents
Changeset List

Path

Size

src/

avalanche/

2 lines

2 lines

4 lines

6 lines

Diff 33518

View Options

src/avalanche/peermanager.cpp

View Options

src/avalanche/processor.cpp

View Options

src/avalanche/proofpool.cpp

View Options

[avalanche] Don't assume shared_ptr for ProofRefClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 33518

src/avalanche/peermanager.cpp

src/avalanche/processor.cpp

src/avalanche/proofpool.cpp

src/net_processing.cpp

[avalanche] Don't assume shared_ptr for ProofRef
ClosedPublic
Actions

Revision Contents
Changeset List