StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Make ConfiguredJarList immutable" am: e1878c101d am: 7ad2fa969b am: 89e8d18655

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1471776

Change-Id: I67c890408cc0d1029420350e54dc3959a75e7fb8
This commit is contained in:
Paul Duffin
2020-10-26 21:17:06 +00:00
committed by Automerger Merge Worker
parent 60b9747bc5 89e8d18655
commit 8df961a826
2 changed files with 23 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

38
android/config.go
View File

@@ -1362,13 +1362,22 @@ func (l *ConfiguredJarList) IndexOfJar(jar string) int {
}
// Append an (apex, jar) pair to the list.
func (l *ConfiguredJarList) Append(apex string, jar string) {
l.apexes = append(l.apexes, apex)
l.jars = append(l.jars, jar)
func (l *ConfiguredJarList) Append(apex string, jar string) ConfiguredJarList {
// Create a copy of the backing arrays before appending to avoid sharing backing
// arrays that are mutated across instances.
apexes := make([]string, 0, len(l.apexes)+1)
copy(apexes, l.apexes)
apexes = append(apexes, apex)
jars := make([]string, 0, len(l.jars)+1)
copy(jars, l.jars)
jars = append(l.jars, jar)
return ConfiguredJarList{apexes, jars}
}
// Filter out sublist.
func (l *ConfiguredJarList) RemoveList(list ConfiguredJarList) {
func (l *ConfiguredJarList) RemoveList(list ConfiguredJarList) ConfiguredJarList {
apexes := make([]string, 0, l.Len())
jars := make([]string, 0, l.Len())
@@ -1380,13 +1389,7 @@ func (l *ConfiguredJarList) RemoveList(list ConfiguredJarList) {
}
}
l.apexes = apexes
l.jars = jars
}
// A copy of itself.
func (l *ConfiguredJarList) CopyOf() ConfiguredJarList {
return ConfiguredJarList{CopyOf(l.apexes), CopyOf(l.jars)}
return ConfiguredJarList{apexes, jars}
}
// A copy of the list of strings containing jar components.
@@ -1461,17 +1464,16 @@ func splitConfiguredJarPair(ctx PathContext, str string) (string, string) {
}
func CreateConfiguredJarList(ctx PathContext, list []string) ConfiguredJarList {
apexes := make([]string, 0, len(list))
jars := make([]string, 0, len(list))
apexes := make([]string, len(list))
jars := make([]string, len(list))
l := ConfiguredJarList{apexes, jars}
for _, apexjar := range list {
for i, apexjar := range list {
apex, jar := splitConfiguredJarPair(ctx, apexjar)
l.Append(apex, jar)
apexes[i] = apex
jars[i] = jar
}
return l
return ConfiguredJarList{apexes, jars}
}
func EmptyConfiguredJarList() ConfiguredJarList {

7
java/dexpreopt_config.go
View File

@@ -81,13 +81,12 @@ func genBootImageConfigs(ctx android.PathContext) map[string]*bootImageConfig {
targets := dexpreoptTargets(ctx)
deviceDir := android.PathForOutput(ctx, ctx.Config().DeviceName())
artModules := global.ArtApexJars.CopyOf()
artModules := global.ArtApexJars
// With EMMA_INSTRUMENT_FRAMEWORK=true the Core libraries depend on jacoco.
if ctx.Config().IsEnvTrue("EMMA_INSTRUMENT_FRAMEWORK") {
artModules.Append("com.android.art", "jacocoagent")
artModules = artModules.Append("com.android.art", "jacocoagent")
}
frameworkModules := global.BootJars.CopyOf()
frameworkModules.RemoveList(artModules)
frameworkModules := global.BootJars.RemoveList(artModules)
artSubdir := "apex/art_boot_images/javalib"
frameworkSubdir := "system/framework"