upgrade to Rust v1.46.0

rust-toolchain

@@ -1 +1 @@
-1.45.2
+1.46.0

src/rust/engine/fs/brfs/src/main.rs

@@ -26,7 +26,7 @@
 #![allow(clippy::new_without_default, clippy::new_ret_no_self)]
 // Arc<Mutex> can be more clear than needing to grok Orderings:
 #![allow(clippy::mutex_atomic)]
-#![type_length_limit = "1303944"]
+#![type_length_limit = "44109434"]
 
 use std::collections::hash_map::Entry::{Occupied, Vacant};
 use std::collections::HashMap;

src/rust/engine/fs/src/glob_matching.rs

@@ -536,6 +536,7 @@ trait GlobMatchingImplementation<E: Display + Send + Sync + 'static>: VFS<E> {
       .into_inner()
       .into_iter()
       .collect::<Vec<_>>();
+    #[allow(clippy::unnecessary_sort_by)]
     path_stats.sort_by(|a, b| a.path().cmp(b.path()));
     path_stats.dedup_by(|a, b| a.path() == b.path());
     Ok(path_stats)

src/rust/engine/fs/src/lib.rs

@@ -504,6 +504,7 @@ impl PosixFS {
           format!("Failed to scan directory {:?}: {}", dir_abs, e),
         )
       })?;
+    #[allow(clippy::unnecessary_sort_by)]
     stats.sort_by(|s1, s2| s1.path().cmp(s2.path()));
     Ok(DirectoryListing(stats))
   }

src/rust/engine/fs/store/src/lib.rs

@@ -26,7 +26,7 @@
 #![allow(clippy::new_without_default, clippy::new_ret_no_self)]
 // Arc<Mutex> can be more clear than needing to grok Orderings:
 #![allow(clippy::mutex_atomic)]
-#![type_length_limit = "8576838"]
+#![type_length_limit = "95595489"]
 
 mod snapshot;
 pub use crate::snapshot::{OneOffStoreFileByDigest, Snapshot, StoreFileByDigest};

src/rust/engine/fs/store/src/snapshot.rs

@@ -42,6 +42,7 @@ impl Snapshot {
     file_digester: S,
     mut path_stats: Vec<PathStat>,
   ) -> Result<Snapshot, String> {
+    #[allow(clippy::unnecessary_sort_by)]
     path_stats.sort_by(|a, b| a.path().cmp(b.path()));
 
     // The helper assumes that if a Path has multiple children, it must be a directory.
@@ -87,6 +88,7 @@ impl Snapshot {
     let mut path_stats =
       Iterator::flatten(path_stats_per_directory.into_iter().map(Vec::into_iter))
         .collect::<Vec<_>>();
+    #[allow(clippy::unnecessary_sort_by)]
     path_stats.sort_by(|l, r| l.path().cmp(&r.path()));
     Ok(Snapshot { digest, path_stats })
   }
@@ -100,6 +102,7 @@ impl Snapshot {
     path_stats: &[PathStat],
   ) -> Result<Digest, String> {
     let mut sorted_path_stats = path_stats.to_owned();
+    #[allow(clippy::unnecessary_sort_by)]
     sorted_path_stats.sort_by(|a, b| a.path().cmp(b.path()));
     Snapshot::ingest_directory_from_sorted_path_stats(store, file_digester, &sorted_path_stats)
       .await

src/rust/engine/process_execution/src/lib.rs

@@ -26,7 +26,7 @@
 #![allow(clippy::new_without_default, clippy::new_ret_no_self)]
 // Arc<Mutex> can be more clear than needing to grok Orderings:
 #![allow(clippy::mutex_atomic)]
-#![type_length_limit = "35811178"]
+#![type_length_limit = "43757804"]
 #[macro_use]
 extern crate derivative;
 

src/rust/engine/src/lib.rs

@@ -30,7 +30,7 @@
 // just the one minor call as unsafe, than to mark the whole function as unsafe which may hide
 // other unsafeness.
 #![allow(clippy::not_unsafe_ptr_arg_deref)]
-#![type_length_limit = "42187898"]
+#![type_length_limit = "43757804"]
 
 mod context;
 mod core;