Fix compile error in c++20 (#510)

gfphoenix78 · web-flow · commit 1e70fdc79610 · 2024-07-10T10:44:01.000+08:00
In c++20, more constraints are applied:

* constructor and destructor are not allowed to have template id
* adding two enum value needs explicit convertion
diff --git a/src/backend/gporca/libgpopt/include/gpopt/base/CStateMachine.h b/src/backend/gporca/libgpopt/include/gpopt/base/CStateMachine.h
@@ -254,14 +254,10 @@ class CStateMachine
 	}
 
 public:
-	CStateMachine<TEnumState, tenumstateSentinel, TEnumEvent,
-				  tenumeventSentinel>(
-		const CStateMachine<TEnumState, tenumstateSentinel, TEnumEvent,
-							tenumeventSentinel> &) = delete;
+	CStateMachine(const CStateMachine &) = delete;
 
 	// ctor
-	CStateMachine<TEnumState, tenumstateSentinel, TEnumEvent,
-				  tenumeventSentinel>()
+	CStateMachine()
 		: m_tenumstate(TesInitial())
 	{
 		GPOS_ASSERT(0 < tenumstateSentinel && 0 < tenumeventSentinel &&
diff --git a/src/backend/gporca/libgpopt/include/gpopt/search/CJobStateMachine.h b/src/backend/gporca/libgpopt/include/gpopt/search/CJobStateMachine.h
@@ -95,8 +95,7 @@ class CJobStateMachine
 	CJobStateMachine(const CJobStateMachine &) = delete;
 
 	// ctor
-	CJobStateMachine<TEnumState, estSentinel, TEnumEvent, eevSentinel>() =
-		default;
+	CJobStateMachine() = default;
 
 	// dtor
 	~CJobStateMachine() = default;
diff --git a/src/backend/gporca/libgpopt/include/gpopt/xforms/CXform.h b/src/backend/gporca/libgpopt/include/gpopt/xforms/CXform.h
@@ -25,7 +25,7 @@
 #include "naucrates/traceflags/traceflags.h"
 
 // Macro for enabling and disabling xforms
-#define GPOPT_DISABLE_XFORM_TF(x) EopttraceDisableXformBase + x
+#define GPOPT_DISABLE_XFORM_TF(x) EopttraceDisableXformBase + static_cast<int>(x)
 #define GPOPT_ENABLE_XFORM(x) GPOS_UNSET_TRACE(GPOPT_DISABLE_XFORM_TF(x))
 #define GPOPT_DISABLE_XFORM(x) GPOS_SET_TRACE(GPOPT_DISABLE_XFORM_TF(x))
 #define GPOPT_FENABLED_XFORM(x) !GPOS_FTRACE(GPOPT_DISABLE_XFORM_TF(x))
diff --git a/src/backend/gporca/libgpopt/include/gpopt/xforms/CXformApply2Join.h b/src/backend/gporca/libgpopt/include/gpopt/xforms/CXformApply2Join.h
@@ -257,7 +257,7 @@ class CXformApply2Join : public CXformExploration
 	CXformApply2Join(const CXformApply2Join &) = delete;
 
 	// ctor for deep pattern
-	explicit CXformApply2Join<TApply, TJoin>(CMemoryPool *mp, BOOL)
+	explicit CXformApply2Join(CMemoryPool *mp, BOOL)
 		:  // pattern
 		  CXformExploration(GPOS_NEW(mp) CExpression(
 			  mp, GPOS_NEW(mp) TApply(mp),
@@ -272,7 +272,7 @@ class CXformApply2Join : public CXformExploration
 	}
 
 	// ctor for shallow pattern
-	explicit CXformApply2Join<TApply, TJoin>(CMemoryPool *mp)
+	explicit CXformApply2Join(CMemoryPool *mp)
 		:  // pattern
 		  CXformExploration(GPOS_NEW(mp) CExpression(
 			  mp, GPOS_NEW(mp) TApply(mp),
@@ -287,14 +287,13 @@ class CXformApply2Join : public CXformExploration
 	}
 
 	// ctor for passed pattern
-	CXformApply2Join<TApply, TJoin>(CMemoryPool *,	// mp
-									CExpression *pexprPattern)
+	CXformApply2Join(CMemoryPool * /* mp */, CExpression *pexprPattern)
 		: CXformExploration(pexprPattern)
 	{
 	}
 
 	// dtor
-	~CXformApply2Join<TApply, TJoin>() override = default;
+	~CXformApply2Join() override = default;
 
 	// is transformation an Apply decorrelation (Apply To Join) xform?
 	BOOL
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CAutoP.h b/src/backend/gporca/libgpos/include/gpos/common/CAutoP.h
@@ -39,14 +39,14 @@ class CAutoP : public CStackObject
 	T *m_object;
 
 public:
-	CAutoP<T>(const CAutoP &) = delete;
+	CAutoP(const CAutoP &) = delete;
 
 	// ctor
-	explicit CAutoP<T>() : m_object(nullptr)
+	explicit CAutoP() : m_object(nullptr)
 	{
 	}
 
-	explicit CAutoP<T>(T *object) : m_object(object)
+	explicit CAutoP(T *object) : m_object(object)
 	{
 	}
 
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CAutoRef.h b/src/backend/gporca/libgpos/include/gpos/common/CAutoRef.h
@@ -30,15 +30,15 @@ class CAutoRef : public CAutoP<T>
 {
 private:
 public:
-	CAutoRef<T>(const CAutoRef &) = delete;
+	CAutoRef(const CAutoRef &) = delete;
 
 	// ctor
-	explicit CAutoRef<T>() : CAutoP<T>()
+	explicit CAutoRef() : CAutoP<T>()
 	{
 	}
 
 	// ctor
-	explicit CAutoRef<T>(T *object) : CAutoP<T>(object)
+	explicit CAutoRef(T *object) : CAutoP<T>(object)
 	{
 	}
 
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CAutoRg.h b/src/backend/gporca/libgpos/include/gpos/common/CAutoRg.h
@@ -34,15 +34,15 @@ class CAutoRg : public CStackObject
 	T *m_object_array;
 
 public:
-	CAutoRg<T>(const CAutoRg &) = delete;
+	CAutoRg(const CAutoRg &) = delete;
 
 	// ctor
-	explicit CAutoRg<T>() : m_object_array(nullptr)
+	explicit CAutoRg() : m_object_array(nullptr)
 	{
 	}
 
 	// ctor
-	explicit CAutoRg<T>(T *object_array) : m_object_array(object_array)
+	explicit CAutoRg(T *object_array) : m_object_array(object_array)
 	{
 	}
 
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CDynamicPtrArray.h b/src/backend/gporca/libgpos/include/gpos/common/CDynamicPtrArray.h
@@ -145,11 +145,10 @@ class CDynamicPtrArray : public CRefCount
 	}
 
 public:
-	CDynamicPtrArray<T, CleanupFn>(const CDynamicPtrArray<T, CleanupFn> &) =
-		delete;
+	CDynamicPtrArray(const CDynamicPtrArray &) = delete;
 
 	// ctor
-	explicit CDynamicPtrArray<T, CleanupFn>(CMemoryPool *mp, ULONG min_size = 4,
+	explicit CDynamicPtrArray(CMemoryPool *mp, ULONG min_size = 4,
 											ULONG expansion_factor = 10)
 		: m_mp(mp),
 		  m_capacity(0),
@@ -165,7 +164,7 @@ class CDynamicPtrArray : public CRefCount
 	}
 
 	// dtor
-	~CDynamicPtrArray<T, CleanupFn>() override
+	~CDynamicPtrArray() override
 	{
 		Clear();
 
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CEnumSet.h b/src/backend/gporca/libgpos/include/gpos/common/CEnumSet.h
@@ -30,22 +30,22 @@ class CEnumSet : public CBitSet
 {
 private:
 public:
-	CEnumSet<T, sentinel_index>(const CEnumSet<T, sentinel_index> &) = delete;
+	CEnumSet(const CEnumSet &) = delete;
 
 	// ctor
-	explicit CEnumSet<T, sentinel_index>(CMemoryPool *mp)
+	explicit CEnumSet(CMemoryPool *mp)
 		: CBitSet(mp, sentinel_index)
 	{
 	}
 
-	explicit CEnumSet<T, sentinel_index>(
-		CMemoryPool *mp, const CEnumSet<T, sentinel_index> &enum_set)
+	explicit CEnumSet(
+		CMemoryPool *mp, const CEnumSet &enum_set)
 		: CBitSet(mp, enum_set)
 	{
 	}
 
 	// dtor
-	~CEnumSet<T, sentinel_index>() override = default;
+	~CEnumSet() override = default;
 
 	// determine if bit is set
 	BOOL
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CEnumSetIter.h b/src/backend/gporca/libgpos/include/gpos/common/CEnumSetIter.h
@@ -30,18 +30,17 @@ class CEnumSetIter : public CBitSetIter
 {
 private:
 public:
-	CEnumSetIter<T, sentinel_index>(const CEnumSetIter<T, sentinel_index> &) =
-		delete;
+	CEnumSetIter(const CEnumSetIter &) = delete;
 
 	// ctor
-	explicit CEnumSetIter<T, sentinel_index>(
+	explicit CEnumSetIter(
 		const CEnumSet<T, sentinel_index> &enum_set)
 		: CBitSetIter(enum_set)
 	{
 	}
 
 	// dtor
-	~CEnumSetIter<T, sentinel_index>() = default;
+	~CEnumSetIter() = default;
 
 	// current enum
 	T
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CHashMap.h b/src/backend/gporca/libgpos/include/gpos/common/CHashMap.h
@@ -177,12 +177,10 @@ class CHashMap : public CRefCount
 	}
 
 public:
-	CHashMap(const CHashMap<K, T, HashFn, EqFn, DestroyKFn, DestroyTFn> &) =
-		delete;
+	CHashMap(const CHashMap &) = delete;
 
 	// ctor
-	CHashMap<K, T, HashFn, EqFn, DestroyKFn, DestroyTFn>(CMemoryPool *mp,
-														 ULONG num_chains = 127)
+	CHashMap(CMemoryPool *mp, ULONG num_chains = 127)
 		: m_mp(mp),
 		  m_num_chains(num_chains),
 		  m_size(0),
@@ -196,7 +194,7 @@ class CHashMap : public CRefCount
 	}
 
 	// dtor
-	~CHashMap<K, T, HashFn, EqFn, DestroyKFn, DestroyTFn>() override
+	~CHashMap() override
 	{
 		// release all hash chains
 		Clear();
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CHashMapIter.h b/src/backend/gporca/libgpos/include/gpos/common/CHashMapIter.h
@@ -59,19 +59,17 @@ class CHashMapIter : public CStackObject
 	}
 
 public:
-	CHashMapIter(const CHashMapIter<K, T, HashFn, EqFn, DestroyKFn, DestroyTFn>
-					 &) = delete;
+	CHashMapIter(const CHashMapIter &) = delete;
 
 	// ctor
-	CHashMapIter<K, T, HashFn, EqFn, DestroyKFn, DestroyTFn>(TMap *ptm)
+	CHashMapIter(TMap *ptm)
 		: m_map(ptm), m_chain_idx(0), m_key_idx(0)
 	{
 		GPOS_ASSERT(nullptr != ptm);
 	}
 
 	// dtor
-	virtual ~CHashMapIter<K, T, HashFn, EqFn, DestroyKFn, DestroyTFn>() =
-		default;
+	virtual ~CHashMapIter() = default;
 
 	// advance iterator to next element
 	BOOL
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CHashSet.h b/src/backend/gporca/libgpos/include/gpos/common/CHashSet.h
@@ -160,10 +160,10 @@ class CHashSet : public CRefCount
 	}
 
 public:
-	CHashSet(const CHashSet<T, HashFn, EqFn, CleanupFn> &) = delete;
+	CHashSet(const CHashSet &) = delete;
 
 	// ctor
-	CHashSet<T, HashFn, EqFn, CleanupFn>(CMemoryPool *mp, ULONG size = 127)
+	CHashSet(CMemoryPool *mp, ULONG size = 127)
 		: m_mp(mp),
 		  m_num_chains(size),
 		  m_size(0),
@@ -177,7 +177,7 @@ class CHashSet : public CRefCount
 	}
 
 	// dtor
-	~CHashSet<T, HashFn, EqFn, CleanupFn>() override
+	~CHashSet() override
 	{
 		// release all hash chains
 		Clear();
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CHashSetIter.h b/src/backend/gporca/libgpos/include/gpos/common/CHashSetIter.h
@@ -35,17 +35,17 @@ class CHashSetIter : public CStackObject
 	BOOL m_is_initialized;
 
 public:
-	CHashSetIter(const CHashSetIter<T, HashFn, EqFn, CleanupFn> &) = delete;
+	CHashSetIter(const CHashSetIter &) = delete;
 
 	// ctor
-	CHashSetIter<T, HashFn, EqFn, CleanupFn>(TSet *set)
+	CHashSetIter(TSet *set)
 		: m_set(set), m_chain_idx(0), m_elem_idx(0)
 	{
 		GPOS_ASSERT(nullptr != set);
 	}
 
 	// dtor
-	virtual ~CHashSetIter<T, HashFn, EqFn, CleanupFn>() = default;
+	virtual ~CHashSetIter() = default;
 
 	// advance iterator to next element
 	BOOL
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CStack.h b/src/backend/gporca/libgpos/include/gpos/common/CStack.h
@@ -38,7 +38,7 @@ class CStack
 	CStack(CStack &) = delete;
 
 	// c'tor
-	CStack<T>(CMemoryPool *mp, ULONG min_size = 4) : m_size(0)
+	CStack(CMemoryPool *mp, ULONG min_size = 4) : m_size(0)
 	{
 		m_dynamic_ptr_array =
 			GPOS_NEW(mp) CDynamicPtrArray<T, CleanupNULL>(mp, min_size, 10);
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CSyncHashtable.h b/src/backend/gporca/libgpos/include/gpos/common/CSyncHashtable.h
@@ -144,7 +144,7 @@ class CSyncHashtable
 	using DestroyEntryFuncPtr = void (*)(T *);
 
 	// ctor
-	CSyncHashtable<T, K>()
+	CSyncHashtable()
 		: m_buckets(nullptr),
 
 		  m_invalid_key(nullptr)
@@ -154,7 +154,7 @@ class CSyncHashtable
 	// dtor
 	// deallocates hashtable internals, does not destroy
 	// client objects
-	~CSyncHashtable<T, K>()
+	~CSyncHashtable()
 	{
 		Cleanup();
 	}
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CSyncHashtableAccessByIter.h b/src/backend/gporca/libgpos/include/gpos/common/CSyncHashtableAccessByIter.h
@@ -58,11 +58,10 @@ class CSyncHashtableAccessByIter : public CSyncHashtableAccessorBase<T, K>
 	}
 
 public:
-	CSyncHashtableAccessByIter<T, K>(const CSyncHashtableAccessByIter<T, K> &) =
-		delete;
+	CSyncHashtableAccessByIter(const CSyncHashtableAccessByIter &) = delete;
 
 	// ctor
-	explicit CSyncHashtableAccessByIter<T, K>(CSyncHashtableIter<T, K> &iter)
+	explicit CSyncHashtableAccessByIter(CSyncHashtableIter<T, K> &iter)
 		: Base(iter.m_ht, iter.m_bucket_idx), m_iter(iter)
 	{
 	}
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CSyncHashtableAccessByKey.h b/src/backend/gporca/libgpos/include/gpos/common/CSyncHashtableAccessByKey.h
@@ -68,11 +68,10 @@ class CSyncHashtableAccessByKey : public CSyncHashtableAccessorBase<T, K>
 #endif	// GPOS_DEBUG
 
 public:
-	CSyncHashtableAccessByKey<T, K>(const CSyncHashtableAccessByKey<T, K> &) =
-		delete;
+	CSyncHashtableAccessByKey(const CSyncHashtableAccessByKey &) = delete;
 
 	// ctor
-	CSyncHashtableAccessByKey<T, K>(CSyncHashtable<T, K> &ht, const K &key)
+	CSyncHashtableAccessByKey(CSyncHashtable<T, K> &ht, const K &key)
 		: Base(ht, ht.GetBucketIndex(key)), m_key(key)
 	{
 	}
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CSyncHashtableAccessorBase.h b/src/backend/gporca/libgpos/include/gpos/common/CSyncHashtableAccessorBase.h
@@ -46,13 +46,13 @@ class CSyncHashtableAccessorBase : public CStackObject
 
 protected:
 	// ctor - protected to restrict instantiation to children
-	CSyncHashtableAccessorBase<T, K>(CSyncHashtable<T, K> &ht, ULONG bucket_idx)
+	CSyncHashtableAccessorBase(CSyncHashtable<T, K> &ht, ULONG bucket_idx)
 		: m_ht(ht), m_bucket(m_ht.GetBucket(bucket_idx))
 	{
 	}
 
 	// dtor
-	virtual ~CSyncHashtableAccessorBase<T, K>() = default;
+	virtual ~CSyncHashtableAccessorBase() = default;
 
 	// accessor to hashtable
 	CSyncHashtable<T, K> &
@@ -130,8 +130,7 @@ class CSyncHashtableAccessorBase : public CStackObject
 	}
 
 public:
-	CSyncHashtableAccessorBase<T, K>(const CSyncHashtableAccessorBase<T, K> &) =
-		delete;
+	CSyncHashtableAccessorBase(const CSyncHashtableAccessorBase &) = delete;
 
 	// unlinks element
 	void
diff --git a/src/backend/gporca/libgpos/include/gpos/common/CSyncHashtableIter.h b/src/backend/gporca/libgpos/include/gpos/common/CSyncHashtableIter.h
@@ -113,10 +113,10 @@ class CSyncHashtableIter
 	BOOL m_invalid_elem_inserted;
 
 public:
-	CSyncHashtableIter<T, K>(const CSyncHashtableIter<T, K> &) = delete;
+	CSyncHashtableIter(const CSyncHashtableIter &) = delete;
 
 	// ctor
-	explicit CSyncHashtableIter<T, K>(CSyncHashtable<T, K> &ht)
+	explicit CSyncHashtableIter(CSyncHashtable<T, K> &ht)
 		: m_ht(ht),
 		  m_bucket_idx(0),
 		  m_invalid_elem(nullptr),
@@ -133,7 +133,7 @@ class CSyncHashtableIter
 	}
 
 	// dtor
-	~CSyncHashtableIter<T, K>()
+	~CSyncHashtableIter()
 	{
 		if (m_invalid_elem_inserted)
 		{
diff --git a/src/backend/gporca/server/src/unittest/gpopt/minidump/CICGTest.cpp b/src/backend/gporca/server/src/unittest/gpopt/minidump/CICGTest.cpp

Original file line number	Diff line number	Diff line change
`@@ -39,14 +39,14 @@ class CAutoP : public CStackObject`
`39`	`39`	`T *m_object;`
`40`	`40`
`41`	`41`	`public:`
`42`		`- CAutoP<T>(const CAutoP &) = delete;`
	`42`	`+ CAutoP(const CAutoP &) = delete;`
`43`	`43`
`44`	`44`	`// ctor`
`45`		`- explicit CAutoP<T>() : m_object(nullptr)`
	`45`	`+ explicit CAutoP() : m_object(nullptr)`
`46`	`46`	`{`
`47`	`47`	`}`
`48`	`48`
`49`		`- explicit CAutoP<T>(T *object) : m_object(object)`
	`49`	`+ explicit CAutoP(T *object) : m_object(object)`
`50`	`50`	`{`
`51`	`51`	`}`
`52`	`52`
Original file line number	Diff line number	Diff line change
`@@ -30,15 +30,15 @@ class CAutoRef : public CAutoP<T>`
`30`	`30`	`{`
`31`	`31`	`private:`
`32`	`32`	`public:`
`33`		`- CAutoRef<T>(const CAutoRef &) = delete;`
	`33`	`+ CAutoRef(const CAutoRef &) = delete;`
`34`	`34`
`35`	`35`	`// ctor`
`36`		`- explicit CAutoRef<T>() : CAutoP<T>()`
	`36`	`+ explicit CAutoRef() : CAutoP<T>()`
`37`	`37`	`{`
`38`	`38`	`}`
`39`	`39`
`40`	`40`	`// ctor`
`41`		`- explicit CAutoRef<T>(T *object) : CAutoP<T>(object)`
	`41`	`+ explicit CAutoRef(T *object) : CAutoP<T>(object)`
`42`	`42`	`{`
`43`	`43`	`}`
`44`	`44`
Original file line number	Diff line number	Diff line change
`@@ -34,15 +34,15 @@ class CAutoRg : public CStackObject`
`34`	`34`	`T *m_object_array;`
`35`	`35`
`36`	`36`	`public:`
`37`		`- CAutoRg<T>(const CAutoRg &) = delete;`
	`37`	`+ CAutoRg(const CAutoRg &) = delete;`
`38`	`38`
`39`	`39`	`// ctor`
`40`		`- explicit CAutoRg<T>() : m_object_array(nullptr)`
	`40`	`+ explicit CAutoRg() : m_object_array(nullptr)`
`41`	`41`	`{`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`// ctor`
`45`		`- explicit CAutoRg<T>(T *object_array) : m_object_array(object_array)`
	`45`	`+ explicit CAutoRg(T *object_array) : m_object_array(object_array)`
`46`	`46`	`{`
`47`	`47`	`}`
`48`	`48`
Original file line number	Diff line number	Diff line change
`@@ -145,11 +145,10 @@ class CDynamicPtrArray : public CRefCount`
`145`	`145`	`}`
`146`	`146`
`147`	`147`	`public:`
`148`		`- CDynamicPtrArray<T, CleanupFn>(const CDynamicPtrArray<T, CleanupFn> &) =`
`149`		`- delete;`
	`148`	`+ CDynamicPtrArray(const CDynamicPtrArray &) = delete;`
`150`	`149`
`151`	`150`	`// ctor`
`152`		`- explicit CDynamicPtrArray<T, CleanupFn>(CMemoryPool *mp, ULONG min_size = 4,`
	`151`	`+ explicit CDynamicPtrArray(CMemoryPool *mp, ULONG min_size = 4,`
`153`	`152`	`ULONG expansion_factor = 10)`
`154`	`153`	`: m_mp(mp),`
`155`	`154`	`m_capacity(0),`
`@@ -165,7 +164,7 @@ class CDynamicPtrArray : public CRefCount`
`165`	`164`	`}`
`166`	`165`
`167`	`166`	`// dtor`
`168`		`- ~CDynamicPtrArray<T, CleanupFn>() override`
	`167`	`+ ~CDynamicPtrArray() override`
`169`	`168`	`{`
`170`	`169`	`Clear();`
`171`	`170`
Original file line number	Diff line number	Diff line change
`@@ -30,22 +30,22 @@ class CEnumSet : public CBitSet`
`30`	`30`	`{`
`31`	`31`	`private:`
`32`	`32`	`public:`
`33`		`- CEnumSet<T, sentinel_index>(const CEnumSet<T, sentinel_index> &) = delete;`
	`33`	`+ CEnumSet(const CEnumSet &) = delete;`
`34`	`34`
`35`	`35`	`// ctor`
`36`		`- explicit CEnumSet<T, sentinel_index>(CMemoryPool *mp)`
	`36`	`+ explicit CEnumSet(CMemoryPool *mp)`
`37`	`37`	`: CBitSet(mp, sentinel_index)`
`38`	`38`	`{`
`39`	`39`	`}`
`40`	`40`
`41`		`- explicit CEnumSet<T, sentinel_index>(`
`42`		`- CMemoryPool *mp, const CEnumSet<T, sentinel_index> &enum_set)`
	`41`	`+ explicit CEnumSet(`
	`42`	`+ CMemoryPool *mp, const CEnumSet &enum_set)`
`43`	`43`	`: CBitSet(mp, enum_set)`
`44`	`44`	`{`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`// dtor`
`48`		`- ~CEnumSet<T, sentinel_index>() override = default;`
	`48`	`+ ~CEnumSet() override = default;`
`49`	`49`
`50`	`50`	`// determine if bit is set`
`51`	`51`	`BOOL`
Original file line number	Diff line number	Diff line change
`@@ -30,18 +30,17 @@ class CEnumSetIter : public CBitSetIter`
`30`	`30`	`{`
`31`	`31`	`private:`
`32`	`32`	`public:`
`33`		`- CEnumSetIter<T, sentinel_index>(const CEnumSetIter<T, sentinel_index> &) =`
`34`		`- delete;`
	`33`	`+ CEnumSetIter(const CEnumSetIter &) = delete;`
`35`	`34`
`36`	`35`	`// ctor`
`37`		`- explicit CEnumSetIter<T, sentinel_index>(`
	`36`	`+ explicit CEnumSetIter(`
`38`	`37`	`const CEnumSet<T, sentinel_index> &enum_set)`
`39`	`38`	`: CBitSetIter(enum_set)`
`40`	`39`	`{`
`41`	`40`	`}`
`42`	`41`
`43`	`42`	`// dtor`
`44`		`- ~CEnumSetIter<T, sentinel_index>() = default;`
	`43`	`+ ~CEnumSetIter() = default;`
`45`	`44`
`46`	`45`	`// current enum`
`47`	`46`	`T`