Merge "Change return value of getTetherApnRequired() to a boolean."

This commit is contained in:
Mark Chien
2019-03-03 06:53:47 +00:00
committed by Gerrit Code Review
2 changed files with 8 additions and 11 deletions

View File

@@ -6907,14 +6907,12 @@ public class TelephonyManager {
} }
/** /**
* Check TETHER_DUN_REQUIRED and TETHER_DUN_APN settings, net.tethering.noprovisioning * Check whether DUN APN is required for tethering.
* SystemProperty to decide whether DUN APN is required for
* tethering.
* *
* @return 0: Not required. 1: required. 2: Not set. * @return {@code true} if DUN APN is required for tethering.
* @hide * @hide
*/ */
public int getTetherApnRequired() { public boolean getTetherApnRequired() {
try { try {
ITelephony telephony = getITelephony(); ITelephony telephony = getITelephony();
if (telephony != null) if (telephony != null)
@@ -6924,7 +6922,7 @@ public class TelephonyManager {
} catch (NullPointerException ex) { } catch (NullPointerException ex) {
Rlog.e(TAG, "hasMatchedTetherApnSetting NPE", ex); Rlog.e(TAG, "hasMatchedTetherApnSetting NPE", ex);
} }
return 2; return false;
} }

View File

@@ -700,13 +700,12 @@ interface ITelephony {
int getPreferredNetworkType(int subId); int getPreferredNetworkType(int subId);
/** /**
* Check TETHER_DUN_REQUIRED and TETHER_DUN_APN settings, net.tethering.noprovisioning * Check whether DUN APN is required for tethering.
* SystemProperty to decide whether DUN APN is required for
* tethering.
* *
* @return 0: Not required. 1: required. 2: Not set. * @return {@code true} if DUN APN is required for tethering.
* @hide
*/ */
int getTetherApnRequired(); boolean getTetherApnRequired();
/** /**
* Enables framework IMS and triggers IMS Registration. * Enables framework IMS and triggers IMS Registration.