Merge pull request #197 from Sebanisu/main

-Werror=sign-conversion on GCC 10.2 and C4702: unreachable code on MSVC 16.9.6

Author: hanickadot

.gitignore

@@ -17,4 +17,6 @@ mtent12.txt
 tests/benchmark-range/*
 !tests/benchmark-range/*.cpp
 !tests/benchmark-range/*.hpp
-build
+build
+cmake-build-*/*
+.idea/*

include/ctre/first.hpp

@@ -398,10 +398,10 @@ template <size_t Capacity> class point_set {
 		auto first = begin();
 		auto last = end();
 		auto it = first;
-		size_t count = std::distance(first, last);
-		while (count > 0) {
+		auto count = std::distance(first, last);
+                while (count != 0) {
 			it = first;
-			size_t step = count / 2;
+			auto step = count / 2;
 			std::advance(it, step);
 			if (*it < obj) {
 				first = ++it;

include/ctre/return_type.hpp

@@ -97,8 +97,9 @@ template <size_t Id, typename Name = void> struct captured_content {
 			} else {
 				return static_cast<size_t>(std::distance(begin(), end()));
 			}
-			#endif
+			#else
 			return static_cast<size_t>(std::distance(begin(), end()));
+			#endif
 		}
 
 #if __has_include(<charconv>)
@@ -126,8 +127,9 @@ template <size_t Id, typename Name = void> struct captured_content {
 			} else {
 				return std::basic_string<char_type>(begin(), end());
 			}
-			#endif
+			#else
 			return std::basic_string<char_type>(begin(), end());
+			#endif
 		}
 
 		constexpr CTRE_FORCE_INLINE auto view() const noexcept {

single-header/ctre-unicode.hpp

@@ -3004,8 +3004,9 @@ template <size_t Id, typename Name = void> struct captured_content {
 			} else {
 				return static_cast<size_t>(std::distance(begin(), end()));
 			}
-			#endif
+			#else
 			return static_cast<size_t>(std::distance(begin(), end()));
+			#endif
 		}
 
 #if __has_include(<charconv>)
@@ -3033,8 +3034,9 @@ template <size_t Id, typename Name = void> struct captured_content {
 			} else {
 				return std::basic_string<char_type>(begin(), end());
 			}
-			#endif
+			#else
 			return std::basic_string<char_type>(begin(), end());
+			#endif
 		}
 
 		constexpr CTRE_FORCE_INLINE auto view() const noexcept {
@@ -3834,10 +3836,10 @@ template <size_t Capacity> class point_set {
 		auto first = begin();
 		auto last = end();
 		auto it = first;
-		size_t count = std::distance(first, last);
-		while (count > 0) {
+		auto count = std::distance(first, last);
+                while (count != 0) {
 			it = first;
-			size_t step = count / 2;
+			auto step = count / 2;
 			std::advance(it, step);
 			if (*it < obj) {
 				first = ++it;

single-header/ctre.hpp

@@ -3001,8 +3001,9 @@ template <size_t Id, typename Name = void> struct captured_content {
 			} else {
 				return static_cast<size_t>(std::distance(begin(), end()));
 			}
-			#endif
+			#else
 			return static_cast<size_t>(std::distance(begin(), end()));
+			#endif
 		}
 
 #if __has_include(<charconv>)
@@ -3030,8 +3031,9 @@ template <size_t Id, typename Name = void> struct captured_content {
 			} else {
 				return std::basic_string<char_type>(begin(), end());
 			}
-			#endif
+			#else
 			return std::basic_string<char_type>(begin(), end());
+			#endif
 		}
 
 		constexpr CTRE_FORCE_INLINE auto view() const noexcept {
@@ -3831,10 +3833,10 @@ template <size_t Capacity> class point_set {
 		auto first = begin();
 		auto last = end();
 		auto it = first;
-		size_t count = std::distance(first, last);
-		while (count > 0) {
+		auto count = std::distance(first, last);
+                while (count != 0) {
 			it = first;
-			size_t step = count / 2;
+			auto step = count / 2;
 			std::advance(it, step);
 			if (*it < obj) {
 				first = ++it;