Skip to content

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Mar 30, 2023

The removed lines have been unused since the abf5d16 commit from #25454.

@DrahtBot
Copy link
Contributor

DrahtBot commented Mar 30, 2023

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK dergoegge, TheCharlatan, Sjors

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

@dergoegge
Copy link
Member

utACK ea7ec78

Copy link
Contributor

@TheCharlatan TheCharlatan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK ea7ec78

@Sjors
Copy link
Member

Sjors commented Mar 30, 2023

ACK ea7ec78

@fanquake fanquake merged commit 5241b8b into bitcoin:master Mar 30, 2023
@hebasto hebasto deleted the 230330-msgmaker branch March 30, 2023 12:54
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Apr 1, 2023
…tances

ea7ec78 refactor: Drop no longer used `CNetMsgMaker` instances (Hennadii Stepanov)

Pull request description:

  The removed lines have been unused since the bitcoin@abf5d16 commit from bitcoin#25454.

ACKs for top commit:
  dergoegge:
    utACK ea7ec78
  Sjors:
    ACK ea7ec78
  TheCharlatan:
    ACK ea7ec78

Tree-SHA512: 9a2a9ff3f124b68a8cd20a637e90885096996c3aa354a4d8adbec98f5761e9e826c1c064ccd90aaf6d72beac61dd9e22c8b76d089e18bba6e0ad51e59a9c7df8
@bitcoin bitcoin locked and limited conversation to collaborators May 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants