Skip to content

Conversation

@jtimon
Copy link
Contributor

@jtimon jtimon commented Apr 4, 2015

As the tittle says, just decouple CAlert from CChainParams by introducing an alertKey parameter in some CAlert methods.

@jtimon jtimon force-pushed the chainparams_alert branch from 9d17171 to f14e687 Compare April 4, 2015 10:59
@jtimon jtimon changed the title Chainparams: Decouple CAlert from CChainParams Trivial: Chainparams: Decouple CAlert from CChainParams Apr 4, 2015
@laanwj
Copy link
Member

laanwj commented Apr 7, 2015

utACK

Edit: This is an obviously correct change but is not for @theuni's trivial branch as it involves code changes.

@theuni
Copy link
Member

theuni commented Apr 7, 2015

ut ack.

@sipa
Copy link
Member

sipa commented Apr 7, 2015

utACK

@laanwj laanwj merged commit f14e687 into bitcoin:master Apr 8, 2015
laanwj added a commit that referenced this pull request Apr 8, 2015
f14e687 Chainparams: Decouple CAlert from CChainParams (Jorge Timón)
@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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants