From b898075f307246743676a16f2204ecde35c65146 Mon Sep 17 00:00:00 2001 From: James Hawkins Date: Tue, 1 Mar 2016 11:21:53 -0800 Subject: system/core/lib[c]utils: Fix signedness comparison warnings. * Store the output of a length variable in size_t. * Annotate unsigned constant values as such. Bug: 27384813 Change-Id: I8504c0a8f5840d4d42e5c0df797a4e5d02d13eb9 --- libcutils/tests/PropertiesTest.cpp | 8 ++++---- libutils/tests/Vector_test.cpp | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/libcutils/tests/PropertiesTest.cpp b/libcutils/tests/PropertiesTest.cpp index 22ca7069d..5f2396b0b 100644 --- a/libcutils/tests/PropertiesTest.cpp +++ b/libcutils/tests/PropertiesTest.cpp @@ -106,14 +106,14 @@ TEST_F(PropertiesTest, SetString) { ResetValue(); // Since the value is null, default value will be returned - int len = property_get(PROPERTY_TEST_KEY, mValue, PROPERTY_TEST_VALUE_DEFAULT); + size_t len = property_get(PROPERTY_TEST_KEY, mValue, PROPERTY_TEST_VALUE_DEFAULT); EXPECT_EQ(strlen(PROPERTY_TEST_VALUE_DEFAULT), len); EXPECT_STREQ(PROPERTY_TEST_VALUE_DEFAULT, mValue); } // Trivial case => get returns what was set { - int len = SetAndGetProperty("hello_world"); + size_t len = SetAndGetProperty("hello_world"); EXPECT_EQ(strlen("hello_world"), len) << "hello_world key"; EXPECT_STREQ("hello_world", mValue); ResetValue(); @@ -122,7 +122,7 @@ TEST_F(PropertiesTest, SetString) { // Set to empty string => get returns default always { const char* EMPTY_STRING_DEFAULT = "EMPTY_STRING"; - int len = SetAndGetProperty("", EMPTY_STRING_DEFAULT); + size_t len = SetAndGetProperty("", EMPTY_STRING_DEFAULT); EXPECT_EQ(strlen(EMPTY_STRING_DEFAULT), len) << "empty key"; EXPECT_STREQ(EMPTY_STRING_DEFAULT, mValue); ResetValue(); @@ -147,7 +147,7 @@ TEST_F(PropertiesTest, SetString) { // Expect that the value set fails since it's too long EXPECT_GT(0, property_set(PROPERTY_TEST_KEY, oneLongerString.c_str())); - int len = property_get(PROPERTY_TEST_KEY, mValue, PROPERTY_TEST_VALUE_DEFAULT); + size_t len = property_get(PROPERTY_TEST_KEY, mValue, PROPERTY_TEST_VALUE_DEFAULT); EXPECT_EQ(strlen(VALID_TEST_VALUE), len) << "set should've failed"; EXPECT_STREQ(VALID_TEST_VALUE, mValue); diff --git a/libutils/tests/Vector_test.cpp b/libutils/tests/Vector_test.cpp index d9b32f9e9..24ecf86a0 100644 --- a/libutils/tests/Vector_test.cpp +++ b/libutils/tests/Vector_test.cpp @@ -89,9 +89,9 @@ TEST_F(VectorTest, SetCapacity_ShrinkBelowSize) { vector.add(4); vector.setCapacity(8); - ASSERT_EQ(8, vector.capacity()); + ASSERT_EQ(8U, vector.capacity()); vector.setCapacity(2); - ASSERT_EQ(8, vector.capacity()); + ASSERT_EQ(8U, vector.capacity()); } // NOTE: All of the tests below are useless because of the "TODO" above. -- cgit v1.2.3