about summary refs log tree commit diff
path: root/src/librustc_errors
diff options
context:
space:
mode:
authorJosh Stone <jistone@redhat.com>2020-05-04 11:52:15 -0700
committerJosh Stone <jistone@redhat.com>2020-05-04 11:52:15 -0700
commit1a439d286530a020bbf7683011f07227e236bd7f (patch)
treef8dc81ca78bef50d6cc047557ed4e03e5857f13c /src/librustc_errors
parent6318d24ad8440fa30428b405be1174478e9536e3 (diff)
downloadrust-1a439d286530a020bbf7683011f07227e236bd7f.tar.gz
rust-1a439d286530a020bbf7683011f07227e236bd7f.zip
Simplify the error Registry methods a little
Diffstat (limited to 'src/librustc_errors')
-rw-r--r--src/librustc_errors/registry.rs9
1 files changed, 3 insertions, 6 deletions
diff --git a/src/librustc_errors/registry.rs b/src/librustc_errors/registry.rs
index 32700c6500b..b1d770d5bd5 100644
--- a/src/librustc_errors/registry.rs
+++ b/src/librustc_errors/registry.rs
@@ -10,12 +10,12 @@ pub struct Registry {
 
 impl Registry {
     pub fn new(long_descriptions: &[(&'static str, Option<&'static str>)]) -> Registry {
-        Registry { long_descriptions: long_descriptions.iter().cloned().collect() }
+        Registry { long_descriptions: long_descriptions.iter().copied().collect() }
     }
 
     /// This will panic if an invalid error code is passed in
     pub fn find_description(&self, code: &str) -> Option<&'static str> {
-        self.try_find_description(code).unwrap()
+        self.long_descriptions[code]
     }
     /// Returns `InvalidErrorCode` if the code requested does not exist in the
     /// registry. Otherwise, returns an `Option` where `None` means the error
@@ -24,9 +24,6 @@ impl Registry {
         &self,
         code: &str,
     ) -> Result<Option<&'static str>, InvalidErrorCode> {
-        if !self.long_descriptions.contains_key(code) {
-            return Err(InvalidErrorCode);
-        }
-        Ok(*self.long_descriptions.get(code).unwrap())
+        self.long_descriptions.get(code).copied().ok_or(InvalidErrorCode)
     }
 }