Bump to Rust 1.81.0 (#513)

* Update to Rust 1.81

* Fix clippy complaints

* Make sorting stable and update trycmd tests

Author: mkeeter

.cargo/config.toml

@@ -0,0 +1,5 @@
+[alias]
+xtask = "run --package xtask --"
+
+[profile.release]
+overflow-checks = true

cmd/counters/src/lib.rs

@@ -512,14 +512,12 @@ fn counters_dump_csv(
 ) {
     // Sort the counters.
     match opts.sort {
-        Some(Order::Value) => counters.sort_unstable_by(
+        Some(Order::Value) => counters.sort_by(
             &mut |_, a: &CounterVariant, _, b: &CounterVariant| {
                 a.total().cmp(&b.total()).reverse()
             },
         ),
-        Some(Order::Alpha) => {
-            counters.sort_unstable_by(&mut |a, _, b, _| a.cmp(b))
-        }
+        Some(Order::Alpha) => counters.sort_by(&mut |a, _, b, _| a.cmp(b)),
         _ => {}
     }
 
@@ -574,13 +572,13 @@ fn counter_dump(counters: &mut Counters, opts: &Options, pad: &str) {
         // Therefore, sort by value by default, so the highest-valued counters
         // are shown first.
         Options { sort: Some(Order::Value), .. }
-        | Options { sort: None, full: false, .. } => counters.sort_unstable_by(
+        | Options { sort: None, full: false, .. } => counters.sort_by(
             &mut |_, a: &CounterVariant, _, b: &CounterVariant| {
                 a.total().cmp(&b.total()).reverse()
             },
         ),
         Options { sort: Some(Order::Alpha), .. } => {
-            counters.sort_unstable_by(&mut |a, _, b, _| a.cmp(b))
+            counters.sort_by(&mut |a, _, b, _| a.cmp(b))
         }
     }
 

cmd/dashboard/src/lib.rs

@@ -79,7 +79,7 @@ impl StatefulList {
 
     fn previous(&mut self) {
         self.state.select(match self.state.selected() {
-            Some(ndx) if ndx == 0 => Some(self.n - 1),
+            Some(0) => Some(self.n - 1),
             Some(ndx) => Some(ndx - 1),
             None => Some(0),
         });
@@ -114,8 +114,6 @@ trait Attributes {
     fn decrease(&mut self, _ndx: usize) -> Option<u8> {
         None
     }
-
-    fn clear(&mut self) {}
 }
 
 struct TempGraph;
@@ -185,12 +183,6 @@ impl Attributes for FanGraph {
         self.0[ndx] = if nval >= 0 { nval as u8 } else { 0 };
         Some(self.0[ndx])
     }
-
-    fn clear(&mut self) {
-        for val in self.0.iter_mut() {
-            *val = 0;
-        }
-    }
 }
 
 struct CurrentGraph;

cmd/monorail/src/lib.rs

@@ -756,7 +756,7 @@ fn monorail_status(
         Value::Enum(m) => {
             let mode = m.disc().to_uppercase();
             let speed = m.contents().and_then(|speed| match speed {
-                Value::Tuple(t) => t.get(0).map(|t| match t {
+                Value::Tuple(t) => t.first().map(|t| match t {
                     Value::Enum(t) => t.disc().replace("Speed", ""),
                     v => panic!("Expected enum, got {:?}", v),
                 }),

cmd/net/src/lib.rs

@@ -14,6 +14,7 @@
 //! - Sidecar
 //! - PSC
 //! - `gimletlet-mgmt`
+//!
 //! These PCAs have the KSZ8463 switch + VSC85x2 PHY which is our standard
 //! management network interface.
 //!

cmd/probe/src/lib.rs

@@ -270,7 +270,7 @@ fn probecmd(context: &mut ExecutionContext) -> Result<()> {
                         Ok("progressing")
                     }
                 })
-                .map_or_else(|_| "unable to step", |s| s)
+                .unwrap_or("unable to step")
                 .to_string();
             core.run()?;
             rval

cmd/ringbuf/src/lib.rs

@@ -161,7 +161,7 @@ fn ringbuf_dump(
             // system, and can't easily be compared. Therefore, sort them by the
             // value of the counter, so the most frequently recorded variants
             // are displayed first.
-            counters.sort_unstable_by(
+            counters.sort_by(
                 &mut |_, a: &CounterVariant, _, b: &CounterVariant| {
                     a.total().cmp(&b.total()).reverse()
                 },

cmd/rpc/src/lib.rs

@@ -157,7 +157,7 @@ fn rpc_listen_one(
     interface: u32,
     port: u32,
 ) -> Result<BTreeSet<Target>> {
-    let socket = match UdpSocket::bind(&format!("[::]:{port}")) {
+    let socket = match UdpSocket::bind(format!("[::]:{port}")) {
         Ok(s) => s,
         Err(e) => {
             if e.kind() == std::io::ErrorKind::PermissionDenied {

humility-core/src/core.rs

@@ -1040,7 +1040,7 @@ impl GDBCore {
                 //
                 let val = u64::from_le_bytes(buf[..].try_into().unwrap());
 
-                if val > std::u32::MAX.into() {
+                if val > u32::MAX.into() {
                     Err(anyhow!("bad 64-bit return on cmd {}: {}", cmd, rstr))
                 } else {
                     Ok(val as u32)

humility-core/src/hubris.rs

@@ -799,7 +799,7 @@ impl HubrisArchive {
         // is less than our base.
         //
         for (&(addr, _depth), (len, goff, origin)) in
-            self.inlined.range(..=(pc, std::isize::MAX)).rev()
+            self.inlined.range(..=(pc, isize::MAX)).rev()
         {
             if addr + len < base {
                 break;
@@ -3046,7 +3046,7 @@ impl HubrisArchive {
             return Ok(v.size);
         }
 
-        if self.ptrtypes.get(&goff).is_some() {
+        if self.ptrtypes.contains_key(&goff) {
             return Ok(4);
         }
 
@@ -3970,7 +3970,7 @@ impl HubrisObjectLoader {
             // If this is a zero-sized symbol or not against an allocated
             // section (e.g., .idolatry), we don't want to keep track of it.
             //
-            if sym.st_size == 0 || allocs.get(&sym.st_shndx).is_none() {
+            if sym.st_size == 0 || !allocs.contains(&sym.st_shndx) {
                 continue;
             }