From: Kenny Root Date: Tue, 12 Apr 2011 17:27:15 +0000 (-0700) Subject: resolved conflicts for merge of 87b3c0dc to honeycomb-plus-aosp X-Git-Url: https://git.saurik.com/android/aapt.git/commitdiff_plain/838914df8530468f5ce5e011a7049a9c48c81e54?hp=-c resolved conflicts for merge of 87b3c0dc to honeycomb-plus-aosp Change-Id: Ia1a0024aabf531438203eb9fea3a10dd15eabe53 --- 838914df8530468f5ce5e011a7049a9c48c81e54 diff --combined ResourceTable.cpp index 5339566,6459d0e..6d5fcc2 --- a/ResourceTable.cpp +++ b/ResourceTable.cpp @@@ -2444,7 -2444,7 +2444,7 @@@ ResourceTable::validateLocalizations(vo if (configSet.count(defaultLocale) == 0) { fprintf(stdout, "aapt: warning: string '%s' has no default translation in %s; found:", String8(nameIter->first).string(), mBundle->getResourceSourceDirs()[0]); - for (set::iterator locales = configSet.begin(); + for (set::const_iterator locales = configSet.begin(); locales != configSet.end(); locales++) { fprintf(stdout, " %s", (*locales).string()); @@@ -2555,7 -2555,7 +2555,7 @@@ ResourceFilter::parse(const char* arg } bool -ResourceFilter::match(int axis, uint32_t value) +ResourceFilter::match(int axis, uint32_t value) const { if (value == 0) { // they didn't specify anything so take everything @@@ -2571,7 -2571,7 +2571,7 @@@ } bool -ResourceFilter::match(const ResTable_config& config) +ResourceFilter::match(const ResTable_config& config) const { if (config.locale) { uint32_t locale = (config.country[1] << 24) | (config.country[0] << 16) @@@ -2624,8 -2624,6 +2624,8 @@@ status_t ResourceTable::flatten(Bundle const size_t N = mOrderedPackages.size(); size_t pi; + const static String16 mipmap16("mipmap"); + bool useUTF8 = !bundle->getWantUTF16() && bundle->isMinSdkAtLeast(SDK_FROYO); // Iterate through all data, collecting all values (strings, @@@ -2648,10 -2646,7 +2648,10 @@@ typeStrings.add(String16(""), false); continue; } - typeStrings.add(t->getName(), false); + const String16 typeName(t->getName()); + typeStrings.add(typeName, false); + + const bool filterable = (typeName != mipmap16); const size_t N = t->getOrderedConfigs().size(); for (size_t ci=0; cigetEntries().size(); for (size_t ei=0; eigetEntries().keyAt(ei); - if (!filter.match(config)) { + if (filterable && !filter.match(config)) { continue; } sp e = c->getEntries().valueAt(ei); @@@ -2742,8 -2737,6 +2742,8 @@@ "Type name %s not found", String8(typeName).string()); + const bool filterable = (typeName != mipmap16); + const size_t N = t != NULL ? t->getOrderedConfigs().size() : 0; // First write the typeSpec chunk, containing information about @@@ -2768,7 -2761,7 +2768,7 @@@ (((uint8_t*)data->editData()) + typeSpecStart + sizeof(ResTable_typeSpec)); memset(typeSpecFlags, 0, sizeof(uint32_t)*N); - + for (size_t ei=0; ei cl = t->getOrderedConfigs().itemAt(ei); if (cl->getPublic()) { @@@ -2776,11 -2769,11 +2776,11 @@@ } const size_t CN = cl->getEntries().size(); for (size_t ci=0; cigetEntries().keyAt(ci))) { + if (filterable && !filter.match(cl->getEntries().keyAt(ci))) { continue; } for (size_t cj=ci+1; cjgetEntries().keyAt(cj))) { + if (filterable && !filter.match(cl->getEntries().keyAt(cj))) { continue; } typeSpecFlags[ei] |= htodl( @@@ -2817,7 -2810,7 +2817,7 @@@ config.screenWidth, config.screenHeight)); - if (!filter.match(config)) { + if (filterable && !filter.match(config)) { continue; } @@@ -3681,7 -3674,9 +3681,9 @@@ sp ResourceTabl { sp p = mPackages.valueFor(package); if (p == NULL) { - if (mIsAppPackage) { + if (mBundle->getIsOverlayPackage()) { + p = new Package(package, 0x00); + } else if (mIsAppPackage) { if (mHaveAppPackage) { fprintf(stderr, "Adding multiple application package resources; only one is allowed.\n" "Use -x to create extended resources.\n");