Skip to content
This repository was archived by the owner on Nov 6, 2020. It is now read-only.

Remove unused code #9884

Merged
merged 1 commit into from
Nov 8, 2018
Merged

Remove unused code #9884

merged 1 commit into from
Nov 8, 2018

Conversation

u2
Copy link
Contributor

@u2 u2 commented Nov 8, 2018

No description provided.

@parity-cla-bot
Copy link

It looks like @u2 signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

sorpaas
sorpaas approved these changes Nov 8, 2018
View reviewed changes
Copy link
Collaborator

@sorpaas sorpaas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed, Memory::dump is not used at all.

@niklasad1 niklasad1 added A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust. labels Nov 8, 2018
@niklasad1 niklasad1 merged commit aafe527 into openethereum:master Nov 8, 2018
@5chdn 5chdn added this to the 2.3 milestone Nov 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@sorpaas sorpaas sorpaas approved these changes

@niklasad1 niklasad1 niklasad1 approved these changes

Assignees
No one assigned
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Milestone
2.3
Development

Successfully merging this pull request may close these issues.
5 participants
@u2 @parity-cla-bot @sorpaas @niklasad1 @5chdn