diff options
author | Cole Faust <colefaust@google.com> | 2024-04-09 14:29:50 -0700 |
---|---|---|
committer | Cole Faust <colefaust@google.com> | 2024-04-09 14:34:41 -0700 |
commit | 1a1a4b20489402b4de369ab96974c92bf56f9123 (patch) | |
tree | da24907f48cdea6f8b44584ef9fad39b18c80c17 | |
parent | c24c225a9adb42797503a98437297846717afebd (diff) | |
download | Nfc-1a1a4b20489402b4de369ab96974c92bf56f9123.tar.gz |
Fix BoxedPrimitiveEquality errorprone issues
https://errorprone.info/bugpattern/BoxedPrimitiveEquality
Bug: 253827323
Test: Presubmits
Change-Id: I31c04d3f18e52442fd20f6a5c8fd2caf3fd433dc
-rw-r--r-- | src/com/android/nfc/cardemulation/AidRoutingManager.java | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/src/com/android/nfc/cardemulation/AidRoutingManager.java b/src/com/android/nfc/cardemulation/AidRoutingManager.java index fe29fea6..9f7a3bb9 100644 --- a/src/com/android/nfc/cardemulation/AidRoutingManager.java +++ b/src/com/android/nfc/cardemulation/AidRoutingManager.java @@ -30,6 +30,7 @@ import java.util.Arrays; import java.util.HashMap; import java.util.HashSet; import java.util.Map; +import java.util.Objects; import java.util.Set; public class AidRoutingManager { @@ -202,11 +203,13 @@ public class AidRoutingManager { //resolution, is routing required or not? private boolean isAidEntryUpdated(HashMap<String, Integer> currRouteForAid, Map.Entry<String, Integer> aidEntry, - HashMap<String, Integer> prevPowerForAid){ - if((currRouteForAid.get(aidEntry.getKey()) != aidEntry.getValue())|| - (mPowerForAid.get(aidEntry.getKey()) != prevPowerForAid.get(aidEntry.getKey()))){ - return true; - } + HashMap<String, Integer> prevPowerForAid) { + if(!Objects.equals(currRouteForAid.get(aidEntry.getKey()), aidEntry.getValue()) || + !Objects.equals( + mPowerForAid.get(aidEntry.getKey()), + prevPowerForAid.get(aidEntry.getKey()))) { + return true; + } return false; } |