Skip to content

[smart_holder] Special case for _clif_automatic policy in return_value_policy_override - #4364

Merged
rwgk merged 1 commit into
pybind:smart_holderfrom
rwgk:sh_return_value_policy_override_clif_automatic
Nov 28, 2022
Merged

[smart_holder] Special case for _clif_automatic policy in return_value_policy_override#4364
rwgk merged 1 commit into
pybind:smart_holderfrom
rwgk:sh_return_value_policy_override_clif_automatic

Conversation

@rwgk

@rwgkrwgk commented Nov 28, 2022

Copy link
Copy Markdown
Collaborator

Description

Special case for _clif_automatic policy in return_value_policy_override

Immediate use case: pybind11_abseil/bstatus_caster.h

Suggested changelog entry:

@rwgk

rwgk commented Nov 28, 2022

Copy link
Copy Markdown
CollaboratorAuthor

This PR passed Google-internal global testing (internal ID OCL:491242725:BASE:491328120:1669646137617:e06ecc8c).

@wangxf123456 could you please review?

@rwgk
rwgk marked this pull request as ready for review November 28, 2022 14:51
@rwgk
rwgk requested a review from henryiii as a code ownerNovember 28, 2022 14:51
@rwgk
rwgk merged commit 8720cf9 into pybind:smart_holderNov 28, 2022
@rwgk
rwgk deleted the sh_return_value_policy_override_clif_automatic branch November 28, 2022 19:10
@github-actionsgithub-actionsBot added the needs changelog Possibly needs a changelog entry label Nov 28, 2022
@rwgkrwgk removed the needs changelog Possibly needs a changelog entry label Nov 28, 2022
@rwgkrwgk mentioned this pull request Feb 11, 2023
rwgk added a commit to rwgk/pybind11 that referenced this pull request Jun 11, 2024
rwgk added a commit that referenced this pull request Jun 11, 2024
Sign up for freeto 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

@rwgk@wangxf123456