From 8d4becef2bede11c7b50d46ceab040886fa56a22 Mon Sep 17 00:00:00 2001 From: Roland Conybeare Date: Sat, 27 Apr 2024 11:55:32 -0400 Subject: [PATCH] xo-ratio: deal: gcc+clang both require pub members... --- include/xo/ratio/ratio.hpp | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/include/xo/ratio/ratio.hpp b/include/xo/ratio/ratio.hpp index 5496e05e..36c8ae30 100644 --- a/include/xo/ratio/ratio.hpp +++ b/include/xo/ratio/ratio.hpp @@ -326,16 +326,7 @@ namespace xo { return lhs <=> rhs; } - /* we want ratio to be a structural type, - * so that we can use an instance as a non-type template parameter. - * - * with private members, clang 18 (we believe incorrectly) complains that ratio is not structural. - */ -#ifdef __clang__ - public: -#else - private: -#endif + public: /* need public members so that a ratio instance can be a non-type template parameter (a structural type) */ /** @defgroup ratio-instance-variables **/ ///@{