xen: fix clippy warning

It seems that now with all our cargo features compatible, clippy is
checking more code in CI. Fix a warning about an error variant being
suffixed with the enum name in mmap_xen.rs that only popped up now.

Signed-off-by: Patrick Roy <[email protected]>

Author: roypat

src/mmap_xen.rs

@@ -56,7 +56,7 @@ pub enum Error {
     Fam(FamError),
     /// Unexpected error.
     #[error("Unexpected error")]
-    UnexpectedError,
+    Unexpected,
     /// Error validating a [`FileOffset`]
     #[error("{0}")]
     ValidateFile(#[from] CheckFileOffsetError),
@@ -298,8 +298,8 @@ impl<B: NewBitmap> MmapRegion<B> {
         Ok(MmapRegion {
             bitmap: B::with_len(range.size),
             size: range.size,
-            prot: range.prot.ok_or(Error::UnexpectedError)?,
-            flags: range.flags.ok_or(Error::UnexpectedError)?,
+            prot: range.prot.ok_or(Error::Unexpected)?,
+            flags: range.flags.ok_or(Error::Unexpected)?,
             file_offset: range.file_offset,
             hugetlbfs: range.hugetlbfs,
             mmap,
@@ -567,8 +567,8 @@ impl MmapXenUnix {
         Ok(Self(
             MmapUnix::new(
                 range.size,
-                range.prot.ok_or(Error::UnexpectedError)?,
-                range.flags.ok_or(Error::UnexpectedError)?,
+                range.prot.ok_or(Error::Unexpected)?,
+                range.flags.ok_or(Error::Unexpected)?,
                 fd,
                 offset,
             )?,
@@ -644,8 +644,8 @@ impl MmapXenForeign {
 
         let unix_mmap = MmapUnix::new(
             size,
-            range.prot.ok_or(Error::UnexpectedError)?,
-            range.flags.ok_or(Error::UnexpectedError)? | MAP_SHARED,
+            range.prot.ok_or(Error::Unexpected)?,
+            range.flags.ok_or(Error::Unexpected)? | MAP_SHARED,
             fd,
             f_offset,
         )?;
@@ -858,7 +858,7 @@ impl MmapXenGrant {
             guest_base: range.addr,
             unix_mmap: None,
             file_offset: range.file_offset.as_ref().unwrap().clone(),
-            flags: range.flags.ok_or(Error::UnexpectedError)?,
+            flags: range.flags.ok_or(Error::Unexpected)?,
             size: 0,
             index: 0,
             domid: range.mmap_data,
@@ -867,11 +867,8 @@ impl MmapXenGrant {
         // Region can't be mapped in advance, partial mapping will be done later via
         // `MmapXenSlice`.
         if mmap_flags.mmap_in_advance() {
-            let (unix_mmap, index) = grant.mmap_range(
-                range.addr,
-                range.size,
-                range.prot.ok_or(Error::UnexpectedError)?,
-            )?;
+            let (unix_mmap, index) =
+                grant.mmap_range(range.addr, range.size, range.prot.ok_or(Error::Unexpected)?)?;
 
             grant.unix_mmap = Some(unix_mmap);
             grant.index = index;
@@ -1173,12 +1170,12 @@ mod tests {
         range.file_offset = Some(FileOffset::new(TempFile::new().unwrap().into_file(), 0));
         range.prot = None;
         let r = MmapXenForeign::new(&range);
-        assert!(matches!(r.unwrap_err(), Error::UnexpectedError));
+        assert!(matches!(r.unwrap_err(), Error::Unexpected));
 
         let mut range = MmapRange::initialized(true);
         range.flags = None;
         let r = MmapXenForeign::new(&range);
-        assert!(matches!(r.unwrap_err(), Error::UnexpectedError));
+        assert!(matches!(r.unwrap_err(), Error::Unexpected));
 
         let mut range = MmapRange::initialized(true);
         range.file_offset = Some(FileOffset::new(TempFile::new().unwrap().into_file(), 1));
@@ -1208,7 +1205,7 @@ mod tests {
         let mut range = MmapRange::initialized(true);
         range.prot = None;
         let r = MmapXenGrant::new(&range, MmapXenFlags::empty());
-        assert!(matches!(r.unwrap_err(), Error::UnexpectedError));
+        assert!(matches!(r.unwrap_err(), Error::Unexpected));
 
         let mut range = MmapRange::initialized(true);
         range.prot = None;
@@ -1218,7 +1215,7 @@ mod tests {
         let mut range = MmapRange::initialized(true);
         range.flags = None;
         let r = MmapXenGrant::new(&range, MmapXenFlags::NO_ADVANCE_MAP);
-        assert!(matches!(r.unwrap_err(), Error::UnexpectedError));
+        assert!(matches!(r.unwrap_err(), Error::Unexpected));
 
         let mut range = MmapRange::initialized(true);
         range.file_offset = Some(FileOffset::new(TempFile::new().unwrap().into_file(), 1));