diff options
| author | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-10 05:01:15 +0000 |
|---|---|---|
| committer | tfarina@chromium.org <tfarina@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-10 05:01:15 +0000 |
| commit | cd62bf6326d15b3ee6f9241dc0801aef5abd787f (patch) | |
| tree | d0ac3e9f4da980ba7965be6d055ead2be5d6626b | |
| parent | 254fed536428e39946d883c75c150740e18e124d (diff) | |
| download | chromium_src-cd62bf6326d15b3ee6f9241dc0801aef5abd787f.zip chromium_src-cd62bf6326d15b3ee6f9241dc0801aef5abd787f.tar.gz chromium_src-cd62bf6326d15b3ee6f9241dc0801aef5abd787f.tar.bz2 | |
Fixes for leveldb env_chromium unittests while building in GN.
This fixes two compilation errors found while building with gcc 4.8.2:
1- unused variable 'orig_bak_files' [-Werror=unused-variable]
2- comparison between signed and unsigned integer expressions
[-Werror=sign-compare]
BUG=361857
TEST=ninja -C out/Debug_gn
TBR=brettw@chromium.org,dgrogan@chromium.org
Review URL: https://codereview.chromium.org/226413009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@262919 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | third_party/leveldatabase/env_chromium_unittest.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/third_party/leveldatabase/env_chromium_unittest.cc b/third_party/leveldatabase/env_chromium_unittest.cc index 092ddfb..fef1498 100644 --- a/third_party/leveldatabase/env_chromium_unittest.cc +++ b/third_party/leveldatabase/env_chromium_unittest.cc @@ -208,7 +208,6 @@ TEST(ChromiumEnv, BackupTables) { // Ensure that deleting an ldb file also deletes its backup. int orig_ldb_files = CountFilesWithExtension(dir, FPL(".ldb")); - int orig_bak_files = CountFilesWithExtension(dir, FPL(".bak")); EXPECT_GT(ldb_files, 0); EXPECT_EQ(ldb_files, bak_files); EXPECT_TRUE(GetFirstLDBFile(dir, &ldb_file)); @@ -228,7 +227,7 @@ TEST(ChromiumEnv, GetChildrenEmptyDir) { std::vector<std::string> result; leveldb::Status status = env->GetChildren(dir.AsUTF8Unsafe(), &result); EXPECT_TRUE(status.ok()); - EXPECT_EQ(0, result.size()); + EXPECT_EQ(0U, result.size()); } TEST(ChromiumEnv, GetChildrenPriorResults) { |