Ignore warnings in some tests; fix ambiguity in ArrayWrapper test

KhronosGroup · Oct 24, 2023 · bbc8197 · bbc8197
1 parent 9c15a1e
commit bbc8197
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 7 deletions.
diff --git a/tests/ArrayProxy/ArrayProxy.cpp b/tests/ArrayProxy/ArrayProxy.cpp
@@ -22,7 +22,7 @@
 #elif defined( __GNUC__ )
 #  pragma GCC diagnostic ignored "-Wunused-but-set-variable"
 #else
-// unknow compiler... just ignore the warnings for yourselves ;)
+// unknown compiler... just ignore the warnings for yourselves ;)
 #endif
 
 #include <iostream>

diff --git a/tests/ArrayProxyNoTemporaries/ArrayProxyNoTemporaries.cpp b/tests/ArrayProxyNoTemporaries/ArrayProxyNoTemporaries.cpp
@@ -17,12 +17,12 @@
 
 #if defined( _MSC_VER )
 // no need to ignore any warnings with MSVC
+#elif defined( __clang__ )
+#  pragma clang diagnostic ignored "-Wunused-variable"
 #elif defined( __GNUC__ )
-#  if ( 9 <= __GNUC__ )
-#    pragma GCC diagnostic ignored "-Winit-list-lifetime"
-#  endif
+#  pragma GCC diagnostic ignored "-Wunused-but-set-variable"
 #else
-// unknow compiler... just ignore the warnings for yourselves ;)
+// unknown compiler... just ignore the warnings for yourselves ;)
 #endif
 
 #include <iostream>

diff --git a/tests/ArrayWrapper/ArrayWrapper.cpp b/tests/ArrayWrapper/ArrayWrapper.cpp
@@ -33,15 +33,17 @@ int main( int /*argc*/, char ** /*argv*/ )
 
   // s1 = aw1;  // 'operator =' is ambiguous
 
-  vk::ArrayWrapper1D<char, 20> aw2( "foobah" );
+  std::string foobah = "foobah";
+
+  vk::ArrayWrapper1D<char, 20> aw2( foobah );
   f( aw2 );
 
   vk::ArrayWrapper1D<char, 5> aw3( { 'f', 'o', 'o', 'b', 'a', 'h' } );
   std::string                 s3 = aw3;
   assert( s3.length() == 5 );
   std::cout << "<" << s3 << ">" << std::endl;
 
-  vk::ArrayWrapper1D<char, 5> aw4( "foobah" );
+  vk::ArrayWrapper1D<char, 5> aw4( foobah );
   std::string                 s4 = aw4;
   assert( s4.length() == 5 );
 

diff --git a/tests/UniqueHandle/UniqueHandle.cpp b/tests/UniqueHandle/UniqueHandle.cpp
@@ -27,6 +27,19 @@ static std::string EngineName = "Vulkan.hpp";
 template <typename T>
 class MyAllocator : public std::allocator<T>
 {
+public:
+  MyAllocator() = default;
+
+  template <class U>
+  MyAllocator( const MyAllocator<U> & /*unused*/ )
+  {
+  }
+
+  template <class U>
+  struct rebind
+  {
+    using other = MyAllocator<U>;
+  };
 };
 
 vk::UniqueDescriptorSetLayout createDescriptorSetLayoutUnique( vk::Device const &                                                                  device,