Make CopyOf generic
Test: util_test.go Change-Id: If8f2534ec454903eb967dc9ec0c8708096d1baff
This commit is contained in:
@@ -25,12 +25,12 @@ import (
|
|||||||
)
|
)
|
||||||
|
|
||||||
// CopyOf returns a new slice that has the same contents as s.
|
// CopyOf returns a new slice that has the same contents as s.
|
||||||
func CopyOf(s []string) []string {
|
func CopyOf[T any](s []T) []T {
|
||||||
// If the input is nil, return nil and not an empty list
|
// If the input is nil, return nil and not an empty list
|
||||||
if s == nil {
|
if s == nil {
|
||||||
return s
|
return s
|
||||||
}
|
}
|
||||||
return append([]string{}, s...)
|
return append([]T{}, s...)
|
||||||
}
|
}
|
||||||
|
|
||||||
// Concat returns a new slice concatenated from the two input slices. It does not change the input
|
// Concat returns a new slice concatenated from the two input slices. It does not change the input
|
||||||
|
@@ -385,7 +385,7 @@ func TestCopyOfEmptyAndNil(t *testing.T) {
|
|||||||
emptyList := []string{}
|
emptyList := []string{}
|
||||||
copyOfEmptyList := CopyOf(emptyList)
|
copyOfEmptyList := CopyOf(emptyList)
|
||||||
AssertBoolEquals(t, "Copy of an empty list should be an empty list and not nil", true, copyOfEmptyList != nil)
|
AssertBoolEquals(t, "Copy of an empty list should be an empty list and not nil", true, copyOfEmptyList != nil)
|
||||||
copyOfNilList := CopyOf(nil)
|
copyOfNilList := CopyOf([]string(nil))
|
||||||
AssertBoolEquals(t, "Copy of a nil list should be a nil list and not an empty list", true, copyOfNilList == nil)
|
AssertBoolEquals(t, "Copy of a nil list should be a nil list and not an empty list", true, copyOfNilList == nil)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@@ -630,5 +630,3 @@ func contains(l []string, s string) bool {
|
|||||||
}
|
}
|
||||||
return false
|
return false
|
||||||
}
|
}
|
||||||
|
|
||||||
var copyOf = android.CopyOf
|
|
||||||
|
@@ -250,8 +250,6 @@ func bcpForDexpreopt(ctx android.PathContext, withUpdatable bool) (android.Writa
|
|||||||
|
|
||||||
var defaultBootclasspathKey = android.NewOnceKey("defaultBootclasspath")
|
var defaultBootclasspathKey = android.NewOnceKey("defaultBootclasspath")
|
||||||
|
|
||||||
var copyOf = android.CopyOf
|
|
||||||
|
|
||||||
func init() {
|
func init() {
|
||||||
android.RegisterMakeVarsProvider(pctx, dexpreoptConfigMakevars)
|
android.RegisterMakeVarsProvider(pctx, dexpreoptConfigMakevars)
|
||||||
}
|
}
|
||||||
|
@@ -299,7 +299,7 @@ func generateSameDirRoboTestConfigJar(ctx android.ModuleContext, outputFile andr
|
|||||||
func (r *robolectricTest) generateRoboSrcJar(ctx android.ModuleContext, outputFile android.WritablePath,
|
func (r *robolectricTest) generateRoboSrcJar(ctx android.ModuleContext, outputFile android.WritablePath,
|
||||||
instrumentedApp *AndroidApp) {
|
instrumentedApp *AndroidApp) {
|
||||||
|
|
||||||
srcJarArgs := copyOf(instrumentedApp.srcJarArgs)
|
srcJarArgs := android.CopyOf(instrumentedApp.srcJarArgs)
|
||||||
srcJarDeps := append(android.Paths(nil), instrumentedApp.srcJarDeps...)
|
srcJarDeps := append(android.Paths(nil), instrumentedApp.srcJarDeps...)
|
||||||
|
|
||||||
for _, m := range ctx.GetDirectDepsWithTag(roboCoverageLibsTag) {
|
for _, m := range ctx.GetDirectDepsWithTag(roboCoverageLibsTag) {
|
||||||
|
Reference in New Issue
Block a user