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

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

Merge "ParcelableHolder uses its address for comparison"

parents 7ac860e5 def77198
Loading
Loading
Loading
Loading
+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:
+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;