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

[smart_holder] Remove added return_value_policy members and try_as_void_ptr_capsule feature. #5160

Merged
merged 10 commits into from
Jun 11, 2024

Conversation

rwgk
Copy link
Collaborator

@rwgk rwgk commented Jun 11, 2024

Description

This PR is to achieve a cleaner separation of changes:

All code removed here will continue to exist in the pybind11clif repo.

The type_caster_odr_guard is not removed here because it can be very helpful for determining where PYBIND11_SMART_HOLDER_TYPE_CASTERS macros are missing.

Suggested changelog entry:

@rwgk rwgk changed the title [smart_holder] WIP — Starting by reviewing diff to master [smart_holder] Remove added return_value_policy members and try_as_void_ptr_capsule feature. Jun 11, 2024
@rwgk rwgk changed the base branch from master to smart_holder June 11, 2024 19:17
@rwgk rwgk marked this pull request as ready for review June 11, 2024 21:11
@rwgk rwgk requested a review from henryiii as a code owner June 11, 2024 21:11
@rwgk rwgk merged commit bf79caa into pybind:smart_holder Jun 11, 2024
150 checks passed
@github-actions github-actions bot added the needs changelog Possibly needs a changelog entry label Jun 11, 2024
@rwgk rwgk removed the needs changelog Possibly needs a changelog entry label Jun 11, 2024
@rwgk rwgk deleted the sh_pure branch June 11, 2024 21:12
rwgk pushed a commit to rwgk/pybind11clif that referenced this pull request Jun 11, 2024
rwgk pushed a commit to google/pybind11clif that referenced this pull request Jun 11, 2024
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.

1 participant