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

Commit 91e53b04 authored by Bernardo Rufino's avatar Bernardo Rufino Committed by Automerger Merge Worker
Browse files

Merge "Deprecate unsafe Parcel.readParcelableArray(ClassLoader)" am: 2ee662c7 

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/1910046

Change-Id: I3f12c9a32b1b2146ec83f0a04eb5768cafd2eecd
parents 4bd88f5c 2ee662c7

core/api/current.txt

+1 −1
Original line number Diff line number Diff line
@@ -31591,7 +31591,7 @@ package android.os { 
    method public <K, V> void readMap(@NonNull java.util.Map<? super K,? super V>, @Nullable ClassLoader, @NonNull Class<K>, @NonNull Class<V>);

    method @Deprecated @Nullable public <T extends android.os.Parcelable> T readParcelable(@Nullable ClassLoader);

    method @Nullable public <T extends android.os.Parcelable> T readParcelable(@Nullable ClassLoader, @NonNull Class<T>);

    method @Nullable public android.os.Parcelable[] readParcelableArray(@Nullable ClassLoader);

    method @Deprecated @Nullable public android.os.Parcelable[] readParcelableArray(@Nullable ClassLoader);

    method @Nullable public <T> T[] readParcelableArray(@Nullable ClassLoader, @NonNull Class<T>);

    method @Deprecated @Nullable public android.os.Parcelable.Creator<?> readParcelableCreator(@Nullable ClassLoader);

    method @Nullable public <T> android.os.Parcelable.Creator<T> readParcelableCreator(@Nullable ClassLoader, @NonNull Class<T>);

core/java/android/os/Parcel.java

+8 −1
Original line number Diff line number Diff line
@@ -4388,9 +4388,16 @@ public final class Parcel { 
     * The given class loader will be used to load any enclosed

     * Parcelables.

     * @return the Parcelable array, or null if the array is null

     *

     * @deprecated Use the type-safer version {@link #readParcelableArray(ClassLoader, Class)}

     *      starting from Android {@link Build.VERSION_CODES#TIRAMISU}. Also consider changing the

     *      format to use {@link #createTypedArray(Parcelable.Creator)} if possible (eg. if the

     *      items' class is final) since this is also more performant. Note that changing to the

     *      latter also requires changing the writes.

     */

    @Deprecated

    @Nullable

    public final Parcelable[] readParcelableArray(@Nullable ClassLoader loader) {

    public Parcelable[] readParcelableArray(@Nullable ClassLoader loader) {

        int N = readInt();

        if (N < 0) {

            return null;