Skip to content

Commit 7ab12fc

Browse files
committed
Use Error variants locally
Add a `use Error::*` line to use error variants locally, while we are at it make usage uniform.
1 parent f33d2b0 commit 7ab12fc

File tree

1 file changed

+10
-8
lines changed

1 file changed

+10
-8
lines changed

src/lib.rs

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -679,22 +679,24 @@ pub enum Error {
679679

680680
impl fmt::Display for Error {
681681
fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
682+
use Error::*;
683+
682684
match *self {
683-
Error::MissingSeparator => write!(f, "missing human-readable separator, \"{}\"", SEP),
684-
Error::InvalidChecksum => write!(f, "invalid checksum"),
685-
Error::InvalidLength => write!(f, "invalid length"),
686-
Error::InvalidChar(n) => write!(f, "invalid character (code={})", n),
687-
Error::InvalidData(n) => write!(f, "invalid data point ({})", n),
688-
Error::InvalidPadding => write!(f, "invalid padding"),
689-
Error::MixedCase => write!(f, "mixed-case strings not allowed"),
685+
MissingSeparator => write!(f, "missing human-readable separator, \"{}\"", SEP),
686+
InvalidChecksum => write!(f, "invalid checksum"),
687+
InvalidLength => write!(f, "invalid length"),
688+
InvalidChar(n) => write!(f, "invalid character (code={})", n),
689+
InvalidData(n) => write!(f, "invalid data point ({})", n),
690+
InvalidPadding => write!(f, "invalid padding"),
691+
MixedCase => write!(f, "mixed-case strings not allowed"),
690692
}
691693
}
692694
}
693695

694696
#[cfg(feature = "std")]
695697
impl std::error::Error for Error {
696698
fn source(&self) -> Option<&(dyn std::error::Error + 'static)> {
697-
use self::Error::*;
699+
use Error::*;
698700

699701
match *self {
700702
MissingSeparator | InvalidChecksum | InvalidLength | InvalidChar(_)

0 commit comments

Comments
 (0)