Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix minor build warning and one log #2260

Merged
merged 1 commit into from
Dec 30, 2018
Merged

fix minor build warning and one log #2260

merged 1 commit into from
Dec 30, 2018

Conversation

garyyu
Copy link
Contributor

@garyyu garyyu commented Dec 30, 2018

A few warning and log clean up:

  • fix 2 build warning which is new imported
  • change the boring repeat string StratumServer to 0, the id here is reserved for the case of multiple stratum server threads, and in Grin we only start one stratum server thread.

WARN grin_servers::mining::stratumserver - (Server ID: StratumServer) Solution Found for block

Now it's:

WARN grin_servers::mining::stratumserver - (Server ID: 0) Solution Found for block

  • Add block hash in the solution found log, like Solution Found for block 2035, hash 08321e6c.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@garyyu garyyu merged commit dbf8e97 into mimblewimble:master Dec 30, 2018
@garyyu garyyu deleted the minor-warning branch January 3, 2019 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant