Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add wrap_pyfunction macro to prelude #1695

Merged
merged 1 commit into from
Jun 24, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
- Add `#[pyo3(text_signature = "...")]` syntax for setting text signature. [#1658](https://github.com/PyO3/pyo3/pull/1658)
- Add support for setting and retrieving exception cause. [#1679](https://github.com/PyO3/pyo3/pull/1679)
- Add FFI definitions from `cpython/pystate.h`.[#1687](https://github.com/PyO3/pyo3/pull/1687/)
- Add `wrap_pyfunction` macro to prelude. [#1695](https://github.com/PyO3/pyo3/pull/1695)

### Changed

Expand Down
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,6 @@ features = ["extension-module"]

```rust
use pyo3::prelude::*;
use pyo3::wrap_pyfunction;

/// Formats the sum of two numbers as string.
#[pyfunction]
Expand Down
1 change: 0 additions & 1 deletion examples/pyo3-benchmarks/src/lib.rs
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
use pyo3::prelude::*;
use pyo3::types::{PyDict, PyTuple};
use pyo3::wrap_pyfunction;

#[pyfunction]
fn none() {}
Expand Down
1 change: 0 additions & 1 deletion examples/pyo3-pytests/src/datetime.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ use pyo3::types::{
PyDate, PyDateAccess, PyDateTime, PyDelta, PyDeltaAccess, PyTime, PyTimeAccess, PyTuple,
PyTzInfo,
};
use pyo3::wrap_pyfunction;

#[pyfunction]
fn make_date(py: Python, year: i32, month: u8, day: u8) -> PyResult<&PyDate> {
Expand Down
1 change: 0 additions & 1 deletion examples/pyo3-pytests/src/misc.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
use pyo3::prelude::*;
use pyo3::wrap_pyfunction;

#[pyfunction]
fn issue_219() {
Expand Down
1 change: 0 additions & 1 deletion examples/pyo3-pytests/src/othermod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
//! The code below just tries to use the most important code generation paths
use pyo3::prelude::*;
use pyo3::wrap_pyfunction;

#[pyclass]
pub struct ModClass {
Expand Down
1 change: 0 additions & 1 deletion examples/pyo3-pytests/src/path.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
use pyo3::prelude::*;
use pyo3::wrap_pyfunction;
use std::path::{Path, PathBuf};

#[pyfunction]
Expand Down
1 change: 0 additions & 1 deletion examples/word-count/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
// https://github.com/tildeio/helix-website/blob/master/crates/word_count/src/lib.rs

use pyo3::prelude::*;
use pyo3::wrap_pyfunction;
use rayon::prelude::*;

/// Searches for the word, parallelized by rayon
Expand Down
1 change: 0 additions & 1 deletion guide/src/ecosystem/logging.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ It's also possible to tweak its configuration (mostly to tune its performance).
```rust
use log::info;
use pyo3::prelude::*;
use pyo3::wrap_pyfunction;

#[pyfunction]
fn log_something() {
Expand Down
6 changes: 1 addition & 5 deletions guide/src/function.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ One way is annotating a function with `#[pyfunction]` and then adding it to the

```rust
use pyo3::prelude::*;
use pyo3::wrap_pyfunction;

#[pyfunction]
fn double(x: usize) -> usize {
Expand Down Expand Up @@ -54,7 +53,7 @@ fn rust2py(py: Python, m: &PyModule) -> PyResult<()> {
Ok(format!("{}", a + b))
}

m.add_function(pyo3::wrap_pyfunction!(sum_as_string, m)?)?;
m.add_function(wrap_pyfunction!(sum_as_string, m)?)?;

Ok(())
}
Expand All @@ -72,7 +71,6 @@ The `#[pyo3]` attribute can be used to modify properties of the generated Python

```rust
use pyo3::prelude::*;
use pyo3::wrap_pyfunction;

#[pyfunction]
#[pyo3(name = "no_args")]
Expand All @@ -97,7 +95,6 @@ The `#[pyfunction]` attribute supports specifying details of argument parsing. T

```rust
use pyo3::prelude::*;
use pyo3::wrap_pyfunction;
use pyo3::types::PyDict;

#[pyfunction(kwds="**")]
Expand Down Expand Up @@ -258,7 +255,6 @@ in Python code.
It is possible to access the module of a `#[pyfunction]` in the function body by using `#[pyo3(pass_module)]` option:

```rust
use pyo3::wrap_pyfunction;
use pyo3::prelude::*;

#[pyfunction]
Expand Down
2 changes: 1 addition & 1 deletion guide/src/module.md
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ dicts or other modules:

```rust
use pyo3::prelude::*;
use pyo3::{wrap_pyfunction, wrap_pymodule};
use pyo3::wrap_pymodule;
use pyo3::types::IntoPyDict;

#[pyfunction]
Expand Down
1 change: 0 additions & 1 deletion guide/src/trait_bounds.md
Original file line number Diff line number Diff line change
Expand Up @@ -461,7 +461,6 @@ It is also required to make the struct public.

```rust
use pyo3::prelude::*;
use pyo3::wrap_pyfunction;
use pyo3::types::PyAny;

pub trait Model {
Expand Down
1 change: 0 additions & 1 deletion src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,6 @@
//!
//! ```rust
//! use pyo3::prelude::*;
//! use pyo3::wrap_pyfunction;
//!
//! /// Formats the sum of two numbers as string.
//! #[pyfunction]
Expand Down
1 change: 0 additions & 1 deletion src/num_bigint.rs
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@
//! ```rust
//! use num_bigint::BigInt;
//! use pyo3::prelude::*;
//! use pyo3::wrap_pyfunction;
//!
//! #[pyfunction]
//! fn add_one(n: BigInt) -> BigInt {
Expand Down
1 change: 0 additions & 1 deletion src/num_complex.rs
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@
//! use nalgebra::base::{dimension::Const, storage::Storage, Matrix};
//! use num_complex::Complex;
//! use pyo3::prelude::*;
//! use pyo3::wrap_pyfunction;
//!
//! type T = Complex<f64>;
//!
Expand Down
1 change: 1 addition & 0 deletions src/prelude.rs
Original file line number Diff line number Diff line change
Expand Up @@ -19,5 +19,6 @@ pub use crate::python::Python;
pub use crate::{FromPyObject, IntoPy, IntoPyPointer, PyTryFrom, PyTryInto, ToPyObject};
// PyModule is only part of the prelude because we need it for the pymodule function
pub use crate::types::{PyAny, PyModule};
pub use crate::wrap_pyfunction;
#[cfg(feature = "macros")]
pub use {crate::proc_macro::*, pyo3_macros::FromPyObject};
2 changes: 1 addition & 1 deletion src/python.rs
Original file line number Diff line number Diff line change
Expand Up @@ -189,7 +189,7 @@ impl<'p> Python<'p> {
///
/// # Examples
/// ```
/// # use pyo3::prelude::*; use pyo3::types::IntoPyDict; use pyo3::wrap_pyfunction;
/// # use pyo3::prelude::*; use pyo3::types::IntoPyDict;
/// use pyo3::exceptions::PyRuntimeError;
/// use std::sync::Arc;
/// use std::thread;
Expand Down
1 change: 0 additions & 1 deletion src/types/module.rs
Original file line number Diff line number Diff line change
Expand Up @@ -346,7 +346,6 @@ impl PyModule {
///
/// ```rust
/// use pyo3::prelude::*;
/// use pyo3::wrap_pyfunction;
///
/// #[pyfunction]
/// fn say_hello() {
Expand Down
1 change: 0 additions & 1 deletion tests/test_bytes.rs
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
use pyo3::prelude::*;
use pyo3::types::PyBytes;
use pyo3::wrap_pyfunction;

mod common;

Expand Down
2 changes: 1 addition & 1 deletion tests/test_exceptions.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
use pyo3::prelude::*;
use pyo3::{exceptions, py_run, wrap_pyfunction, PyErr, PyResult};
use pyo3::{exceptions, py_run, PyErr, PyResult};
use std::error::Error;
use std::fmt;
#[cfg(not(target_os = "windows"))]
Expand Down
1 change: 0 additions & 1 deletion tests/test_macros.rs
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
//! Ensure that pyo3 macros can be used inside macro_rules!
use pyo3::prelude::*;
use pyo3::wrap_pyfunction;

#[macro_use]
mod common;
Expand Down
2 changes: 1 addition & 1 deletion tests/test_module.rs
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
use pyo3::prelude::*;

use pyo3::py_run;
use pyo3::types::{IntoPyDict, PyDict, PyTuple};
use pyo3::{py_run, wrap_pyfunction};
mod common;

#[pyclass]
Expand Down
1 change: 0 additions & 1 deletion tests/test_pyfunction.rs
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ use pyo3::prelude::*;
use pyo3::types::PyCFunction;
#[cfg(not(Py_LIMITED_API))]
use pyo3::types::{PyDateTime, PyFunction};
use pyo3::wrap_pyfunction;

mod common;

Expand Down
1 change: 0 additions & 1 deletion tests/test_string.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
use pyo3::prelude::*;
use pyo3::wrap_pyfunction;

mod common;

Expand Down
2 changes: 1 addition & 1 deletion tests/test_text_signature.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
use pyo3::prelude::*;
use pyo3::{types::PyType, wrap_pyfunction, wrap_pymodule, PyCell};
use pyo3::{types::PyType, wrap_pymodule, PyCell};

mod common;

Expand Down
2 changes: 1 addition & 1 deletion tests/test_various.rs
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
use pyo3::prelude::*;
use pyo3::types::{PyDict, PyTuple};
use pyo3::{py_run, wrap_pyfunction, PyCell};
use pyo3::{py_run, PyCell};

use std::fmt;

Expand Down
2 changes: 1 addition & 1 deletion tests/test_wrap_pyfunction_deduction.rs
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
use pyo3::{prelude::*, types::PyCFunction, wrap_pyfunction};
use pyo3::{prelude::*, types::PyCFunction};

#[pyfunction]
fn f() {}
Expand Down
1 change: 0 additions & 1 deletion tests/ui/invalid_result_conversion.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
//! *doesn't* implement `From<MyError> for PyErr` won't be automatically
//! converted when using `#[pyfunction]`.
use pyo3::prelude::*;
use pyo3::wrap_pyfunction;

use std::fmt;

Expand Down
4 changes: 2 additions & 2 deletions tests/ui/invalid_result_conversion.stderr
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
error[E0277]: the trait bound `Result<(), MyError>: IntoPyCallbackOutput<_>` is not satisfied
--> $DIR/invalid_result_conversion.rs:22:1
--> $DIR/invalid_result_conversion.rs:21:1
|
22 | #[pyfunction]
21 | #[pyfunction]
| ^^^^^^^^^^^^^ the trait `IntoPyCallbackOutput<_>` is not implemented for `Result<(), MyError>`
|
::: $WORKSPACE/src/callback.rs
Expand Down