diff options
author | Adam Lesinski <adamlesinski@google.com> | 2014-11-10 17:32:27 -0800 |
---|---|---|
committer | Adam Lesinski <adamlesinski@google.com> | 2014-11-10 17:39:47 -0800 |
commit | c3344e8a6d7402326204d1232a4c5ffa382f70a9 (patch) | |
tree | ab7863cddb787caecfc4ac303bdcbf3cca13e831 /tools/split-select | |
parent | 85d558cd486d195aabfc4b43cff8f338126f60a5 (diff) | |
download | frameworks_base-c3344e8a6d7402326204d1232a4c5ffa382f70a9.zip frameworks_base-c3344e8a6d7402326204d1232a4c5ffa382f70a9.tar.gz frameworks_base-c3344e8a6d7402326204d1232a4c5ffa382f70a9.tar.bz2 |
split-select: fix OSX build
Clang is a bit stricter about what template definition it chooses
for mixed types that are normally possible to cast between.
Operations like std::find(int iterator, int iterator, enum) will fail,
as Clang won't implicitly cast an enum to an int here.
Change-Id: Ib2276c382ebf2b877e1045d32b1a9d0a1fa4d4a4
Diffstat (limited to 'tools/split-select')
-rw-r--r-- | tools/split-select/RuleGenerator.cpp | 2 | ||||
-rw-r--r-- | tools/split-select/RuleGenerator_test.cpp | 8 | ||||
-rw-r--r-- | tools/split-select/TestRules.cpp | 2 |
3 files changed, 6 insertions, 6 deletions
diff --git a/tools/split-select/RuleGenerator.cpp b/tools/split-select/RuleGenerator.cpp index 72bb0c7..83c9795 100644 --- a/tools/split-select/RuleGenerator.cpp +++ b/tools/split-select/RuleGenerator.cpp @@ -38,7 +38,7 @@ sp<Rule> RuleGenerator::generateDensity(const Vector<int>& allDensities, size_t densityRule->op = Rule::AND_SUBRULES; const bool hasAnyDensity = std::find(allDensities.begin(), - allDensities.end(), ResTable_config::DENSITY_ANY) != allDensities.end(); + allDensities.end(), (int) ResTable_config::DENSITY_ANY) != allDensities.end(); if (hasAnyDensity) { sp<Rule> version = new Rule(); diff --git a/tools/split-select/RuleGenerator_test.cpp b/tools/split-select/RuleGenerator_test.cpp index 778d604..470cadc 100644 --- a/tools/split-select/RuleGenerator_test.cpp +++ b/tools/split-select/RuleGenerator_test.cpp @@ -47,10 +47,10 @@ TEST(RuleGeneratorTest, testAbiRules) { } TEST(RuleGeneratorTest, densityConstantsAreSane) { - EXPECT_LT(263, ConfigDescription::DENSITY_XHIGH); - EXPECT_GT(262, ConfigDescription::DENSITY_HIGH); - EXPECT_LT(363, ConfigDescription::DENSITY_XXHIGH); - EXPECT_GT(362, ConfigDescription::DENSITY_XHIGH); + EXPECT_LT(263, (int) ConfigDescription::DENSITY_XHIGH); + EXPECT_GT(262, (int) ConfigDescription::DENSITY_HIGH); + EXPECT_LT(363, (int) ConfigDescription::DENSITY_XXHIGH); + EXPECT_GT(362, (int) ConfigDescription::DENSITY_XHIGH); } TEST(RuleGeneratorTest, testDensityRules) { diff --git a/tools/split-select/TestRules.cpp b/tools/split-select/TestRules.cpp index f980dc4..86ccd6a 100644 --- a/tools/split-select/TestRules.cpp +++ b/tools/split-select/TestRules.cpp @@ -75,7 +75,7 @@ const Rule AlwaysTrue() { const char*, const char*, const sp<Rule>& actual, const Rule& expected) { const String8 expectedStr(expected.toJson()); - const String8 actualStr(actual != NULL ? actual->toJson() : ""); + const String8 actualStr(actual != NULL ? actual->toJson() : String8()); if (expectedStr != actualStr) { return ::testing::AssertionFailure() |