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

native_proto_caster.h: Add *_with_smart_holder_support_enabled specializations. #170

Merged
merged 1 commit into from
Aug 2, 2024

Conversation

copybara-service[bot]
Copy link

native_proto_caster.h: Add *_with_smart_holder_support_enabled specializations.

The

  • copyable_holder_caster_shared_ptr_with_smart_holder_support_enabled and

  • move_only_holder_caster_unique_ptr_with_smart_holder_support_enabled

SFINAE helpers were introduced with pybind/pybind11#5257. They need to be specialized here (std::false_type) because native_proto_caster.h has its own specializations for

  • copyable_holder_caster<ProtoType, std::shared_ptr<ProtoType>> and

  • move_only_holder_caster<ProtoType, std::unique_ptr<ProtoType>>.

@copybara-service copybara-service bot force-pushed the cl/658807442 branch 3 times, most recently from ca57662 to 62c190f Compare August 2, 2024 19:08
…ializations.

The

* `copyable_holder_caster_shared_ptr_with_smart_holder_support_enabled` and

* `move_only_holder_caster_unique_ptr_with_smart_holder_support_enabled`

SFINAE helpers were introduced with pybind/pybind11#5257. They need to be specialized here (`std::false_type`) because native_proto_caster.h has its own specializations for

* `copyable_holder_caster<ProtoType, std::shared_ptr<ProtoType>>` and

* `move_only_holder_caster<ProtoType, std::unique_ptr<ProtoType>>`.

PiperOrigin-RevId: 658867114
@copybara-service copybara-service bot merged commit 51fc4ab into main Aug 2, 2024
@copybara-service copybara-service bot deleted the cl/658807442 branch August 2, 2024 19:15
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.

0 participants