diff options
author | benchan@chromium.org <benchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-09 08:39:23 +0000 |
---|---|---|
committer | benchan@chromium.org <benchan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-09 08:39:23 +0000 |
commit | ce671d2d1c7d43a56e62e1bd495392e2e5739d25 (patch) | |
tree | fd41c824ca9908a703d44674f5bfafc7365f41bd /chromeos/dbus | |
parent | 12f007bfc595dd82c194992e5e9c62b7384be066 (diff) | |
download | chromium_src-ce671d2d1c7d43a56e62e1bd495392e2e5739d25.zip chromium_src-ce671d2d1c7d43a56e62e1bd495392e2e5739d25.tar.gz chromium_src-ce671d2d1c7d43a56e62e1bd495392e2e5739d25.tar.bz2 |
Fix gcc 4.7 compilation errors in chromeos_unittests
gcc 4.7 gives the following error when compiling chromeos_unittests on Linux (with chromeos=1):
../../chromeos/dbus/cros_disks_client_unittest.cc: In member function 'virtual void chromeos::CrosDisksClientTest_DiskInfo_Test::TestBody()':
../../chromeos/dbus/cros_disks_client_unittest.cc:125:3: error: converting 'false' to pointer type for argument 1 of 'char testing::internal::IsNullLiteralHelper(testing::internal::Secret*)' [-Werror=conversion-null]
See googletest issue 322 (https://code.google.com/p/googletest/issues/detail?id=322) for details.
This CL works around the issue by switching the unit test from EXPECT_EQ(false, something) to EXPECT_TRUE(false == something)
BUG=None
TEST=Build and run chromeos_unittests with gcc 4.7
Review URL: https://chromiumcodereview.appspot.com/18339011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@210529 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chromeos/dbus')
-rw-r--r-- | chromeos/dbus/cros_disks_client_unittest.cc | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/chromeos/dbus/cros_disks_client_unittest.cc b/chromeos/dbus/cros_disks_client_unittest.cc index 91abc8f..ce2c77c 100644 --- a/chromeos/dbus/cros_disks_client_unittest.cc +++ b/chromeos/dbus/cros_disks_client_unittest.cc @@ -122,7 +122,14 @@ TEST(CrosDisksClientTest, DiskInfo) { EXPECT_EQ(kDeviceFile, result.device_path()); EXPECT_EQ(kDeviceIsDrive, result.is_drive()); EXPECT_EQ(kDeviceIsReadOnly, result.is_read_only()); - EXPECT_EQ(kDevicePresentationHide, result.is_hidden()); + // Use EXPECT_TRUE(kDevicePresentationHide == result.is_hidden()) instead of + // EXPECT_EQ(kDevicePresentationHide, result.is_hidden()) as gcc 4.7 issues + // the following warning on EXPECT_EQ(false, x), which is turned into an error + // with -Werror=conversion-null: + // + // converting 'false' to pointer type for argument 1 of + // 'char testing::internal::IsNullLiteralHelper(testing::internal::Secret*)' + EXPECT_TRUE(kDevicePresentationHide == result.is_hidden()); EXPECT_EQ(kDeviceIsMediaAvailable, result.has_media()); EXPECT_EQ(kDeviceIsOnBootDevice, result.on_boot_device()); EXPECT_EQ(kNativePath, result.system_path()); |