[Bf-blender-cvs] [f93ba581206] temp-cxx-upgrade: Fix BLI_vector_test.cc on Linux platform

Sergey Sharybin noreply at git.blender.org
Thu Jun 18 15:28:23 CEST 2020


Commit: f93ba581206b142769d28ff607688802f3f5bf34
Author: Sergey Sharybin
Date:   Thu Jun 18 15:27:46 2020 +0200
Branches: temp-cxx-upgrade
https://developer.blender.org/rBf93ba581206b142769d28ff607688802f3f5bf34

Fix BLI_vector_test.cc on Linux platform

Quite annoying that GCC has different ideas of implicit sign
conversion than MSVC.

===================================================================

M	tests/gtests/blenlib/BLI_vector_test.cc

===================================================================

diff --git a/tests/gtests/blenlib/BLI_vector_test.cc b/tests/gtests/blenlib/BLI_vector_test.cc
index 7581419e629..1848f841acf 100644
--- a/tests/gtests/blenlib/BLI_vector_test.cc
+++ b/tests/gtests/blenlib/BLI_vector_test.cc
@@ -305,8 +305,8 @@ TEST(vector, BecomeLarge)
     vec.append(i * 5);
   }
   EXPECT_EQ(vec.size(), 100u);
-  for (int i = 0; i < 100; i++) {
-    EXPECT_EQ(vec[i], i * 5);
+  for (uint i = 0; i < 100; i++) {
+    EXPECT_EQ(vec[i], static_cast<uint>(i * 5));
   }
 }



More information about the Bf-blender-cvs mailing list