summaryrefslogtreecommitdiff
path: root/tools/aapt2/ResourceTable.cpp
diff options
context:
space:
mode:
author Jeremy Meyer <jakmcbane@google.com> 2022-05-20 20:35:42 +0000
committer Jeremy Meyer <jakmcbane@google.com> 2022-05-31 23:43:03 +0000
commit56f36e8f2fa808128bc68d4ae9adad9e65b90367 (patch)
tree07b10081f9a44efde8a3b1cc0c33264253451ca8 /tools/aapt2/ResourceTable.cpp
parent14be35a5de080e7ae797edbecd94e0684244d647 (diff)
Move StringPool to libandroidfw
Test: verified affected tests pass Bug: 232940948 Change-Id: I22089893d7e5013f759c39ce190bec07fa6435db
Diffstat (limited to 'tools/aapt2/ResourceTable.cpp')
-rw-r--r--tools/aapt2/ResourceTable.cpp24
1 files changed, 13 insertions, 11 deletions
diff --git a/tools/aapt2/ResourceTable.cpp b/tools/aapt2/ResourceTable.cpp
index 0f5118da9408..cb4811445ed1 100644
--- a/tools/aapt2/ResourceTable.cpp
+++ b/tools/aapt2/ResourceTable.cpp
@@ -464,20 +464,21 @@ ResourceTableView ResourceTable::GetPartitionedView(const ResourceTableViewOptio
return view;
}
-bool ResourceTable::AddResource(NewResource&& res, IDiagnostics* diag) {
+bool ResourceTable::AddResource(NewResource&& res, android::IDiagnostics* diag) {
CHECK(diag != nullptr) << "Diagnostic pointer is null";
const bool validate = validation_ == Validation::kEnabled;
- const Source source = res.value ? res.value->GetSource() : Source{};
+ const android::Source source = res.value ? res.value->GetSource() : android::Source{};
if (validate && !res.allow_mangled && !IsValidResourceEntryName(res.name.entry)) {
- diag->Error(DiagMessage(source)
+ diag->Error(android::DiagMessage(source)
<< "resource '" << res.name << "' has invalid entry name '" << res.name.entry);
return false;
}
if (res.id.has_value() && !res.id->first.is_valid()) {
- diag->Error(DiagMessage(source) << "trying to add resource '" << res.name << "' with ID "
- << res.id->first << " but that ID is invalid");
+ diag->Error(android::DiagMessage(source)
+ << "trying to add resource '" << res.name << "' with ID " << res.id->first
+ << " but that ID is invalid");
return false;
}
@@ -513,7 +514,7 @@ bool ResourceTable::AddResource(NewResource&& res, IDiagnostics* diag) {
if (res.id.has_value()) {
if (entry->id && entry->id.value() != res.id->first) {
if (res.id->second != OnIdConflict::CREATE_ENTRY) {
- diag->Error(DiagMessage(source)
+ diag->Error(android::DiagMessage(source)
<< "trying to add resource '" << res.name << "' with ID " << res.id->first
<< " but resource already has ID " << entry->id.value());
return false;
@@ -541,9 +542,9 @@ bool ResourceTable::AddResource(NewResource&& res, IDiagnostics* diag) {
if (res.overlayable.has_value()) {
if (entry->overlayable_item) {
- diag->Error(DiagMessage(res.overlayable->source)
+ diag->Error(android::DiagMessage(res.overlayable->source)
<< "duplicate overlayable declaration for resource '" << res.name << "'");
- diag->Error(DiagMessage(entry->overlayable_item.value().source)
+ diag->Error(android::DiagMessage(entry->overlayable_item.value().source)
<< "previous declaration here");
return false;
}
@@ -581,9 +582,10 @@ bool ResourceTable::AddResource(NewResource&& res, IDiagnostics* diag) {
break;
case CollisionResult::kConflict:
- diag->Error(DiagMessage(source) << "duplicate value for resource '" << res.name << "' "
- << "with config '" << res.config << "'");
- diag->Error(DiagMessage(source) << "resource previously defined here");
+ diag->Error(android::DiagMessage(source)
+ << "duplicate value for resource '" << res.name << "' "
+ << "with config '" << res.config << "'");
+ diag->Error(android::DiagMessage(source) << "resource previously defined here");
return false;
case CollisionResult::kKeepOriginal: