Fix tests for windows

rHermes · Jun 8, 2024 · ba14eb5 · ba14eb5
1 parent dc5d1f8
commit ba14eb5
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/.github/workflows/cmake-multi-platform.yml b/.github/workflows/cmake-multi-platform.yml
@@ -25,20 +25,20 @@ jobs:
       matrix:
         os: [ubuntu-latest, windows-latest]
         build_type: [Release, Debug, RelWithDebInfo]
-        cpp_compiler: [g++-13, clang++-15, cl]
+        cpp_compiler: [g++, clang++-15, cl]
         include:
           - os: windows-latest
             c_compiler: cl
             cpp_compiler: cl
           - os: ubuntu-latest
-            c_compiler: gcc-13
-            cpp_compiler: g++-13
+            c_compiler: gcc
+            cpp_compiler: g++
           - os: ubuntu-latest
             c_compiler: clang-15
             cpp_compiler: clang++-15
         exclude:
           - os: windows-latest
-            cpp_compiler: g++-13
+            cpp_compiler: g++
           - os: windows-latest
             cpp_compiler: clang++-15
           - os: ubuntu-latest

diff --git a/tests/fast_pimpl_test.hpp b/tests/fast_pimpl_test.hpp
@@ -11,8 +11,13 @@ struct FastPimplTestImpl;
 class FastPimplTest
 {
 private:
+#ifdef _WIN32
   hage::ForwardDeclaredStorage<details::FastPimplTestImpl, 40, 8> m_impl;
-
+#elif defined(__APPLE__)
+  hage::ForwardDeclaredStorage<details::FastPimplTestImpl, 32, 8> m_impl;
+#else
+  hage::ForwardDeclaredStorage<details::FastPimplTestImpl, 40, 8> m_impl;
+#endif
 public:
   FastPimplTest();
   ~FastPimplTest();