Skip to content

Conversation

@laanwj
Copy link
Member

@laanwj laanwj commented Mar 21, 2012

As these are equivalent functions, this cleans up the source a bit.

Also remove PrintConsole() as this was the only use.

As these are equivalent functions, this cleans up the source a bit.
Also remove PrintConsole() as this was the only use.
@gavinandresen
Copy link
Contributor

ACK. I love pulls that remove code...

@sipa
Copy link
Member

sipa commented Mar 22, 2012

ACK

@laanwj
Copy link
Member Author

laanwj commented Mar 22, 2012

Indeed, removing code always feels good

laanwj added a commit that referenced this pull request Mar 22, 2012
replace call to PrintConsole with call to error()
@laanwj laanwj merged commit 5cd806a into bitcoin:master Mar 22, 2012
@luke-jr luke-jr mentioned this pull request Mar 25, 2012
coblee referenced this pull request in litecoin-project/litecoin Jul 17, 2012
replace call to PrintConsole with call to error()
@laanwj laanwj deleted the 2012_03_printconsole_error branch April 9, 2014 14:14
suprnurd pushed a commit to chaincoin-legacy/chaincoin that referenced this pull request Dec 5, 2017
da406e3 Masternode sync improvements
    - add simple helpers for few more sync states (use them where appropriate instead of old code + rpc output)
    - use new helpers to avoid meaningless message processing
    - actually fail if sync shouldn't continue due to lack of info, make sure Reset is used to quit failed state
@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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants