Skip to content

Conversation

@jnewbery
Copy link
Contributor

Final step in #11518

NodeConn -> P2PConnection
NodeConnCB -> P2PInterface

This is basically just a rename. Should be an easy review.

NodeConn -> P2PConnection
NodeConnCB -> P2PInterface
@fanquake
Copy link
Member

After merging the only mention of NodeConn or NodeConnCB I could find was in the 0.15 release notes.

Compiled and ran the testsuite.

ACK 873beca

@practicalswift
Copy link
Contributor

utACK 873beca

Copy link
Member

@maflcko maflcko left a comment

Choose a reason for hiding this comment

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

utACK 873beca

mininode_lock,
msg_block,
msg_getdata,
NODE_NETWORK,
Copy link
Member

Choose a reason for hiding this comment

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

style-nit: Unused import

@maflcko maflcko merged commit 873beca into bitcoin:master Nov 30, 2017
maflcko pushed a commit that referenced this pull request Nov 30, 2017
873beca [tests] Rename NodeConn and NodeConnCB (John Newbery)

Pull request description:

  Final step in #11518

  NodeConn -> P2PConnection
  NodeConnCB -> P2PInterface

  This is basically just a rename. Should be an easy review.

Tree-SHA512: fe1204b2b3d8182c5e324ffa7cb4099a47ef8536380e0bb9d37a5fccf76a24f548d1f1a7988ab8f830986a3058b670696de3fc891af5e5f75dbeb4e3273005d7
@jnewbery jnewbery deleted the rename_node_conn branch November 30, 2017 23:25
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Apr 8, 2020
873beca [tests] Rename NodeConn and NodeConnCB (John Newbery)

Pull request description:

  Final step in bitcoin#11518

  NodeConn -> P2PConnection
  NodeConnCB -> P2PInterface

  This is basically just a rename. Should be an easy review.

Tree-SHA512: fe1204b2b3d8182c5e324ffa7cb4099a47ef8536380e0bb9d37a5fccf76a24f548d1f1a7988ab8f830986a3058b670696de3fc891af5e5f75dbeb4e3273005d7
gades pushed a commit to cosanta/cosanta-core that referenced this pull request Jun 30, 2021
873beca [tests] Rename NodeConn and NodeConnCB (John Newbery)

Pull request description:

  Final step in bitcoin#11518

  NodeConn -> P2PConnection
  NodeConnCB -> P2PInterface

  This is basically just a rename. Should be an easy review.

Tree-SHA512: fe1204b2b3d8182c5e324ffa7cb4099a47ef8536380e0bb9d37a5fccf76a24f548d1f1a7988ab8f830986a3058b670696de3fc891af5e5f75dbeb4e3273005d7
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants