Merge pull request #32 from ReflectCxx/release-2.0.CleanSfinae

replaced SFINAE with if constexpr(), cleaner approach

Author: neeraj31285

ReflectionTemplateLib/builder/inc/Builder.h

@@ -118,17 +118,7 @@ namespace rtl {
 				const std::string& pFunction);
 
 			template<class _recordType, class ..._signature>
-			constexpr const access::Function
-			build(enable_if_same<_recordType&, typename detail::TypeId<_signature...>::HEAD > *_= nullptr) const;
-
-			template<class _recordType, class ..._signature>
-			constexpr const access::Function
-			build(enable_if_same<const _recordType&, typename detail::TypeId<_signature...>::HEAD > *_= nullptr) const;
-
-			template<class _recordType, class ..._signature>
-			constexpr const access::Function
-			build(enable_if_not_same<_recordType&, typename detail::TypeId<_signature...>::HEAD > *_= nullptr,
-			      enable_if_not_same<const _recordType&, typename detail::TypeId<_signature...>::HEAD > *__= nullptr) const;
+			constexpr const access::Function build() const;
 
 			template<class _recordType, class _returnType, class ..._signature>
 			constexpr const access::Function build(_returnType(_recordType::* pFunctor)(_signature...)) const;

ReflectionTemplateLib/builder/inc/Builder.hpp

@@ -113,27 +113,19 @@ namespace rtl {
 
 
 		template<class _recordType, class ..._signature>
-		inline constexpr const access::Function
-		Builder<TypeQ::Mute>::build(enable_if_same<_recordType&, typename detail::TypeId<_signature...>::HEAD > *_) const
+		inline constexpr const access::Function Builder<TypeQ::Mute>::build() const
 		{
-			return buildCopyConstructor<_recordType, _signature...>();
-		}
-
-
-		template<class _recordType, class ..._signature>
-		inline constexpr const access::Function
-		Builder<TypeQ::Mute>::build(enable_if_same<const _recordType&, typename detail::TypeId<_signature...>::HEAD > *_) const
-		{
-			return buildConstCopyConstructor<_recordType, _signature...>();
-		}
-
-
-		template<class _recordType, class ..._signature>
-		inline constexpr const access::Function
-		Builder<TypeQ::Mute>::build(enable_if_not_same<_recordType&, typename detail::TypeId<_signature...>::HEAD > *_,
-					    enable_if_not_same<const _recordType&, typename detail::TypeId<_signature...>::HEAD > *__) const
-		{
-			return buildConstructor<_recordType, _signature...>();
+			if constexpr (std::is_same_v<_recordType&, typename detail::TypeId<_signature...>::HEAD>) 
+			{
+				return buildCopyConstructor<_recordType, _signature...>();
+			}
+			else if constexpr (std::is_same_v<const _recordType&, typename detail::TypeId<_signature...>::HEAD>) 
+			{
+				return buildConstCopyConstructor<_recordType, _signature...>();
+			}
+			else {
+				return buildConstructor<_recordType, _signature...>();
+			}
 		}