Use template specialization solution

This commit is contained in:
Christian Semmler 2025-05-18 11:08:15 -07:00
parent c695834d66
commit 254ae2d3a8
3 changed files with 19 additions and 12 deletions

View File

@ -8,7 +8,7 @@ include(CheckCXXSourceCompiles)
include(CMakeDependentOption)
include(CMakePushCheckState)
set(CMAKE_CXX_STANDARD 17)
set(CMAKE_CXX_STANDARD 11)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
set(CMAKE_CXX_EXTENSIONS OFF)

View File

@ -15,6 +15,23 @@ class LegoROI;
// VTABLE: LEGO1 0x100d9248
// class MxPtrList<LegoROI *>
// Specialize MxPtrList for LegoROI* in order to fix Destroy.
// TODO: Change to using constexpr in MxPtrList<T>::Destroy once C++17 standard works.
template <>
class MxPtrList<LegoROI*> : public MxList<LegoROI**> {
public:
MxPtrList(MxBool p_ownership) { SetOwnership(p_ownership); }
static void Destroy(LegoROI** p_obj) { delete[] p_obj; }
void SetOwnership(MxBool p_ownership)
{
MxCollection<LegoROI**>::SetDestroy(
p_ownership ? MxPtrList<LegoROI*>::Destroy : MxCollection<LegoROI**>::Destroy
);
}
};
// VTABLE: LEGO1 0x100d9260
// SIZE 0x18
class LegoROIMapList : public MxPtrList<LegoROI*> {

View File

@ -5,8 +5,6 @@
#include "mxcore.h"
#include "mxtypes.h"
#include <type_traits>
template <class T>
class MxList;
template <class T>
@ -73,15 +71,7 @@ class MxPtrList : public MxList<T*> {
public:
MxPtrList(MxBool p_ownership) { SetOwnership(p_ownership); }
static void Destroy(T* p_obj)
{
if constexpr (std::is_pointer<T>::value) {
delete[] p_obj;
}
else {
delete p_obj;
}
}
static void Destroy(T* p_obj) { delete p_obj; }
void SetOwnership(MxBool p_ownership)
{