Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e1d9d12b authored by Yi Kong's avatar Yi Kong Committed by Automerger Merge Worker
Browse files

Merge "Fix -Wunused-variable compiler warning" into main am: 76a85e92 

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2858973



Change-Id: Ie5bd3e2d561a863c50b6e4310f02b0a2785cbd45
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 479bb71c 76a85e92

tools/aapt2/cmd/Link.cpp

+4 −4
Original line number Diff line number Diff line
@@ -388,7 +388,7 @@ ResourceFileFlattener::ResourceFileFlattener(const ResourceFileFlattenerOptions& 
  // Build up the rules for degrading newer attributes to older ones.

  // NOTE(adamlesinski): These rules are hardcoded right now, but they should be

  // generated from the attribute definitions themselves (b/62028956).

  if (const SymbolTable::Symbol* s = symm->FindById(R::attr::paddingHorizontal)) {

  if (symm->FindById(R::attr::paddingHorizontal)) {

    std::vector<ReplacementAttr> replacements{

        {"paddingLeft", R::attr::paddingLeft, Attribute(android::ResTable_map::TYPE_DIMENSION)},

        {"paddingRight", R::attr::paddingRight, Attribute(android::ResTable_map::TYPE_DIMENSION)},
@@ -397,7 +397,7 @@ ResourceFileFlattener::ResourceFileFlattener(const ResourceFileFlattenerOptions& 
        util::make_unique<DegradeToManyRule>(std::move(replacements));

  }



  if (const SymbolTable::Symbol* s = symm->FindById(R::attr::paddingVertical)) {

  if (symm->FindById(R::attr::paddingVertical)) {

    std::vector<ReplacementAttr> replacements{

        {"paddingTop", R::attr::paddingTop, Attribute(android::ResTable_map::TYPE_DIMENSION)},

        {"paddingBottom", R::attr::paddingBottom, Attribute(android::ResTable_map::TYPE_DIMENSION)},
@@ -406,7 +406,7 @@ ResourceFileFlattener::ResourceFileFlattener(const ResourceFileFlattenerOptions& 
        util::make_unique<DegradeToManyRule>(std::move(replacements));

  }



  if (const SymbolTable::Symbol* s = symm->FindById(R::attr::layout_marginHorizontal)) {

  if (symm->FindById(R::attr::layout_marginHorizontal)) {

    std::vector<ReplacementAttr> replacements{

        {"layout_marginLeft", R::attr::layout_marginLeft,

         Attribute(android::ResTable_map::TYPE_DIMENSION)},
@@ -417,7 +417,7 @@ ResourceFileFlattener::ResourceFileFlattener(const ResourceFileFlattenerOptions& 
        util::make_unique<DegradeToManyRule>(std::move(replacements));

  }



  if (const SymbolTable::Symbol* s = symm->FindById(R::attr::layout_marginVertical)) {

  if (symm->FindById(R::attr::layout_marginVertical)) {

    std::vector<ReplacementAttr> replacements{

        {"layout_marginTop", R::attr::layout_marginTop,

         Attribute(android::ResTable_map::TYPE_DIMENSION)},

tools/aapt2/link/ManifestFixer.cpp

+1 −1
Original line number Diff line number Diff line
@@ -338,7 +338,7 @@ static bool VerifyUsesFeature(xml::Element* el, android::SourcePathDiagnostics* 
  }



  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>");