Replace iterator constructors with a template

Author: wravery

include/graphqlservice/GraphQLResponse.h

@@ -68,10 +68,12 @@ struct [[nodiscard]] IdType
 	// Implicit ByteData constructors
 	GRAPHQLRESPONSE_EXPORT IdType(size_t count, typename ByteData::value_type value = 0);
 	GRAPHQLRESPONSE_EXPORT IdType(std::initializer_list<typename ByteData::value_type> values);
-	GRAPHQLRESPONSE_EXPORT IdType(
-		typename ByteData::const_iterator begin, typename ByteData::const_iterator end);
-	GRAPHQLRESPONSE_EXPORT IdType(
-		typename ByteData::const_pointer begin, typename ByteData::const_pointer end);
+
+	template <typename InputIt>
+	IdType(InputIt begin, InputIt end)
+		: _data { ByteData { begin, end } }
+	{
+	}
 
 	// Assignment
 	GRAPHQLRESPONSE_EXPORT IdType& operator=(IdType&& rhs) noexcept;
@@ -80,7 +82,6 @@ struct [[nodiscard]] IdType
 	// Conversion
 	GRAPHQLRESPONSE_EXPORT IdType(ByteData&& data) noexcept;
 	GRAPHQLRESPONSE_EXPORT IdType& operator=(ByteData&& data) noexcept;
-	GRAPHQLRESPONSE_EXPORT IdType& operator=(std::initializer_list<typename ByteData::value_type> values);
 
 	GRAPHQLRESPONSE_EXPORT IdType(OpaqueString&& opaque) noexcept;
 	GRAPHQLRESPONSE_EXPORT IdType& operator=(OpaqueString&& opaque) noexcept;

src/GraphQLResponse.cpp

@@ -41,16 +41,6 @@ IdType::IdType(std::initializer_list<typename ByteData::value_type> values)
 {
 }
 
-IdType::IdType(typename ByteData::const_iterator begin, typename ByteData::const_iterator end)
-	: _data { ByteData { begin, end } }
-{
-}
-
-IdType::IdType(typename ByteData::const_pointer begin, typename ByteData::const_pointer end)
-	: _data { ByteData { begin, end } }
-{
-}
-
 IdType& IdType::operator=(IdType&& rhs) noexcept
 {
 	if (&rhs != this)
@@ -72,12 +62,6 @@ IdType& IdType::operator=(ByteData&& data) noexcept
 	return *this;
 }
 
-IdType& IdType::operator=(std::initializer_list<typename IdType::ByteData::value_type> values)
-{
-	_data = ByteData { values };
-	return *this;
-}
-
 IdType::IdType(OpaqueString&& opaque) noexcept
 	: _data { std::move(opaque) }
 {