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

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

Merge "setParcelable uses const ref instead of pointer" am: e42a54b1 am:... 

Merge "setParcelable uses const ref instead of pointer" am: e42a54b1 am: 2350502a am: d4a8d871

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/1508296

Change-Id: I5c850f2cf77f4d040ecbe8891498786bce01cd97
parents 615f465c d4a8d871

libs/binder/ndk/include_cpp/android/binder_parcelable_utils.h

+5 −3
Original line number Original line Diff line number Diff line
@@ -82,13 +82,13 @@ class AParcelableHolder { 
    }

    }





    template <typename T>

    template <typename T>

    bool setParcelable(T* p) {

    bool setParcelable(const T& p) {

        if (p && this->mStability > T::_aidl_stability) {

        if (this->mStability > T::_aidl_stability) {

            return false;

            return false;

        }

        }

        AParcel_reset(mParcel.get());

        AParcel_reset(mParcel.get());

        AParcel_writeString(mParcel.get(), T::descriptor, strlen(T::descriptor));

        AParcel_writeString(mParcel.get(), T::descriptor, strlen(T::descriptor));

        p->writeToParcel(mParcel.get());

        p.writeToParcel(mParcel.get());

        return true;

        return true;

    }

    }
@@ -112,6 +112,8 @@ class AParcelableHolder { 
        return std::move(ret);

        return std::move(ret);

    }

    }





    void reset() { AParcel_reset(mParcel.get()); }



   private:

   private:

    mutable ndk::ScopedAParcel mParcel;

    mutable ndk::ScopedAParcel mParcel;

    parcelable_stability_t mStability;

    parcelable_stability_t mStability;