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

Commit 152e11a1 authored by Jack Yu's avatar Jack Yu
Browse files

Added hashcode for ApnSetting class

Without hashCode, ApnSetting can't be used as a key in the
map.

Fix: 162663245
Test: Manual
Merged-In: I9445ec854709996c80830e4c8c498e664f1e4f9c
Change-Id: I9445ec854709996c80830e4c8c498e664f1e4f9c
(cherry picked from commit fe59b726)
parent 4f2a849e
Loading
Loading
Loading
Loading
+2 −0
Original line number Diff line number Diff line
@@ -1743,6 +1743,8 @@ public class TelephonyRegistry extends ITelephonyRegistry.Stub {
                        preciseState.getApnSetting());
                PreciseDataConnectionState oldState = mPreciseDataConnectionStates.get(phoneId)
                        .remove(key);
                log("Jack: oldState=" + oldState);
                log("Jack: newState=" + preciseState);
                if (!Objects.equals(oldState, preciseState)) {
                    for (Record r : mRecords) {
                        if (r.matchPhoneStateListenerEvent(
+9 −5
Original line number Diff line number Diff line
@@ -1214,12 +1214,16 @@ public class ApnSetting implements Parcelable {
        return false;
    }

    // TODO - if we have this function we should also have hashCode.
    // Also should handle changes in type order and perhaps case-insensitivity.
    @Override
    public int hashCode() {
        return Objects.hash(mApnName, mProxyAddress, mProxyPort, mMmsc, mMmsProxyAddress,
                mMmsProxyPort, mUser, mPassword, mAuthType, mApnTypeBitmask, mId, mOperatorNumeric,
                mProtocol, mRoamingProtocol, mMtu, mCarrierEnabled, mNetworkTypeBitmask, mProfileId,
                mPersistent, mMaxConns, mWaitTime, mMaxConnsTime, mMvnoType, mMvnoMatchData,
                mApnSetId, mCarrierId, mSkip464Xlat);
    }

    /**
     * @hide
     */
    @Override
    public boolean equals(Object o) {
        if (o instanceof ApnSetting == false) {
            return false;