Zengtudor/pybind11
1
0
Fork 0
mirror of https://github.com/pybind/pybind11.git synced 2024-11-29 08:32:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Rename holder to shared_ptr_holder to improve readability. No functional changes.

Browse Source
This commit is contained in:
Ralf W. Grosse-Kunstleve 2024-07-03 10:49:17 -07:00
parent 17c0354050
commit 2837df178e
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
include/pybind11/cast.h
View File

@ -857,8 +857,8 @@ public:
// static_cast works around compiler error with MSVC 17 and CUDA 10.2
// see issue #2180
explicit operator type &() { return *(static_cast<type *>(this->value)); }
explicit operator std::shared_ptr<type> *() { return std::addressof(holder); }
explicit operator std::shared_ptr<type> &() { return holder; }
explicit operator std::shared_ptr<type> *() { return std::addressof(shared_ptr_holder); }
explicit operator std::shared_ptr<type> &() { return shared_ptr_holder; }
static handle
cast(const std::shared_ptr<type> &src, return_value_policy policy, handle parent) {
@ -872,7 +872,7 @@ protected:
bool load_value_shared_ptr(value_and_holder &&v_h) {
if (v_h.holder_constructed()) {
value = v_h.value_ptr();
holder = v_h.template holder<std::shared_ptr<type>>();
shared_ptr_holder = v_h.template holder<std::shared_ptr<type>>();
return true;
}
throw cast_error("Unable to cast from non-held to held instance (T& to Holder<T>) "
@ -909,7 +909,8 @@ protected:
copyable_holder_caster sub_caster(*cast.first);
if (sub_caster.load(src, convert)) {
value = cast.second(sub_caster.value);
holder = std::shared_ptr<type>(sub_caster.holder, (type *) value);
shared_ptr_holder
= std::shared_ptr<type>(sub_caster.shared_ptr_holder, (type *) value);
return true;
}
}
@ -918,7 +919,7 @@ protected:
static bool try_direct_conversions(handle) { return false; }
std::shared_ptr<type> holder;
std::shared_ptr<type> shared_ptr_holder;
};
/// Specialize for the common std::shared_ptr, so users don't need to