Merge "Stop using namespace std."

This commit is contained in:
Dan Albert
2015-03-05 23:31:41 +00:00
committed by Gerrit Code Review
3 changed files with 19 additions and 26 deletions

View File

@@ -481,8 +481,9 @@ static void get_outline(image_info* image)
// assuming the image is a round rect, compute the radius by marching
// diagonally from the top left corner towards the center
image->outlineAlpha = max(max_alpha_over_row(image->rows[innerMidY], innerStartX, innerEndX),
max_alpha_over_col(image->rows, innerMidX, innerStartY, innerStartY));
image->outlineAlpha = std::max(
max_alpha_over_row(image->rows[innerMidY], innerStartX, innerEndX),
max_alpha_over_col(image->rows, innerMidX, innerStartY, innerStartY));
int diagonalInset = 0;
find_max_opacity(image->rows, innerStartX, innerStartY, innerMidX, innerMidY, 1, 1,

View File

@@ -1802,7 +1802,7 @@ status_t ResourceTable::addIncludedResources(Bundle* bundle, const sp<AaptAssets
}
const ResTable& featureTable = featureAssetManager.getResources(false);
mTypeIdOffset = max(mTypeIdOffset,
mTypeIdOffset = std::max(mTypeIdOffset,
findLargestTypeIdForPackage(featureTable, mAssetsPackage));
}
@@ -2667,20 +2667,16 @@ ResourceTable::validateLocalizations(void)
const String8 defaultLocale;
// For all strings...
for (map<String16, map<String8, SourcePos> >::iterator nameIter = mLocalizations.begin();
nameIter != mLocalizations.end();
nameIter++) {
const map<String8, SourcePos>& configSrcMap = nameIter->second;
for (const auto& nameIter : mLocalizations) {
const std::map<String8, SourcePos>& configSrcMap = nameIter.second;
// Look for strings with no default localization
if (configSrcMap.count(defaultLocale) == 0) {
SourcePos().warning("string '%s' has no default translation.",
String8(nameIter->first).string());
String8(nameIter.first).string());
if (mBundle->getVerbose()) {
for (map<String8, SourcePos>::const_iterator locales = configSrcMap.begin();
locales != configSrcMap.end();
locales++) {
locales->second.printf("locale %s found", locales->first.string());
for (const auto& locale : configSrcMap) {
locale.second.printf("locale %s found", locale.first.string());
}
}
// !!! TODO: throw an error here in some circumstances
@@ -2691,8 +2687,8 @@ ResourceTable::validateLocalizations(void)
const char* allConfigs = mBundle->getConfigurations().string();
const char* start = allConfigs;
const char* comma;
set<String8> missingConfigs;
std::set<String8> missingConfigs;
AaptLocaleValue locale;
do {
String8 config;
@@ -2726,13 +2722,11 @@ ResourceTable::validateLocalizations(void)
if (!missingConfigs.empty()) {
String8 configStr;
for (set<String8>::iterator iter = missingConfigs.begin();
iter != missingConfigs.end();
iter++) {
configStr.appendFormat(" %s", iter->string());
for (const auto& iter : missingConfigs) {
configStr.appendFormat(" %s", iter.string());
}
SourcePos().warning("string '%s' is missing %u required localizations:%s",
String8(nameIter->first).string(),
String8(nameIter.first).string(),
(unsigned int)missingConfigs.size(),
configStr.string());
}

View File

@@ -16,8 +16,6 @@
#include <queue>
#include <set>
using namespace std;
class XMLNode;
class ResourceTable;
@@ -28,7 +26,7 @@ enum {
XML_COMPILE_STRIP_WHITESPACE = 1<<3,
XML_COMPILE_STRIP_RAW_VALUES = 1<<4,
XML_COMPILE_UTF8 = 1<<5,
XML_COMPILE_STANDARD_RESOURCE =
XML_COMPILE_STRIP_COMMENTS | XML_COMPILE_ASSIGN_ATTRIBUTE_IDS
| XML_COMPILE_STRIP_WHITESPACE | XML_COMPILE_STRIP_RAW_VALUES
@@ -115,7 +113,7 @@ public:
* and would mess up iteration order for the existing
* resources.
*/
queue<CompileResourceWorkItem>& getWorkQueue() {
std::queue<CompileResourceWorkItem>& getWorkQueue() {
return mWorkQueue;
}
@@ -577,10 +575,10 @@ private:
size_t mNumLocal;
SourcePos mCurrentXmlPos;
Bundle* mBundle;
// key = string resource name, value = set of locales in which that name is defined
map<String16, map<String8, SourcePos> > mLocalizations;
queue<CompileResourceWorkItem> mWorkQueue;
std::map<String16, std::map<String8, SourcePos>> mLocalizations;
std::queue<CompileResourceWorkItem> mWorkQueue;
};
#endif