Skip to content

Commit

Permalink
Disable PyModule::filename on PyPy
Browse files Browse the repository at this point in the history
  • Loading branch information
messense authored and davidhewitt committed Nov 1, 2022
1 parent 522ac1c commit 5bbd21c
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions newsfragments/2715.fixed.md
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Disable `PyModule::filename` on PyPy
1 change: 1 addition & 0 deletions pyo3-ffi/src/moduleobject.rs
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ extern "C" {
#[cfg(not(all(windows, PyPy)))]
#[deprecated(note = "Python 3.2")]
pub fn PyModule_GetFilename(arg1: *mut PyObject) -> *const c_char;
#[cfg(not(PyPy))]
pub fn PyModule_GetFilenameObject(arg1: *mut PyObject) -> *mut PyObject;
// skipped non-limited _PyModule_Clear
// skipped non-limited _PyModule_ClearDict
Expand Down
2 changes: 1 addition & 1 deletion src/types/module.rs
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,7 @@ impl PyModule {
/// Returns the filename (the `__file__` attribute) of the module.
///
/// May fail if the module does not have a `__file__` attribute.
#[cfg(not(all(windows, PyPy)))]
#[cfg(not(PyPy))]
pub fn filename(&self) -> PyResult<&str> {
unsafe {
self.py()
Expand Down

0 comments on commit 5bbd21c

Please sign in to comment.