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"

Browse Source
This commit is contained in:
Treehugger Robot
2018-02-12 22:44:39 +00:00
committed by Gerrit Code Review
parent 492f249dbb fdbe7d1ca9
commit 0872f6d5e3
3 changed files with 9 additions and 7 deletions
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 define stubs-to-aidl-parcelables
gen := $(TARGET_OUT_COMMON_INTERMEDIATES)/$1.aidl gen := $(TARGET_OUT_COMMON_INTERMEDIATES)/$1.aidl
aidl_parcelables += $$(gen) 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: $$@ @echo Extract SDK parcelables: $$@
rm -f $$@ rm -f $$@
$(HOST_OUT_EXECUTABLES)/sdkparcelables $$< $$@ $(HOST_OUT_EXECUTABLES)/sdkparcelables $$< $$@

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

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

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

@@ -28,7 +28,7 @@ class ParcelableDetectorTest {
val parcelables = ParcelableDetector.ancestorsToParcelables(ancestorMap) val parcelables = ParcelableDetector.ancestorsToParcelables(ancestorMap)
assertEquals(parcelables, listOf("android/os/Parcelable", "android/test/Parcelable")) assertEquals(parcelables, listOf("android/test/Parcelable"))
} }
@Test @Test
@@ -39,7 +39,7 @@ class ParcelableDetectorTest {
val parcelables = ParcelableDetector.ancestorsToParcelables(ancestorMap) val parcelables = ParcelableDetector.ancestorsToParcelables(ancestorMap)
assertEquals(parcelables, listOf("android/os/Parcelable", "android/test/Parcelable")) assertEquals(parcelables, listOf("android/test/Parcelable"))
} }
@Test @Test
@@ -51,7 +51,7 @@ class ParcelableDetectorTest {
val parcelables = ParcelableDetector.ancestorsToParcelables(ancestorMap) 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 @Test
@@ -63,7 +63,7 @@ class ParcelableDetectorTest {
val parcelables = ParcelableDetector.ancestorsToParcelables(ancestorMap) 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"))
} }
} }