Skip to content

Allow #[pymethods] to use lifetime#461

Merged
kngwyu merged 2 commits intoPyO3:masterfrom
kngwyu:pymethod-with-lifetime
Apr 29, 2019
Merged

Allow #[pymethods] to use lifetime#461
kngwyu merged 2 commits intoPyO3:masterfrom
kngwyu:pymethod-with-lifetime

Conversation

@kngwyu
Copy link
Member

@kngwyu kngwyu commented Apr 24, 2019

E.g., allow

fn method<'py>(py: Python<'py>) -> &'py PyList {...}

@kngwyu kngwyu force-pushed the pymethod-with-lifetime branch from 999177b to cf68984 Compare April 26, 2019 05:28
@codecov
Copy link

codecov bot commented Apr 26, 2019

Codecov Report

Merging #461 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #461      +/-   ##
==========================================
- Coverage   87.87%   87.83%   -0.04%     
==========================================
  Files          65       64       -1     
  Lines        3398     3388      -10     
==========================================
- Hits         2986     2976      -10     
  Misses        412      412
Impacted Files Coverage Δ
src/marshal.rs

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5bf17f...5a63bdd. Read the comment docs.

Copy link
Member

@konstin konstin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks you!

@kngwyu kngwyu merged commit be10800 into PyO3:master Apr 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants