Use a macro to get the status values

The bodies of the status methods are identical, so might as well.

Signed-off-by: mulhern <[email protected]>

src/cachedev.rs

@@ -714,8 +714,7 @@ impl CacheDev {
 
     /// Get the current status of the cache device.
     pub fn status(&self, dm: &DM) -> DmResult<CacheDevStatus> {
-        let (_, status) = dm.table_status(&DevId::Name(self.name()), &DmOptions::new())?;
-        get_status(&status)?.parse()
+        status!(self, dm)
     }
 }
 

src/shared_macros.rs

@@ -47,3 +47,13 @@ macro_rules! table {
         &$s.table
     };
 }
+
+macro_rules! status {
+    ($s:ident, $dm:ident) => {
+        get_status(
+            &$dm.table_status(&DevId::Name($s.name()), &DmOptions::new())?
+                .1,
+        )?
+        .parse()
+    };
+}

src/thindev.rs

@@ -381,8 +381,7 @@ impl ThinDev {
 
     /// Get the current status of the thin device.
     pub fn status(&self, dm: &DM) -> DmResult<ThinStatus> {
-        let (_, table) = dm.table_status(&DevId::Name(self.name()), &DmOptions::new())?;
-        get_status(&table)?.parse()
+        status!(self, dm)
     }
 
     /// Set the table for the thin device's target

src/thinpooldev.rs

@@ -548,8 +548,7 @@ impl ThinPoolDev {
     /// Get the current status of the thinpool.
     /// Returns an error if there was an error getting the status value.
     pub fn status(&self, dm: &DM) -> DmResult<ThinPoolStatus> {
-        let (_, status) = dm.table_status(&DevId::Name(self.name()), &DmOptions::new())?;
-        get_status(&status)?.parse()
+        status!(self, dm)
     }
 
     /// Set the table for the existing metadata device.