fix clippy and msrv

PyO3 · Oct 22, 2024 · 3021c46 · 3021c46
1 parent c783679
commit 3021c46
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/pyo3-macros-backend/src/module.rs b/pyo3-macros-backend/src/module.rs
@@ -355,7 +355,8 @@ pub fn pymodule_module_impl(
         options.submodule.is_some(),
         options
             .supports_free_threaded
-            .is_some_and(|op| op.value.value),
+            .map(|op| op.value.value)
+            .unwrap_or(false),
     );
 
     Ok(quote!(
@@ -402,7 +403,8 @@ pub fn pymodule_function_impl(
         false,
         options
             .supports_free_threaded
-            .is_some_and(|op| op.value.value),
+            .map(|op| op.value.value)
+            .unwrap_or(false),
     );
 
     // Module function called with optional Python<'_> marker as first arg, followed by the module.

diff --git a/src/types/module.rs b/src/types/module.rs
@@ -560,6 +560,7 @@ impl<'py> PyModuleMethods<'py> for Bound<'py, PyModule> {
                 0..=c_int::MAX => Ok(()),
             }
         }
+        #[cfg(any(Py_LIMITED_API, not(Py_GIL_DISABLED)))]
         Ok(())
     }
 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -560,6 +560,7 @@ impl<'py> PyModuleMethods<'py> for Bound<'py, PyModule> { @@
 ..=c_int::MAX => Ok(()),
                 }
             }
+            #[cfg(any(Py_LIMITED_API, not(Py_GIL_DISABLED)))]
             Ok(())
         }
     }
@@ Expand Down @@