Merge pull request #3703 from davidhewitt/module2

implement `PyModuleMethods`

src/prelude.rs

@@ -34,6 +34,7 @@ pub use crate::types::float::PyFloatMethods;
 pub use crate::types::frozenset::PyFrozenSetMethods;
 pub use crate::types::list::PyListMethods;
 pub use crate::types::mapping::PyMappingMethods;
+pub use crate::types::module::PyModuleMethods;
 pub use crate::types::sequence::PySequenceMethods;
 pub use crate::types::set::PySetMethods;
 pub use crate::types::string::PyStringMethods;

src/py_result_ext.rs

@@ -1,4 +1,4 @@
-use crate::{types::any::PyAnyMethods, Bound, PyAny, PyResult};
+use crate::{types::any::PyAnyMethods, Bound, PyAny, PyResult, PyTypeCheck};
 
 mod sealed {
     use super::*;
@@ -11,10 +11,16 @@ mod sealed {
 use sealed::Sealed;
 
 pub(crate) trait PyResultExt<'py>: Sealed {
+    fn downcast_into<T: PyTypeCheck>(self) -> PyResult<Bound<'py, T>>;
     unsafe fn downcast_into_unchecked<T>(self) -> PyResult<Bound<'py, T>>;
 }
 
 impl<'py> PyResultExt<'py> for PyResult<Bound<'py, PyAny>> {
+    #[inline]
+    fn downcast_into<T: PyTypeCheck>(self) -> PyResult<Bound<'py, T>> where {
+        self.and_then(|instance| instance.downcast_into().map_err(Into::into))
+    }
+
     #[inline]
     unsafe fn downcast_into_unchecked<T>(self) -> PyResult<Bound<'py, T>> {
         self.map(|instance| instance.downcast_into_unchecked())

src/types/mod.rs

@@ -294,7 +294,7 @@ mod iterator;
 pub(crate) mod list;
 pub(crate) mod mapping;
 mod memoryview;
-mod module;
+pub(crate) mod module;
 mod none;
 mod notimplemented;
 mod num;