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

Commit a539a7a3 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "ParcelableHolder shouldn't write anything for empty case" am: 97c2606b 

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

Change-Id: I5f288f4e83a26b378bed41edc082ed501bb1f5a1
parents db8a9b5d 97c2606b

core/java/android/os/ParcelableHolder.java

+15 −5
Original line number Diff line number Diff line
@@ -170,16 +170,21 @@ public final class ParcelableHolder implements Parcelable { 


        mParcelable = null;



        int dataSize = parcel.readInt();

        if (dataSize < 0) {

            throw new IllegalArgumentException("dataSize from parcel is negative");

        } else if (dataSize == 0) {

            if (mParcel != null) {

                mParcel.recycle();

                mParcel = null;

            }

            return;

        }

        if (mParcel == null) {

            mParcel = Parcel.obtain();

        }

        mParcel.setDataPosition(0);

        mParcel.setDataSize(0);



        int dataSize = parcel.readInt();

        if (dataSize < 0) {

            throw new IllegalArgumentException("dataSize from parcel is negative");

        }

        int dataStartPos = parcel.dataPosition();



        mParcel.appendFrom(parcel, dataStartPos, dataSize);
@@ -196,6 +201,11 @@ public final class ParcelableHolder implements Parcelable { 
            return;

        }



        if (mParcelable == null) {

            parcel.writeInt(0);

            return;

        }



        int sizePos = parcel.dataPosition();

        parcel.writeInt(0);

        int dataStartPos = parcel.dataPosition();