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

Commit aa874961 authored by Amit Mahajan's avatar Amit Mahajan Committed by markchien
Browse files

Change return value of getTetherApnRequired() to a boolean. 

Test: Basic sanity - verified tethering for T-Mobile
Bug: 63150712
Change-Id: I03ce7b6bcf4bbd9d0dfcd733141c0d20d9b3a790
parent 346bae7c

telephony/java/android/telephony/TelephonyManager.java

+4 −6
Original line number Diff line number Diff line
@@ -6274,14 +6274,12 @@ public class TelephonyManager { 
    }



    /**

     * Check TETHER_DUN_REQUIRED and TETHER_DUN_APN settings, net.tethering.noprovisioning

     * SystemProperty to decide whether DUN APN is required for

     * tethering.

     * Check 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

     */

    public int getTetherApnRequired() {

    public boolean getTetherApnRequired() {

        try {

            ITelephony telephony = getITelephony();

            if (telephony != null)
@@ -6291,7 +6289,7 @@ public class TelephonyManager { 
        } catch (NullPointerException ex) {

            Rlog.e(TAG, "hasMatchedTetherApnSetting NPE", ex);

        }

        return 2;

        return false;

    }

telephony/java/com/android/internal/telephony/ITelephony.aidl

+4 −5
Original line number Diff line number Diff line
@@ -681,13 +681,12 @@ interface ITelephony { 
    int getPreferredNetworkType(int subId);



    /**

     * Check TETHER_DUN_REQUIRED and TETHER_DUN_APN settings, net.tethering.noprovisioning

     * SystemProperty to decide whether DUN APN is required for

     * tethering.

     * Check 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.