Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix unused-but-set-variable warnings"

Browse Source
This commit is contained in:
Treehugger Robot
2022-08-12 20:00:13 +00:00
committed by Gerrit Code Review
parent 1db210c09c b172c3161f
commit eab658e213
2 changed files with 1 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
tools/aapt/Command.cpp
View File

@@ -1028,7 +1028,6 @@ int doDump(Bundle* bundle)
// These permissions are required by services implementing services // These permissions are required by services implementing services
// the system binds to (IME, Accessibility, PrintServices, etc.) // the system binds to (IME, Accessibility, PrintServices, etc.)
bool hasBindDeviceAdminPermission = false; bool hasBindDeviceAdminPermission = false;
bool hasBindInputMethodPermission = false;
bool hasBindAccessibilityServicePermission = false; bool hasBindAccessibilityServicePermission = false;
bool hasBindPrintServicePermission = false; bool hasBindPrintServicePermission = false;
bool hasBindNfcServicePermission = false; bool hasBindNfcServicePermission = false;
@@ -1738,7 +1737,6 @@ int doDump(Bundle* bundle)
hasMetaHostPaymentCategory = false; hasMetaHostPaymentCategory = false;
hasMetaOffHostPaymentCategory = false; hasMetaOffHostPaymentCategory = false;
hasBindDeviceAdminPermission = false; hasBindDeviceAdminPermission = false;
hasBindInputMethodPermission = false;
hasBindAccessibilityServicePermission = false; hasBindAccessibilityServicePermission = false;
hasBindPrintServicePermission = false; hasBindPrintServicePermission = false;
hasBindNfcServicePermission = false; hasBindNfcServicePermission = false;
@@ -1852,9 +1850,7 @@ int doDump(Bundle* bundle)
String8 permission = AaptXml::getAttribute(tree, PERMISSION_ATTR, String8 permission = AaptXml::getAttribute(tree, PERMISSION_ATTR,
&error); &error);
if (error == "") { if (error == "") {
if (permission == "android.permission.BIND_INPUT_METHOD") { if (permission ==
hasBindInputMethodPermission = true;
} else if (permission ==
"android.permission.BIND_ACCESSIBILITY_SERVICE") { "android.permission.BIND_ACCESSIBILITY_SERVICE") {
hasBindAccessibilityServicePermission = true; hasBindAccessibilityServicePermission = true;
} else if (permission == } else if (permission ==

8
tools/aapt/ResourceTable.cpp
View File

@@ -2970,14 +2970,6 @@ status_t ResourceTable::flatten(Bundle* bundle, const sp<const ResourceFilter>&
} }
e->setNameIndex(keyStrings.add(e->getName(), true)); e->setNameIndex(keyStrings.add(e->getName(), true));
// If this entry has no values for other configs,
// and is the default config, then it is special. Otherwise
// we want to add it with the config info.
ConfigDescription* valueConfig = NULL;
if (N != 1 || config == nullConfig) {
valueConfig = &config;
}
status_t err = e->prepareFlatten(&valueStrings, this, status_t err = e->prepareFlatten(&valueStrings, this,
&configTypeName, &config); &configTypeName, &config);
if (err != NO_ERROR) { if (err != NO_ERROR) {