Skip to content

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.

1 participant