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.

Sign up for GitHub

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

Jump to bottom

Misc cleanup #1384

Merged
merged 3 commits into from
Nov 19, 2018
Merged

Misc cleanup #1384

merged 3 commits into from
Nov 19, 2018

Conversation

cryptocode
Copy link
Contributor

Silence a few warnings/minor fixups:

  • mdb_val#epoch wasn't initialized by all c'tors
  • dead stores
  • Unused forward declaration

@rkeene rkeene added the quality improvements This item indicates the need for or supplies changes that improve maintainability label Nov 16, 2018
@rkeene rkeene added this to the V18.0 milestone Nov 16, 2018
@rkeene rkeene requested a review from argakiig November 16, 2018 15:17
@rkeene rkeene merged commit 753647b into nanocurrency:master Nov 19, 2018
@cryptocode cryptocode deleted the clean/misc branch November 19, 2018 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@argakiig argakiig argakiig approved these changes

Assignees

@cryptocode cryptocode

Labels
quality improvements This item indicates the need for or supplies changes that improve maintainability
Projects
None yet
Milestone
V18.0
Development

Successfully merging this pull request may close these issues.
3 participants
@cryptocode @argakiig @rkeene