diff --git a/pyo3-macros-backend/src/method.rs b/pyo3-macros-backend/src/method.rs index 9bd33cb5a8e..3d97e70ad56 100644 --- a/pyo3-macros-backend/src/method.rs +++ b/pyo3-macros-backend/src/method.rs @@ -116,12 +116,12 @@ impl FnType { } FnType::FnClass | FnType::FnNewClass => { quote! { - _pyo3::types::PyType::from_type_ptr(_py, _slf as *mut _pyo3::ffi::PyTypeObject), + _pyo3::types::PyType::from_type_ptr(py, _slf as *mut _pyo3::ffi::PyTypeObject), } } FnType::FnModule => { quote! { - _py.from_borrowed_ptr::<_pyo3::types::PyModule>(_slf), + py.from_borrowed_ptr::<_pyo3::types::PyModule>(_slf), } } } @@ -156,7 +156,7 @@ impl ExtractErrorMode { impl SelfType { pub fn receiver(&self, cls: &syn::Type, error_mode: ExtractErrorMode) -> TokenStream { - let py = syn::Ident::new("_py", Span::call_site()); + let py = syn::Ident::new("py", Span::call_site()); let slf = syn::Ident::new("_slf", Span::call_site()); match self { SelfType::Receiver { span, mutable } => { @@ -421,7 +421,7 @@ impl<'a> FnSpec<'a> { ) -> Result { let deprecations = &self.deprecations; let self_arg = self.tp.self_arg(cls, ExtractErrorMode::Raise); - let py = syn::Ident::new("_py", Span::call_site()); + let py = syn::Ident::new("py", Span::call_site()); let func_name = &self.name; let rust_call = |args: Vec| { diff --git a/pyo3-macros-backend/src/pyclass.rs b/pyo3-macros-backend/src/pyclass.rs index ca5d233eed0..b4c412e78ae 100644 --- a/pyo3-macros-backend/src/pyclass.rs +++ b/pyo3-macros-backend/src/pyclass.rs @@ -1070,7 +1070,7 @@ impl<'a> PyClassImplsBuilder<'a> { quote! { impl _pyo3::impl_::pyclass::PyClassWithFreeList for #cls { #[inline] - fn get_free_list(_py: _pyo3::Python<'_>) -> &mut _pyo3::impl_::freelist::FreeList<*mut _pyo3::ffi::PyObject> { + fn get_free_list(py: _pyo3::Python<'_>) -> &mut _pyo3::impl_::freelist::FreeList<*mut _pyo3::ffi::PyObject> { static mut FREELIST: *mut _pyo3::impl_::freelist::FreeList<*mut _pyo3::ffi::PyObject> = 0 as *mut _; unsafe { if FREELIST.is_null() { diff --git a/pyo3-macros-backend/src/pymethod.rs b/pyo3-macros-backend/src/pymethod.rs index d5860f83809..87d745dcbab 100644 --- a/pyo3-macros-backend/src/pymethod.rs +++ b/pyo3-macros-backend/src/pymethod.rs @@ -490,7 +490,7 @@ fn impl_call_setter( let name = &spec.name; let fncall = if py_arg.is_some() { - quote!(#cls::#name(#slf, _py, _val)) + quote!(#cls::#name(#slf, py, _val)) } else { quote!(#cls::#name(#slf, _val)) }; @@ -556,18 +556,18 @@ pub fn impl_py_setter_def( let associated_method = quote! { #cfg_attrs unsafe fn #wrapper_ident( - _py: _pyo3::Python<'_>, + py: _pyo3::Python<'_>, _slf: *mut _pyo3::ffi::PyObject, _value: *mut _pyo3::ffi::PyObject, ) -> _pyo3::PyResult<::std::os::raw::c_int> { - let _value = _py + let _value = py .from_borrowed_ptr_or_opt(_value) .ok_or_else(|| { _pyo3::exceptions::PyAttributeError::new_err("can't delete attribute") })?; let _val = _pyo3::FromPyObject::extract(_value)?; - _pyo3::callback::convert(_py, #setter_impl) + _pyo3::callback::convert(py, #setter_impl) } }; @@ -603,7 +603,7 @@ fn impl_call_getter( let name = &spec.name; let fncall = if py_arg.is_some() { - quote!(#cls::#name(#slf, _py)) + quote!(#cls::#name(#slf, py)) } else { quote!(#cls::#name(#slf)) }; @@ -650,7 +650,7 @@ pub fn impl_py_getter_def( } => { let call = impl_call_getter(cls, spec, self_type)?; quote! { - _pyo3::callback::convert(_py, #call) + _pyo3::callback::convert(py, #call) } } }; @@ -682,7 +682,7 @@ pub fn impl_py_getter_def( let associated_method = quote! { #cfg_attrs unsafe fn #wrapper_ident( - _py: _pyo3::Python<'_>, + py: _pyo3::Python<'_>, _slf: *mut _pyo3::ffi::PyObject ) -> _pyo3::PyResult<*mut _pyo3::ffi::PyObject> { #body @@ -1083,7 +1083,7 @@ impl SlotDef { spec.name.span() => format!("`{}` must be `unsafe fn`", method_name) ); } - let py = syn::Ident::new("_py", Span::call_site()); + let py = syn::Ident::new("py", Span::call_site()); let arg_types: &Vec<_> = &arguments.iter().map(|arg| arg.ffi_type()).collect(); let arg_idents: &Vec<_> = &(0..arguments.len()) .map(|i| format_ident!("arg{}", i)) @@ -1191,7 +1191,7 @@ impl SlotFragmentDef { let fragment_trait = format_ident!("PyClass{}SlotFragment", fragment); let method = syn::Ident::new(fragment, Span::call_site()); let wrapper_ident = format_ident!("__pymethod_{}__", fragment); - let py = syn::Ident::new("_py", Span::call_site()); + let py = syn::Ident::new("py", Span::call_site()); let arg_types: &Vec<_> = &arguments.iter().map(|arg| arg.ffi_type()).collect(); let arg_idents: &Vec<_> = &(0..arguments.len()) .map(|i| format_ident!("arg{}", i))