Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0872f6d5 authored by Treehugger Robot's avatar Treehugger Robot Committed by Gerrit Code Review
Browse files

Merge "Don't put android.os.Parcelable in framework.aidl"

parents 492f249d fdbe7d1c

Android.mk

+1 −1
Original line number Diff line number Diff line
@@ -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 $$< $$@

tools/sdkparcelables/src/com/android/sdkparcelables/ParcelableDetector.kt

+4 −2
Original line number Diff line number Diff line
@@ -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

            }

tools/sdkparcelables/tests/com/android/sdkparcelables/ParcelableDetectorTest.kt

+4 −4
Original line number Diff line number Diff line
@@ -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"))

    }



}