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

Replace deprecated hashdb trait names #1394

Merged
merged 3 commits into from
Jun 23, 2016

Conversation

keorn
Copy link

@keorn keorn commented Jun 22, 2016

No description provided.

@parity-cla-bot
Copy link

It looks like @keorn hasn'signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement at the following URL: https://cla.ethcore.io

Once you've signed, plesae reply to this thread with [clabot:check] to prove it.

Many thanks,

Ethcore CLA Bot

@keorn
Copy link
Author

keorn commented Jun 22, 2016

[clabot:check]

@parity-cla-bot
Copy link

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

Many thanks,

Ethcore CLA Bot

@@ -109,7 +103,7 @@ pub trait HashDB : AsHashDB {
/// assert_eq!(m.get(key).unwrap(), d);
/// }
/// ```
fn remove(&mut self, key: &H256) { self.kill(key) }
fn remove(&mut self, key: &H256);
Copy link
Collaborator

Choose a reason for hiding this comment

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

This file is incorrectly formatted. Use tabs instead of spaces

@rphmeier rphmeier added A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed. A5-grumble 🔥 Pull request has minor issues that must be addressed before merging. and removed A4-gotissues 💥 Pull request is reviewed and has significant issues which must be addressed. labels Jun 22, 2016
@rphmeier rphmeier added A8-looksgood 🦄 Pull request is reviewed well. and removed A5-grumble 🔥 Pull request has minor issues that must be addressed before merging. labels Jun 22, 2016
@gavofyork
Copy link
Contributor

been waiting for this for a while.

@gavofyork gavofyork merged commit be8f922 into openethereum:master Jun 23, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants