Skip to content

add InitMessage() to noui and use debug.log for GUI#2171

Merged
laanwj merged 1 commit intobitcoin:masterfrom
Diapolo:init
Jan 19, 2013
Merged

add InitMessage() to noui and use debug.log for GUI#2171
laanwj merged 1 commit intobitcoin:masterfrom
Diapolo:init

Conversation

@Diapolo
Copy link
Copy Markdown

@Diapolo Diapolo commented Jan 11, 2013

  • this pull adds an InitMessage() function to noui.cpp, which outputs init
    messages to debug.log (this allows to remove some printf() calls from
    init.cpp)
  • change InitMessage() in bitcoin.cpp to also write init messages to
    debug.log to ensure nothting is missing in the log because of the
    removal of printf() calls in init.cpp

- this pull adds an InitMessage() function to noui.cpp, which outputs init
  messages to debug.log (this allows to remove some printf() calls from
  init.cpp)
- change InitMessage() in bitcoin.cpp to also write init messages to
  debug.log to ensure nothting is missing in the log because of the
  removal of printf() calls in init.cpp
@laanwj
Copy link
Copy Markdown
Member

laanwj commented Jan 13, 2013

ACK

1 similar comment
@sipa
Copy link
Copy Markdown
Member

sipa commented Jan 13, 2013

ACK

@BitcoinPullTester
Copy link
Copy Markdown

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/bb41a87d57ac885dcd951a11827ca57329224802 for binaries and test log.

@Diapolo
Copy link
Copy Markdown
Author

Diapolo commented Jan 19, 2013

/merge me ;)?

laanwj added a commit that referenced this pull request Jan 19, 2013
add InitMessage() to noui and use debug.log for GUI
@laanwj laanwj merged commit 0c16cc7 into bitcoin:master Jan 19, 2013
laudney pushed a commit to reddcoin-project/reddcoin-3.10 that referenced this pull request Mar 19, 2014
add InitMessage() to noui and use debug.log for GUI
@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.

4 participants