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

Merge "Remove unnecessary ConfiguredJarList.apex(int) method"

Browse Source
This commit is contained in:
Paul Duffin
2020-10-26 20:13:45 +00:00
committed by Gerrit Code Review
parent 93d8c6443e 1e8c6072bb
commit 728e626cb6
1 changed files with 3 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
android/config.go
View File

@@ -1336,11 +1336,6 @@ func (l *ConfiguredJarList) Len() int {
return len(l.jars)
}
// Apex component of idx-th pair on the list.
func (l *ConfiguredJarList) apex(idx int) string {
return l.apexes[idx]
}
// Jar component of idx-th pair on the list.
func (l *ConfiguredJarList) Jar(idx int) string {
return l.jars[idx]
@@ -1354,7 +1349,7 @@ func (l *ConfiguredJarList) ContainsJar(jar string) bool {
// If the list contains the given (apex, jar) pair.
func (l *ConfiguredJarList) containsApexJarPair(apex, jar string) bool {
for i := 0; i < l.Len(); i++ {
if apex == l.apex(i) && jar == l.Jar(i) {
if apex == l.apexes[i] && jar == l.jars[i] {
return true
}
}
@@ -1378,7 +1373,7 @@ func (l *ConfiguredJarList) RemoveList(list ConfiguredJarList) {
jars := make([]string, 0, l.Len())
for i, jar := range l.jars {
apex := l.apex(i)
apex := l.apexes[i]
if !list.containsApexJarPair(apex, jar) {
apexes = append(apexes, apex)
jars = append(jars, jar)
@@ -1404,7 +1399,7 @@ func (l *ConfiguredJarList) CopyOfApexJarPairs() []string {
pairs := make([]string, 0, l.Len())
for i, jar := range l.jars {
apex := l.apex(i)
apex := l.apexes[i]
pairs = append(pairs, apex+":"+jar)
}