Fix after merge

ShaharNaveh · ShaharNaveh · commit 18acc26a5952 · 2026-05-16T13:33:26.000+03:00
diff --git a/crates/vm/src/stdlib/_io.rs b/crates/vm/src/stdlib/_io.rs
@@ -147,7 +147,6 @@ mod _io {
     use num_traits::ToPrimitive;
     use std::io::{self, Cursor, SeekFrom, prelude::*};
 
-    #[allow(clippy::let_and_return)]
     fn validate_whence(whence: i32) -> bool {
         host_io::validate_whence(whence)
     }
@@ -204,7 +203,7 @@ mod _io {
         ))
     }
 
-    #[derive(FromArgs)]
+    #[derive(Clone, Copy, FromArgs)]
     pub(super) struct OptionalSize {
         // In a few functions, the default value is -1 rather than None.
         // Make sure the default value doesn't affect compatibility.
@@ -213,7 +212,6 @@ mod _io {
     }
 
     impl OptionalSize {
-        #[allow(clippy::wrong_self_convention)]
         pub(super) fn to_usize(self) -> Option<usize> {
             self.size?.to_usize()
         }
@@ -2907,7 +2905,7 @@ mod _io {
             Ok(())
         }
 
-        #[allow(clippy::type_complexity)]
+        #[expect(clippy::type_complexity, reason = "ignore warning for now")]
         fn find_coder(
             buffer: &PyObject,
             encoding: &str,
diff --git a/crates/vm/src/stdlib/os.rs b/crates/vm/src/stdlib/os.rs
@@ -653,7 +653,10 @@ pub(super) mod _os {
             match self.stat(self.stat_dir_fd(), FollowSymlinks(follow_symlinks), vm) {
                 Ok(stat_obj) => {
                     let st_mode: i32 = stat_obj.get_attr("st_mode", vm)?.try_into_value(vm)?;
-                    #[allow(clippy::unnecessary_cast)]
+                    #[allow(
+                        clippy::unnecessary_cast,
+                        reason = "'st_mode' and 'S_IFMT' are not u32 on all platforms"
+                    )]
                     Ok((st_mode as u32 & libc::S_IFMT as u32) == mode_bits)
                 }
                 Err(e) => {
@@ -1244,6 +1247,7 @@ pub(super) mod _os {
             #[cfg(not(windows))]
             #[allow(clippy::useless_conversion, reason = "needed for 32-bit platforms")]
             let st_blksize = i64::from(stat.st_blksize);
+
             #[cfg(not(windows))]
             #[allow(clippy::useless_conversion, reason = "needed for 32-bit platforms")]
             let st_blocks = i64::from(stat.st_blocks);

Original file line number	Diff line number	Diff line change
`@@ -147,7 +147,6 @@ mod _io {`
`147`	`147`	`use num_traits::ToPrimitive;`
`148`	`148`	`use std::io::{self, Cursor, SeekFrom, prelude::*};`
`149`	`149`
`150`		`- #[allow(clippy::let_and_return)]`
`151`	`150`	`fn validate_whence(whence: i32) -> bool {`
`152`	`151`	`host_io::validate_whence(whence)`
`153`	`152`	`}`
`@@ -204,7 +203,7 @@ mod _io {`
`204`	`203`	`))`
`205`	`204`	`}`
`206`	`205`
`207`		`- #[derive(FromArgs)]`
	`206`	`+ #[derive(Clone, Copy, FromArgs)]`
`208`	`207`	`pub(super) struct OptionalSize {`
`209`	`208`	`// In a few functions, the default value is -1 rather than None.`
`210`	`209`	`// Make sure the default value doesn't affect compatibility.`
`@@ -213,7 +212,6 @@ mod _io {`
`213`	`212`	`}`
`214`	`213`
`215`	`214`	`impl OptionalSize {`
`216`		`- #[allow(clippy::wrong_self_convention)]`
`217`	`215`	`pub(super) fn to_usize(self) -> Option<usize> {`
`218`	`216`	`self.size?.to_usize()`
`219`	`217`	`}`
`@@ -2907,7 +2905,7 @@ mod _io {`
`2907`	`2905`	`Ok(())`
`2908`	`2906`	`}`
`2909`	`2907`
`2910`		`- #[allow(clippy::type_complexity)]`
	`2908`	`+ #[expect(clippy::type_complexity, reason = "ignore warning for now")]`
`2911`	`2909`	`fn find_coder(`
`2912`	`2910`	`buffer: &PyObject,`
`2913`	`2911`	`encoding: &str,`