drgreen/android_frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Don't put android.os.Parcelable in framework.aidl" am: 0872f6d5e3 am: 6864210fe2

Browse Source 
am: 37ba3de333

Change-Id: Ib8d8277171ede82ea4626b6d2de2fcb9c1a1d307
This commit is contained in:
Colin Cross 2018-02-13 08:41:41 +00:00 committed by android-build-merger
commit 7d07ad1ef2
3 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Android.mk
View File

@ -36,7 +36,7 @@ aidl_parcelables :=
define stubs-to-aidl-parcelables
gen := $(TARGET_OUT_COMMON_INTERMEDIATES)/$1.aidl
aidl_parcelables += $$(gen)
$$(gen): $(call java-lib-header-files,$1) | $(HOST_OUT_EXECUTABLES)/sdkparcelables
$$(gen): $(call java-lib-header-files,$1) $(HOST_OUT_EXECUTABLES)/sdkparcelables
@echo Extract SDK parcelables: $$@
rm -f $$@
$(HOST_OUT_EXECUTABLES)/sdkparcelables $$< $$@

6
tools/sdkparcelables/src/com/android/sdkparcelables/ParcelableDetector.kt
View File

@ -27,6 +27,8 @@ class ParcelableDetector {
impl.build()
return impl.parcelables
}
const val PARCELABLE_CLASS = "android/os/Parcelable"
}
private class Impl(val ancestors: Map<String, Ancestors>) {
@ -35,7 +37,7 @@ class ParcelableDetector {
fun build() {
val classList = ancestors.keys
classList.filterTo(parcelables, this::isParcelable)
classList.filterTo(parcelables, { (it != PARCELABLE_CLASS) && isParcelable(it) })
parcelables.sort()
}
@ -44,7 +46,7 @@ class ParcelableDetector {
return false
}
if (c == "android/os/Parcelable") {
if (c == PARCELABLE_CLASS) {
return true
}

8
tools/sdkparcelables/tests/com/android/sdkparcelables/ParcelableDetectorTest.kt
View File

@ -28,7 +28,7 @@ class ParcelableDetectorTest {
val parcelables = ParcelableDetector.ancestorsToParcelables(ancestorMap)
assertEquals(parcelables, listOf("android/os/Parcelable", "android/test/Parcelable"))
assertEquals(parcelables, listOf("android/test/Parcelable"))
}
@Test
@ -39,7 +39,7 @@ class ParcelableDetectorTest {
val parcelables = ParcelableDetector.ancestorsToParcelables(ancestorMap)
assertEquals(parcelables, listOf("android/os/Parcelable", "android/test/Parcelable"))
assertEquals(parcelables, listOf("android/test/Parcelable"))
}
@Test
@ -51,7 +51,7 @@ class ParcelableDetectorTest {
val parcelables = ParcelableDetector.ancestorsToParcelables(ancestorMap)
assertEquals(parcelables, listOf("android/os/Parcelable", "android/test/Parcelable", "android/test/SuperParcelable"))
assertEquals(parcelables, listOf("android/test/Parcelable", "android/test/SuperParcelable"))
}
@Test
@ -63,7 +63,7 @@ class ParcelableDetectorTest {
val parcelables = ParcelableDetector.ancestorsToParcelables(ancestorMap)
assertEquals(parcelables, listOf("android/os/Parcelable", "android/test/IParcelable", "android/test/Parcelable"))
assertEquals(parcelables, listOf("android/test/IParcelable", "android/test/Parcelable"))
}
}