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

remove util-error #9054

Merged
merged 3 commits into from
Jul 6, 2018
Merged

remove util-error #9054

merged 3 commits into from
Jul 6, 2018

Conversation

debris
Copy link
Collaborator

@debris debris commented Jul 5, 2018

simplifies error handling in journaldb and fixes misuses of UtilError in other modules

@debris debris added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Jul 5, 2018
@debris debris requested a review from dvdplm July 5, 2018 15:05
@5chdn 5chdn added this to the 2.0 milestone Jul 5, 2018
Copy link
Collaborator

@dvdplm dvdplm left a comment

Choose a reason for hiding this comment

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

lgtm, afaict there is nothing in the current codebase that actually does anything with BaseDataError (dunno, like taking different fault recovery actions depending on the variant etc) so might as well simplify.

@@ -152,7 +152,7 @@ impl Default for ClientConfig {
}
#[cfg(test)]
mod test {
use super::{DatabaseCompactionProfile, Mode};
use super::{DatabaseCompactionProfile};
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove parentheses!

@@ -24,7 +24,7 @@ pub enum Error {
/// TrieDB-related error.
Trie(TrieError),
/// Util error
Copy link
Collaborator

@niklasad1 niklasad1 Jul 5, 2018

Choose a reason for hiding this comment

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

s/Util error/Io error?

Copy link
Collaborator

@niklasad1 niklasad1 left a comment

Choose a reason for hiding this comment

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

More code removed than added 👍

@niklasad1 niklasad1 added the A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging. label Jul 6, 2018
@5chdn 5chdn removed the A0-pleasereview 🤓 Pull request needs code review. label Jul 6, 2018
@debris debris added A0-pleasereview 🤓 Pull request needs code review. and removed A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging. labels Jul 6, 2018
@niklasad1 niklasad1 added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Jul 6, 2018
@niklasad1 niklasad1 merged commit 8d171a3 into master Jul 6, 2018
@niklasad1 niklasad1 deleted the remove_util_error branch July 6, 2018 13:09
dvdplm added a commit that referenced this pull request Jul 6, 2018
ordian added a commit to ordian/parity that referenced this pull request Jul 9, 2018
…rp_sync_on_light_client

* 'master' of https://github.com/paritytech/parity:
  docs: add changelog for 1.10.9 stable and 1.11.6 beta (openethereum#9069)
  Enable test in `miner/pool/test` (openethereum#9072)
  fetch: replace futures-timer with tokio-timer (openethereum#9066)
  remove util-error (openethereum#9054)
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. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants