Add a comment for a mysterious Boolean map value
PiperOrigin-RevId: 271892448
diff --git a/src/tools/android/java/com/google/devtools/build/android/AndroidResourceSymbolSink.java b/src/tools/android/java/com/google/devtools/build/android/AndroidResourceSymbolSink.java
index 88d2866..bba4f32 100644
--- a/src/tools/android/java/com/google/devtools/build/android/AndroidResourceSymbolSink.java
+++ b/src/tools/android/java/com/google/devtools/build/android/AndroidResourceSymbolSink.java
@@ -22,10 +22,13 @@
void acceptSimpleResource(DependencyInfo dependencyInfo, ResourceType type, String name);
+ // "inlineable" below affects how resource IDs are assigned by
+ // PlaceholderIdFieldInitializerBuilder to attempt to match the final IDs assigned by aapt1. This
+ // shouldn't matter, but legacy tests with ODR violations might be relying on this.
void acceptStyleableResource(
DependencyInfo dependencyInfo,
FullyQualifiedName key,
- Map<FullyQualifiedName, Boolean> attrs);
+ Map<FullyQualifiedName, /*inlineable=*/ Boolean> attrs);
/**
* Marks a resource as public.
diff --git a/src/tools/android/java/com/google/devtools/build/android/PlaceholderIdFieldInitializerBuilder.java b/src/tools/android/java/com/google/devtools/build/android/PlaceholderIdFieldInitializerBuilder.java
index 6b2d3e2..efce1dd 100644
--- a/src/tools/android/java/com/google/devtools/build/android/PlaceholderIdFieldInitializerBuilder.java
+++ b/src/tools/android/java/com/google/devtools/build/android/PlaceholderIdFieldInitializerBuilder.java
@@ -189,7 +189,8 @@
private final Map<ResourceType, SortedMap<String, Optional<Integer>>> publicIds =
new EnumMap<>(ResourceType.class);
- private final Map<String, Map<String, Boolean>> styleableAttrs = new LinkedHashMap<>();
+ private final Map<String, Map<String, /*inlineable=*/ Boolean>> styleableAttrs =
+ new LinkedHashMap<>();
private PlaceholderIdFieldInitializerBuilder(AndroidFrameworkAttrIdProvider androidIdProvider) {
this.androidIdProvider = androidIdProvider;
