Merge pull request #584 from swimos/rust-1.73

Upgrades to rust 1.73

Author: SirCipher

api/swim_api/src/downlink/mod.rs

@@ -32,6 +32,8 @@ pub enum DownlinkKind {
     Event,
     /// Accepts key-value pairs and maintains a state as a map.
     Map,
+    /// Accepts map updates but does not maintain any state.
+    MapEvent,
 }
 
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]

api/swim_api/src/protocol/map/mod.rs

@@ -456,9 +456,9 @@ impl<K: StructuralWritable, V: StructuralWritable> Encoder<MapOperation<K, V>>
     }
 }
 
-/// Repreesentation of map lane messages (used to form the body of Recon messages when operating)
+/// Representation of map lane messages (used to form the body of Recon messages when operating)
 /// on downlinks. This extends [`MapOperation`] with `Take` (retain the first `n` items) and `Drop`
-/// (remove teh first `n` items). We never use these internally but must support them for communicating
+/// (remove the first `n` items). We never use these internally but must support them for communicating
 /// with other implementations.
 #[derive(Copy, Clone, Debug, PartialEq, Eq, Form, Hash)]
 #[form_root(::swim_form)]

api/swim_model/src/attr.rs

@@ -138,7 +138,7 @@ impl From<Attr> for Value {
 
 impl PartialOrd for Attr {
     fn partial_cmp(&self, other: &Self) -> Option<Ordering> {
-        Some(self.compare(other))
+        Some(self.cmp(other))
     }
 }
 

api/swim_model/src/http/header.rs

@@ -259,7 +259,7 @@ impl Name {
 
 impl PartialOrd for Name {
     fn partial_cmp(&self, other: &Self) -> Option<Ordering> {
-        self.str_value().partial_cmp(other.str_value())
+        Some(self.cmp(other))
     }
 }
 

api/swim_model/src/item.rs

@@ -75,7 +75,7 @@ impl Item {
 
 impl PartialOrd for Item {
     fn partial_cmp(&self, other: &Self) -> Option<Ordering> {
-        Some(self.compare(other))
+        Some(self.cmp(other))
     }
 }
 

api/swim_model/src/text/mod.rs

@@ -345,7 +345,7 @@ impl PartialEq<&mut str> for Text {
 
 impl PartialOrd for Text {
     fn partial_cmp(&self, other: &Self) -> Option<Ordering> {
-        self.as_str().partial_cmp(other.as_str())
+        Some(self.cmp(other))
     }
 }
 

api/swim_model/src/value.rs

@@ -685,7 +685,7 @@ impl Eq for Value {}
 
 impl PartialOrd for Value {
     fn partial_cmp(&self, other: &Self) -> Option<Ordering> {
-        Some(self.compare(other))
+        Some(self.cmp(other))
     }
 }
 

azure-pipelines.yml

@@ -3,7 +3,7 @@ pr: ["main", "byte-routing"]
 
 variables:
   RUSTFLAGS: -Dwarnings
-  RUST_VERSION: 1.72.0
+  RUST_VERSION: 1.73.0
 
 stages:
   - stage: Lint

ci/azure-code-coverage.yml

@@ -23,7 +23,7 @@ jobs:
     steps:
       - template: azure-install-rust.yml
         parameters:
-          rust_version: 1.72.0
+          rust_version: 1.73.0
       - template: azure-install-sccache.yml
 
       - script: cargo install cargo-tarpaulin

client/runtime/src/models.rs

@@ -163,7 +163,7 @@ impl DownlinkRuntime {
                     );
                     runtime.run().await;
                 }
-                DownlinkKind::Map => {
+                DownlinkKind::Map | DownlinkKind::MapEvent => {
                     let strategy = if config.abort_on_bad_frames {
                         ReportStrategy::new(AlwaysAbortStrategy).boxed()
                     } else {