/bitcoin

Avoid unnecessary work in SetNetworkActive #10824

Merged
merged 1 commit into from Jul 26, 2017
+6 −4

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
P2P
Projects
None yet
Milestone
No milestone
6 participants
@promag @jonasschnelli @MeshCollider @TheBlueMatt @luke-jr @laanwj
@promag
Contributor

promag commented Jul 14, 2017

This PR adds an early return to avoid unnecessary notifications when the status doesn't change.
@jonasschnelli
Member

jonasschnelli commented Jul 14, 2017

utACK a2420ae

jonasschnelli added the P2P label Jul 14, 2017

@MeshCollider

MeshCollider commented Jul 14, 2017

utACK a2420ae
@TheBlueMatt
Contributor

TheBlueMatt commented Jul 14, 2017

utACK a2420ae
@luke-jr
Member

luke-jr commented Jul 26, 2017

utACK

@laanwj laanwj merged commit a2420ae into bitcoin:master Jul 26, 2017
1 check passed

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@laanwj laanwj added a commit that referenced this pull request Jul 26, 2017

@laanwj laanwj Merge #10824: Avoid unnecessary work in SetNetworkActive 
a2420ae Avoid unnecessary work in SetNetworkActive (João Barbosa)

Pull request description:

  This PR adds an early return to avoid unnecessary notifications when the status doesn't change.

Tree-SHA512: 85d05ca6fa36cb581f94bc154d08bd72cd53f6a857173c6fb2f184058f9c0208c4cf5e5d196825a78339902d8f256688eb6793f99abc7be9c7cfac85136180d9
This commit was signed with a verified signature.
@laanwj laanwj Wladimir J. van der Laan
GPG key ID: 1E4AED62986CD25D Learn about signing commits
9dd6a2b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment