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

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

Merge "ParcelableHolder uses its address for comparison" am: dad360f4 

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

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: Iee74d06a1077b36ab92f111dd7f660394ec864fe
parents 99c72813 dad360f4

libs/binder/include/binder/ParcelableHolder.h

+6 −12
Original line number Diff line number Diff line
@@ -104,28 +104,22 @@ public: 
    Stability getStability() const override { return mStability; }



    inline bool operator!=(const ParcelableHolder& rhs) const {

        return std::tie(mParcelable, mParcelPtr, mStability) !=

                std::tie(rhs.mParcelable, rhs.mParcelPtr, rhs.mStability);

        return this != &rhs;

    }

    inline bool operator<(const ParcelableHolder& rhs) const {

        return std::tie(mParcelable, mParcelPtr, mStability) <

                std::tie(rhs.mParcelable, rhs.mParcelPtr, rhs.mStability);

        return this < &rhs;

    }

    inline bool operator<=(const ParcelableHolder& rhs) const {

        return std::tie(mParcelable, mParcelPtr, mStability) <=

                std::tie(rhs.mParcelable, rhs.mParcelPtr, rhs.mStability);

        return this <= &rhs;

    }

    inline bool operator==(const ParcelableHolder& rhs) const {

        return std::tie(mParcelable, mParcelPtr, mStability) ==

                std::tie(rhs.mParcelable, rhs.mParcelPtr, rhs.mStability);

        return this == &rhs;

    }

    inline bool operator>(const ParcelableHolder& rhs) const {

        return std::tie(mParcelable, mParcelPtr, mStability) >

                std::tie(rhs.mParcelable, rhs.mParcelPtr, rhs.mStability);

        return this > &rhs;

    }

    inline bool operator>=(const ParcelableHolder& rhs) const {

        return std::tie(mParcelable, mParcelPtr, mStability) >=

                std::tie(rhs.mParcelable, rhs.mParcelPtr, rhs.mStability);

        return this >= &rhs;

    }



private:

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

+6 −18
Original line number Diff line number Diff line
@@ -114,24 +114,12 @@ class AParcelableHolder { 


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



    inline bool operator!=(const AParcelableHolder& rhs) const {

        return std::tie(mParcel, mStability) != std::tie(rhs.mParcel, rhs.mStability);

    }

    inline bool operator<(const AParcelableHolder& rhs) const {

        return std::tie(mParcel, mStability) < std::tie(rhs.mParcel, rhs.mStability);

    }

    inline bool operator<=(const AParcelableHolder& rhs) const {

        return std::tie(mParcel, mStability) <= std::tie(rhs.mParcel, rhs.mStability);

    }

    inline bool operator==(const AParcelableHolder& rhs) const {

        return std::tie(mParcel, mStability) == std::tie(rhs.mParcel, rhs.mStability);

    }

    inline bool operator>(const AParcelableHolder& rhs) const {

        return std::tie(mParcel, mStability) > std::tie(rhs.mParcel, rhs.mStability);

    }

    inline bool operator>=(const AParcelableHolder& rhs) const {

        return std::tie(mParcel, mStability) >= std::tie(rhs.mParcel, rhs.mStability);

    }

    inline bool operator!=(const AParcelableHolder& rhs) const { return this != &rhs; }

    inline bool operator<(const AParcelableHolder& rhs) const { return this < &rhs; }

    inline bool operator<=(const AParcelableHolder& rhs) const { return this <= &rhs; }

    inline bool operator==(const AParcelableHolder& rhs) const { return this == &rhs; }

    inline bool operator>(const AParcelableHolder& rhs) const { return this > &rhs; }

    inline bool operator>=(const AParcelableHolder& rhs) const { return this >= &rhs; }



   private:

    mutable ndk::ScopedAParcel mParcel;