Address PR feedback

src/common/disk.rs

@@ -149,6 +149,8 @@ impl Disk {
 
     /// Returns number of bytes read and written by the disk
     ///
+    /// ⚠️ Note that FreeBSD is not yet supported
+    ///
     /// ```no_run
     /// use sysinfo::Disks;
     ///

src/common/mod.rs

@@ -39,6 +39,7 @@ pub(crate) mod user;
 ///     println!("[{:?}] disk usage: {:?}", disk.name(), disk.usage());
 /// }
 /// ```
+#[cfg(any(feature = "disk", feature = "system"))]
 #[derive(Debug, Default, Clone, Copy, PartialEq, Eq, PartialOrd)]
 pub struct DiskUsage {
     /// Total number of written bytes.

src/unix/apple/macos/disk.rs

@@ -115,11 +115,7 @@ pub(crate) fn get_disk_type(bsd_name: &[u8]) -> Option<DiskKind> {
         }) {
             Some(disk_type)
         } else {
-            // Many external drive vendors do not advertise their device's storage medium.
-            //
-            // In these cases, assuming that there were _any_ properties about them registered, we fallback
-            // to `HDD` when no storage medium is provided by the device instead of `Unknown`.
-            Some(DiskKind::HDD)
+            Some(DiskKind::Unknown(-1))
         }
     })
 }

src/unknown/disk.rs

@@ -8,11 +8,11 @@ pub(crate) struct DiskInner;
 
 impl DiskInner {
     pub(crate) fn kind(&self) -> DiskKind {
-        unreachable!()
+        DiskKind::Unknown(-1)
     }
 
     pub(crate) fn name(&self) -> &OsStr {
-        unreachable!()
+        OsStr::new("")
     }
 
     pub(crate) fn file_system(&self) -> &OsStr {
@@ -44,7 +44,7 @@ impl DiskInner {
     }
 
     pub(crate) fn usage(&self) -> DiskUsage {
-        unreachable!()
+        DiskUsage::default()
     }
 }
 

src/windows/disk.rs

@@ -417,12 +417,10 @@ fn get_disk_io(device_path: &[u16], handle: Option<HandleWrapper>) -> Option<(u6
         sysinfo_debug!("Error: DeviceIoControl(IOCTL_DISK_PERFORMANCE) = {:?}", err);
         err
     })
-    .ok()
-    .and_then(|_| {
-        disk_perf
-            .BytesRead
-            .try_into()
-            .ok()
-            .zip(disk_perf.BytesWritten.try_into().ok())
-    })
+    .ok()?;
+
+    Some((
+        disk_perf.BytesRead.try_into().ok()?,
+        disk_perf.BytesWritten.try_into().ok()?,
+    ))
 }