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

Commit 3a2bc9f7 authored by Nathan Harold's avatar Nathan Harold Committed by android-build-merger
Browse files

Merge "Additional checks for ModemActivityInfo.isValid()" into nyc-dev am: f6496db8

am: 79593949

* commit '79593949':
  Additional checks for ModemActivityInfo.isValid()

Change-Id: I62f8aaa28c2fed3aada88b9e724b9548f02ec4c9
parents ad82eae2 79593949
Loading
Loading
Loading
Loading
+18 −6
Original line number Diff line number Diff line
@@ -151,12 +151,24 @@ public class ModemActivityInfo implements Parcelable {
     * @return if the record is valid
     */
    public boolean isValid() {
        int totalTxTimeMs = 0;
        int txTime [] = getTxTimeMillis();
        for (int i = 0; i < TX_POWER_LEVELS; i++) {
            totalTxTimeMs += txTime[i];
        for (int txVal : getTxTimeMillis()) {
            if(txVal < 0) {
                return false;
            }
        }

        return ((getIdleTimeMillis() >= 0) && (getSleepTimeMillis() >= 0)
                && (getRxTimeMillis() >= 0) && (getEnergyUsed() >= 0) && !isEmpty());
    }
        return ((getIdleTimeMillis() >= 0) && (totalTxTimeMs >= 0)
                && (getSleepTimeMillis() >= 0) && (getIdleTimeMillis() >= 0));

    private boolean isEmpty() {
        for (int txVal : getTxTimeMillis()) {
            if(txVal != 0) {
                return false;
            }
        }

        return ((getIdleTimeMillis() == 0) && (getSleepTimeMillis() == 0)
                && (getRxTimeMillis() == 0) && (getEnergyUsed() == 0));
    }
}