commit d650788b5add05310920d5bb7ad37fba7c96e1c0
author Yi Kong <yikong@google.com> Tue Dec 05 17:06:08 2023 +0900
committer Yi Kong <yikong@google.com> Tue Dec 05 17:16:46 2023 +0900
tree 7e30cae6decdde8abf9fc7c5f023cb9c1786c343
parent a2262c1f7bb02a7c186297b69af25df399085802

Fix -Wunused-variable compiler warning

frameworks/base/tools/aapt2/link/ManifestFixer.cpp:341:23: error: unused variable 'attr' [-Werror,-Wunused-variable]
  341 |   if (xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, "glEsVersion")) {
      |                       ^~~~

Test: presubmit
Change-Id: Icec4737a5f8076173a2aff54532a79fa123df326

tools/aapt2/link/ManifestFixer.cpp

diff --git a/tools/aapt2/link/ManifestFixer.cpp b/tools/aapt2/link/ManifestFixer.cpp
index c4f6e70..0b16e2c 100644
--- a/tools/aapt2/link/ManifestFixer.cpp
+++ b/tools/aapt2/link/ManifestFixer.cpp
@@ -338,7 +338,7 @@
   }
 
   bool has_gl_es_version = false;
-  if (xml::Attribute* attr = el->FindAttribute(xml::kSchemaAndroid, "glEsVersion")) {
+  if (el->FindAttribute(xml::kSchemaAndroid, "glEsVersion")) {
     if (has_name) {
       diag->Error(android::DiagMessage(el->line_number)
                   << "cannot define both android:name and android:glEsVersion in <uses-feature>");