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

Merge "Update iterators during aapt2 diff" into sc-dev

Browse Source
This commit is contained in:
Ryan Mitchell
2021-03-30 23:00:10 +00:00
committed by Android (Google) Code Review
parent 2fcbd1290d 1499f504e4
commit 2f7bef0bbd
2 changed files with 52 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
tools/aapt2/Debug.cpp
View File

@@ -300,6 +300,10 @@ void Debug::PrintTable(const ResourceTable& table, const DebugPrintTableOptions&
break; break;
} }
if (entry->visibility.staged_api) {
printer->Print(" STAGED");
}
if (entry->overlayable_item) { if (entry->overlayable_item) {
printer->Print(" OVERLAYABLE"); printer->Print(" OVERLAYABLE");
} }

69
tools/aapt2/cmd/Diff.cpp
View File

@@ -83,7 +83,7 @@ static void EmitDiffLine(const Source& source, const StringPiece& message) {
} }
static bool IsSymbolVisibilityDifferent(const Visibility& vis_a, const Visibility& vis_b) { static bool IsSymbolVisibilityDifferent(const Visibility& vis_a, const Visibility& vis_b) {
return vis_a.level != vis_b.level; return vis_a.level != vis_b.level || vis_a.staged_api != vis_b.staged_api;
} }
template <typename Id> template <typename Id>
@@ -176,17 +176,23 @@ static bool EmitResourceTypeDiff(IAaptContext* context, LoadedApk* apk_a,
diff = true; diff = true;
} else { } else {
const auto& entry_a = *entry_a_iter; const auto& entry_a = *entry_a_iter;
const auto& entry_b = *entry_a_iter; const auto& entry_b = *entry_b_iter;
if (IsSymbolVisibilityDifferent(entry_a->visibility, entry_b->visibility)) { if (IsSymbolVisibilityDifferent(entry_a->visibility, entry_b->visibility)) {
std::stringstream str_stream; std::stringstream str_stream;
str_stream << pkg_a.name << ":" << type_a.type << "/" << entry_a->name str_stream << pkg_a.name << ":" << type_a.type << "/" << entry_a->name
<< " has different visibility ("; << " has different visibility (";
if (entry_b->visibility.staged_api) {
str_stream << "STAGED ";
}
if (entry_b->visibility.level == Visibility::Level::kPublic) { if (entry_b->visibility.level == Visibility::Level::kPublic) {
str_stream << "PUBLIC"; str_stream << "PUBLIC";
} else { } else {
str_stream << "PRIVATE"; str_stream << "PRIVATE";
} }
str_stream << " vs "; str_stream << " vs ";
if (entry_a->visibility.staged_api) {
str_stream << "STAGED ";
}
if (entry_a->visibility.level == Visibility::Level::kPublic) { if (entry_a->visibility.level == Visibility::Level::kPublic) {
str_stream << "PUBLIC"; str_stream << "PUBLIC";
} else { } else {
@@ -218,6 +224,12 @@ static bool EmitResourceTypeDiff(IAaptContext* context, LoadedApk* apk_a,
diff |= EmitResourceEntryDiff(context, apk_a, pkg_a, type_a, entry_a, apk_b, pkg_b, type_b, diff |= EmitResourceEntryDiff(context, apk_a, pkg_a, type_a, entry_a, apk_b, pkg_b, type_b,
entry_b); entry_b);
} }
if (entry_a_iter != type_a.entries.end()) {
++entry_a_iter;
}
if (entry_b_iter != type_b.entries.end()) {
++entry_b_iter;
}
} }
return diff; return diff;
} }
@@ -242,16 +254,18 @@ static bool EmitResourcePackageDiff(IAaptContext* context, LoadedApk* apk_a,
EmitDiffLine(apk_b->GetSource(), str_stream.str()); EmitDiffLine(apk_b->GetSource(), str_stream.str());
diff = true; diff = true;
} else { } else {
if (type_a_iter->visibility_level != type_b_iter->visibility_level) { const auto& type_a = *type_a_iter;
const auto& type_b = *type_b_iter;
if (type_a.visibility_level != type_b.visibility_level) {
std::stringstream str_stream; std::stringstream str_stream;
str_stream << pkg_a.name << ":" << type_a_iter->type << " has different visibility ("; str_stream << pkg_a.name << ":" << type_a.type << " has different visibility (";
if (type_b_iter->visibility_level == Visibility::Level::kPublic) { if (type_b.visibility_level == Visibility::Level::kPublic) {
str_stream << "PUBLIC"; str_stream << "PUBLIC";
} else { } else {
str_stream << "PRIVATE"; str_stream << "PRIVATE";
} }
str_stream << " vs "; str_stream << " vs ";
if (type_a_iter->visibility_level == Visibility::Level::kPublic) { if (type_a.visibility_level == Visibility::Level::kPublic) {
str_stream << "PUBLIC"; str_stream << "PUBLIC";
} else { } else {
str_stream << "PRIVATE"; str_stream << "PRIVATE";
@@ -259,18 +273,17 @@ static bool EmitResourcePackageDiff(IAaptContext* context, LoadedApk* apk_a,
str_stream << ")"; str_stream << ")";
EmitDiffLine(apk_b->GetSource(), str_stream.str()); EmitDiffLine(apk_b->GetSource(), str_stream.str());
diff = true; diff = true;
} else if (IsIdDiff(type_a_iter->visibility_level, type_a_iter->id, } else if (IsIdDiff(type_a.visibility_level, type_a.id, type_b.visibility_level, type_b.id)) {
type_b_iter->visibility_level, type_b_iter->id)) {
std::stringstream str_stream; std::stringstream str_stream;
str_stream << pkg_a.name << ":" << type_a_iter->type << " has different public ID ("; str_stream << pkg_a.name << ":" << type_a.type << " has different public ID (";
if (type_b_iter->id) { if (type_b.id) {
str_stream << "0x" << std::hex << type_b_iter->id.value(); str_stream << "0x" << std::hex << type_b.id.value();
} else { } else {
str_stream << "none"; str_stream << "none";
} }
str_stream << " vs "; str_stream << " vs ";
if (type_a_iter->id) { if (type_a.id) {
str_stream << "0x " << std::hex << type_a_iter->id.value(); str_stream << "0x " << std::hex << type_a.id.value();
} else { } else {
str_stream << "none"; str_stream << "none";
} }
@@ -278,7 +291,13 @@ static bool EmitResourcePackageDiff(IAaptContext* context, LoadedApk* apk_a,
EmitDiffLine(apk_b->GetSource(), str_stream.str()); EmitDiffLine(apk_b->GetSource(), str_stream.str());
diff = true; diff = true;
} }
diff |= EmitResourceTypeDiff(context, apk_a, pkg_a, *type_a_iter, apk_b, pkg_b, *type_b_iter); diff |= EmitResourceTypeDiff(context, apk_a, pkg_a, type_a, apk_b, pkg_b, type_b);
}
if (type_a_iter != pkg_a.types.end()) {
++type_a_iter;
}
if (type_b_iter != pkg_b.types.end()) {
++type_b_iter;
} }
} }
return diff; return diff;
@@ -305,17 +324,19 @@ static bool EmitResourceTableDiff(IAaptContext* context, LoadedApk* apk_a, Loade
EmitDiffLine(apk_b->GetSource(), str_stream.str()); EmitDiffLine(apk_b->GetSource(), str_stream.str());
diff = true; diff = true;
} else { } else {
if (package_a_iter->id != package_b_iter->id) { const auto& package_a = *package_a_iter;
const auto& package_b = *package_b_iter;
if (package_a.id != package_b.id) {
std::stringstream str_stream; std::stringstream str_stream;
str_stream << "package '" << package_a_iter->name << "' has different id ("; str_stream << "package '" << package_a.name << "' has different id (";
if (package_b_iter->id) { if (package_b.id) {
str_stream << "0x" << std::hex << package_b_iter->id.value(); str_stream << "0x" << std::hex << package_b.id.value();
} else { } else {
str_stream << "none"; str_stream << "none";
} }
str_stream << " vs "; str_stream << " vs ";
if (package_a_iter->id) { if (package_a.id) {
str_stream << "0x" << std::hex << package_b_iter->id.value(); str_stream << "0x" << std::hex << package_b.id.value();
} else { } else {
str_stream << "none"; str_stream << "none";
} }
@@ -323,7 +344,13 @@ static bool EmitResourceTableDiff(IAaptContext* context, LoadedApk* apk_a, Loade
EmitDiffLine(apk_b->GetSource(), str_stream.str()); EmitDiffLine(apk_b->GetSource(), str_stream.str());
diff = true; diff = true;
} }
diff |= EmitResourcePackageDiff(context, apk_a, *package_a_iter, apk_b, *package_b_iter); diff |= EmitResourcePackageDiff(context, apk_a, package_a, apk_b, package_b);
}
if (package_a_iter != table_a.packages.end()) {
++package_a_iter;
}
if (package_b_iter != table_b.packages.end()) {
++package_b_iter;
} }
} }