One way to deal with the order dependency issue. This is not the best way, more like a proof of concept.

This commit is contained in:
Ralf W. Grosse-Kunstleve 2023-07-27 21:27:44 -07:00
parent 7f8b2081ff
commit eb09c6c1b9

View File

@ -713,15 +713,30 @@ public:
// if we can find an exact match (or, for a simple C++ type, an inherited match); if
// so, we can safely reinterpret_cast to the relevant pointer.
if (bases.size() > 1) {
type_info *best_base = nullptr;
if (no_cpp_mi) {
for (auto *base : bases) {
if (no_cpp_mi ? PyType_IsSubtype(base->type, typeinfo->type)
: base->type == typeinfo->type) {
this_.load_value(
reinterpret_cast<instance *>(src.ptr())->get_value_and_holder(base));
return true;
if (PyType_IsSubtype(base->type, typeinfo->type)) {
if (best_base == nullptr
|| PyType_IsSubtype(base->type, best_base->type)) {
best_base = base;
}
}
}
} else {
for (auto *base : bases) {
if (base->type == typeinfo->type) {
best_base = base;
break;
}
}
}
if (best_base != nullptr) {
this_.load_value(
reinterpret_cast<instance *>(src.ptr())->get_value_and_holder(best_base));
return true;
}
}
// Case 2c: C++ multiple inheritance is involved and we couldn't find an exact type
// match in the registered bases, above, so try implicit casting (needed for proper C++