From 06c7bbee7c48293406e19e79d28616908c56a45e Mon Sep 17 00:00:00 2001 From: Andres Morales Date: Mon, 24 Nov 2014 08:16:47 -0800 Subject: Fix NfcDiscoveryParameters Added a new field without adding it to .equals(). Leads to P2P not getting enabled in SW. Bug: 18501664 Change-Id: I602045cbbd84a87edff5b82a3c0db96c9159e21f --- src/com/android/nfc/NfcDiscoveryParameters.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/com/android/nfc/NfcDiscoveryParameters.java b/src/com/android/nfc/NfcDiscoveryParameters.java index a71ad949..1149836d 100644 --- a/src/com/android/nfc/NfcDiscoveryParameters.java +++ b/src/com/android/nfc/NfcDiscoveryParameters.java @@ -72,6 +72,7 @@ public final class NfcDiscoveryParameters { static final int NFC_POLL_DEFAULT = -1; + // NOTE: when adding a new field, don't forget to update equals() and toString() below private int mTechMask = 0; private boolean mEnableLowPowerDiscovery = true; private boolean mEnableReaderMode = false; @@ -117,7 +118,8 @@ public final class NfcDiscoveryParameters { return mTechMask == params.mTechMask && (mEnableLowPowerDiscovery == params.mEnableLowPowerDiscovery) && (mEnableReaderMode == params.mEnableReaderMode) && - (mEnableHostRouting == params.mEnableHostRouting); + (mEnableHostRouting == params.mEnableHostRouting) + && (mEnableP2p == params.mEnableP2p); } @Override @@ -130,7 +132,8 @@ public final class NfcDiscoveryParameters { } sb.append("mEnableLPD: " + Boolean.toString(mEnableLowPowerDiscovery) + "\n"); sb.append("mEnableReader: " + Boolean.toString(mEnableReaderMode) + "\n"); - sb.append("mEnableHostRouting: " + Boolean.toString(mEnableHostRouting)); + sb.append("mEnableHostRouting: " + Boolean.toString(mEnableHostRouting) + "\n"); + sb.append("mEnableP2p: " + Boolean.toString(mEnableP2p)); return sb.toString(); } -- cgit v1.2.3