Replace std::integral concept with requires clause because Apple Clang doesn't support it yet (#963)

bluetarpmedia · web-flow · commit f146ef7c7f05 · 2024-02-05T09:58:51.000-08:00
Apple Clang doesn't support the `std::integral` concept yet.

Signed-off-by: Neil Henderson &lt;2060747+bluetarpmedia@users.noreply.github.com&gt;
diff --git a/source/common.h b/source/common.h
@@ -533,7 +533,9 @@ auto contains(
 
 
 //  Print an integer with 1,000's separators (always commas, not locale-driven)
-auto print_with_thousands(std::integral auto val)
+template <typename T>
+    requires std::is_integral_v<T>  // Note: `std::integral` concept not yet available in Apple Clang
+auto print_with_thousands(T val)
     -> std::string
 {
     auto ret = std::to_string(val % 10);

Original file line number	Diff line number	Diff line change
`@@ -533,7 +533,9 @@ auto contains(`
`533`	`533`
`534`	`534`
`535`	`535`	`// Print an integer with 1,000's separators (always commas, not locale-driven)`
`536`		`-auto print_with_thousands(std::integral auto val)`
	`536`	`+template <typename T>`
	`537`	+ requires std::is_integral_v<T> // Note: `std::integral` concept not yet available in Apple Clang
	`538`	`+auto print_with_thousands(T val)`
`537`	`539`	`-> std::string`
`538`	`540`	`{`
`539`	`541`	`auto ret = std::to_string(val % 10);`