Clean up
Wink Saville [Fri, 22 Feb 2013 23:08:33 +0000 (15:08 -0800)]
 - Add @Overrides where needed.
 - Update javadoc comments
 - Remove extra semi-colons
 - Rename DataConnection.java to DataConnectionBase.java
 - Rename GsmDataConnection.java to DataConnection.java
 - Add defaults to switch statements
 - Remove/fix most "if (false)" statements. Fixed by using a CONSTANT
 - Fix hidden variables by hoisting to base class or renaming
 - Tweak some debug output

Change-Id: If38de2fdeaacafbf40cdfd7f84dc5c52030ba2a3

131 files changed:
src/java/android/provider/Telephony.java
src/java/android/telephony/CellBroadcastMessage.java
src/java/android/telephony/SmsCbCmasInfo.java
src/java/android/telephony/SmsCbEtwsInfo.java
src/java/android/telephony/SmsCbLocation.java
src/java/android/telephony/SmsMessage.java
src/java/android/telephony/gsm/SmsManager.java
src/java/android/telephony/gsm/SmsMessage.java
src/java/com/android/internal/telephony/BaseCommands.java
src/java/com/android/internal/telephony/Call.java
src/java/com/android/internal/telephony/CallForwardInfo.java
src/java/com/android/internal/telephony/CallManager.java
src/java/com/android/internal/telephony/CallTracker.java
src/java/com/android/internal/telephony/CommandsInterface.java
src/java/com/android/internal/telephony/Connection.java
src/java/com/android/internal/telephony/DataCallState.java
src/java/com/android/internal/telephony/DefaultPhoneNotifier.java
src/java/com/android/internal/telephony/DriverCall.java
src/java/com/android/internal/telephony/IccPhoneBookInterfaceManager.java
src/java/com/android/internal/telephony/IccPhoneBookInterfaceManagerProxy.java
src/java/com/android/internal/telephony/IccSmsInterfaceManager.java
src/java/com/android/internal/telephony/IccSmsInterfaceManagerProxy.java
src/java/com/android/internal/telephony/IccUtils.java
src/java/com/android/internal/telephony/MccTable.java
src/java/com/android/internal/telephony/OperatorInfo.java
src/java/com/android/internal/telephony/Phone.java
src/java/com/android/internal/telephony/PhoneBase.java
src/java/com/android/internal/telephony/PhoneFactory.java
src/java/com/android/internal/telephony/PhoneProxy.java
src/java/com/android/internal/telephony/PhoneStateIntentReceiver.java
src/java/com/android/internal/telephony/PhoneSubInfo.java
src/java/com/android/internal/telephony/PhoneSubInfoProxy.java
src/java/com/android/internal/telephony/RIL.java
src/java/com/android/internal/telephony/RestrictedState.java
src/java/com/android/internal/telephony/RetryManager.java
src/java/com/android/internal/telephony/SMSDispatcher.java
src/java/com/android/internal/telephony/ServiceStateTracker.java
src/java/com/android/internal/telephony/SmsMessageBase.java
src/java/com/android/internal/telephony/SmsResponse.java
src/java/com/android/internal/telephony/SmsUsageMonitor.java
src/java/com/android/internal/telephony/TelephonyCapabilities.java
src/java/com/android/internal/telephony/WapPushOverSms.java
src/java/com/android/internal/telephony/cat/CatCmdMessage.java
src/java/com/android/internal/telephony/cat/CatService.java
src/java/com/android/internal/telephony/cat/CommandDetails.java
src/java/com/android/internal/telephony/cat/CommandParams.java
src/java/com/android/internal/telephony/cat/CommandParamsFactory.java
src/java/com/android/internal/telephony/cat/Duration.java
src/java/com/android/internal/telephony/cat/IconLoader.java
src/java/com/android/internal/telephony/cat/Input.java
src/java/com/android/internal/telephony/cat/Item.java
src/java/com/android/internal/telephony/cat/Menu.java
src/java/com/android/internal/telephony/cat/ResultCode.java
src/java/com/android/internal/telephony/cat/ResultException.java
src/java/com/android/internal/telephony/cat/TextMessage.java
src/java/com/android/internal/telephony/cat/Tone.java
src/java/com/android/internal/telephony/cat/ToneSettings.java
src/java/com/android/internal/telephony/cat/ValueParser.java
src/java/com/android/internal/telephony/cdma/CDMALTEPhone.java
src/java/com/android/internal/telephony/cdma/CDMAPhone.java
src/java/com/android/internal/telephony/cdma/CdmaCall.java
src/java/com/android/internal/telephony/cdma/CdmaCallTracker.java
src/java/com/android/internal/telephony/cdma/CdmaCallWaitingNotification.java
src/java/com/android/internal/telephony/cdma/CdmaConnection.java
src/java/com/android/internal/telephony/cdma/CdmaInformationRecords.java
src/java/com/android/internal/telephony/cdma/CdmaLteServiceStateTracker.java
src/java/com/android/internal/telephony/cdma/CdmaMmiCode.java
src/java/com/android/internal/telephony/cdma/CdmaSMSDispatcher.java
src/java/com/android/internal/telephony/cdma/CdmaServiceStateTracker.java
src/java/com/android/internal/telephony/cdma/CdmaSubscriptionSourceManager.java
src/java/com/android/internal/telephony/cdma/EriManager.java
src/java/com/android/internal/telephony/cdma/RuimPhoneBookInterfaceManager.java
src/java/com/android/internal/telephony/cdma/RuimSmsInterfaceManager.java
src/java/com/android/internal/telephony/cdma/SignalToneUtil.java
src/java/com/android/internal/telephony/cdma/SmsMessage.java
src/java/com/android/internal/telephony/cdma/TtyIntent.java
src/java/com/android/internal/telephony/cdma/sms/BearerData.java
src/java/com/android/internal/telephony/dataconnection/ApnContext.java
src/java/com/android/internal/telephony/dataconnection/ApnSetting.java
src/java/com/android/internal/telephony/dataconnection/DataConnection.java
src/java/com/android/internal/telephony/dataconnection/DataConnectionAc.java
src/java/com/android/internal/telephony/dataconnection/DataConnectionBase.java [new file with mode: 0644]
src/java/com/android/internal/telephony/dataconnection/DataConnectionTracker.java
src/java/com/android/internal/telephony/dataconnection/DataConnectionTrackerBase.java [new file with mode: 0644]
src/java/com/android/internal/telephony/dataconnection/GsmDataConnection.java [deleted file]
src/java/com/android/internal/telephony/dataconnection/GsmDataConnectionTracker.java [deleted file]
src/java/com/android/internal/telephony/gsm/GSMPhone.java
src/java/com/android/internal/telephony/gsm/GsmCall.java
src/java/com/android/internal/telephony/gsm/GsmCallTracker.java
src/java/com/android/internal/telephony/gsm/GsmConnection.java
src/java/com/android/internal/telephony/gsm/GsmMmiCode.java
src/java/com/android/internal/telephony/gsm/GsmSMSDispatcher.java
src/java/com/android/internal/telephony/gsm/GsmServiceStateTracker.java
src/java/com/android/internal/telephony/gsm/GsmSmsAddress.java
src/java/com/android/internal/telephony/gsm/SimPhoneBookInterfaceManager.java
src/java/com/android/internal/telephony/gsm/SimSmsInterfaceManager.java
src/java/com/android/internal/telephony/gsm/SmsMessage.java
src/java/com/android/internal/telephony/gsm/SuppServiceNotification.java
src/java/com/android/internal/telephony/gsm/UsimPhoneBookManager.java
src/java/com/android/internal/telephony/sip/SipCallBase.java
src/java/com/android/internal/telephony/sip/SipCommandInterface.java
src/java/com/android/internal/telephony/sip/SipConnectionBase.java
src/java/com/android/internal/telephony/sip/SipPhone.java
src/java/com/android/internal/telephony/sip/SipPhoneBase.java
src/java/com/android/internal/telephony/test/ModelInterpreter.java
src/java/com/android/internal/telephony/test/SimulatedCommands.java
src/java/com/android/internal/telephony/test/SimulatedGsmCallState.java
src/java/com/android/internal/telephony/uicc/AdnRecord.java
src/java/com/android/internal/telephony/uicc/AdnRecordCache.java
src/java/com/android/internal/telephony/uicc/AdnRecordLoader.java
src/java/com/android/internal/telephony/uicc/CsimFileHandler.java
src/java/com/android/internal/telephony/uicc/IccCardApplicationStatus.java
src/java/com/android/internal/telephony/uicc/IccCardProxy.java
src/java/com/android/internal/telephony/uicc/IccCardStatus.java
src/java/com/android/internal/telephony/uicc/IccFileHandler.java
src/java/com/android/internal/telephony/uicc/IccIoResult.java
src/java/com/android/internal/telephony/uicc/IccRecords.java
src/java/com/android/internal/telephony/uicc/IccServiceTable.java
src/java/com/android/internal/telephony/uicc/IccUtils.java
src/java/com/android/internal/telephony/uicc/IsimFileHandler.java
src/java/com/android/internal/telephony/uicc/IsimUiccRecords.java
src/java/com/android/internal/telephony/uicc/RuimFileHandler.java
src/java/com/android/internal/telephony/uicc/RuimRecords.java
src/java/com/android/internal/telephony/uicc/SIMFileHandler.java
src/java/com/android/internal/telephony/uicc/SIMRecords.java
src/java/com/android/internal/telephony/uicc/SpnOverride.java
src/java/com/android/internal/telephony/uicc/UiccCard.java
src/java/com/android/internal/telephony/uicc/UiccCardApplication.java
src/java/com/android/internal/telephony/uicc/UiccController.java
src/java/com/android/internal/telephony/uicc/UsimFileHandler.java
src/java/com/android/internal/telephony/uicc/VoiceMailConstants.java

index 50f3203..7d312e9 100644 (file)
@@ -25,7 +25,6 @@ import android.content.Intent;
 import android.database.Cursor;
 import android.database.sqlite.SqliteWrapper;
 import android.net.Uri;
-import android.os.Environment;
 import android.telephony.SmsMessage;
 import android.text.TextUtils;
 import android.telephony.Rlog;
@@ -44,8 +43,6 @@ import java.util.regex.Pattern;
  */
 public final class Telephony {
     private static final String TAG = "Telephony";
-    private static final boolean DEBUG = true;
-    private static final boolean LOCAL_LOGV = false;
 
     // Constructor
     public Telephony() {
@@ -1243,7 +1240,6 @@ public final class Telephony {
      */
     public static final class Threads implements ThreadsColumns {
         private static final String[] ID_PROJECTION = { BaseColumns._ID };
-        private static final String STANDARD_ENCODING = "UTF-8";
         private static final Uri THREAD_ID_CONTENT_URI = Uri.parse(
                 "content://mms-sms/threadID");
         public static final Uri CONTENT_URI = Uri.withAppendedPath(
index 36c238d..f745ba5 100644 (file)
@@ -19,7 +19,6 @@ package android.telephony;
 import android.content.ContentValues;
 import android.content.Context;
 import android.database.Cursor;
-import android.graphics.Typeface;
 import android.os.Parcel;
 import android.os.Parcelable;
 import android.provider.Telephony;
@@ -27,10 +26,7 @@ import android.telephony.SmsCbCmasInfo;
 import android.telephony.SmsCbEtwsInfo;
 import android.telephony.SmsCbLocation;
 import android.telephony.SmsCbMessage;
-import android.text.Spannable;
-import android.text.SpannableStringBuilder;
 import android.text.format.DateUtils;
-import android.text.style.StyleSpan;
 
 /**
  * Application wrapper for {@link SmsCbMessage}. This is Parcelable so that
@@ -76,10 +72,12 @@ public class CellBroadcastMessage implements Parcelable {
     }
 
     /** Parcelable: no special flags. */
+    @Override
     public int describeContents() {
         return 0;
     }
 
+    @Override
     public void writeToParcel(Parcel out, int flags) {
         mSmsCbMessage.writeToParcel(out, flags);
         out.writeLong(mDeliveryTime);
@@ -88,10 +86,12 @@ public class CellBroadcastMessage implements Parcelable {
 
     public static final Parcelable.Creator<CellBroadcastMessage> CREATOR
             = new Parcelable.Creator<CellBroadcastMessage>() {
+        @Override
         public CellBroadcastMessage createFromParcel(Parcel in) {
             return new CellBroadcastMessage(in);
         }
 
+        @Override
         public CellBroadcastMessage[] newArray(int size) {
             return new CellBroadcastMessage[size];
         }
index 7a89d94..c912924 100644 (file)
@@ -297,10 +297,12 @@ public class SmsCbCmasInfo implements Parcelable {
     /** Creator for unparcelling objects. */
     public static final Parcelable.Creator<SmsCbCmasInfo>
             CREATOR = new Parcelable.Creator<SmsCbCmasInfo>() {
+        @Override
         public SmsCbCmasInfo createFromParcel(Parcel in) {
             return new SmsCbCmasInfo(in);
         }
 
+        @Override
         public SmsCbCmasInfo[] newArray(int size) {
             return new SmsCbCmasInfo[size];
         }
index f208fbf..286efd6 100644 (file)
@@ -163,7 +163,7 @@ public class SmsCbEtwsInfo implements Parcelable {
         time.second = second;
 
         // Timezone offset is in quarter hours.
-        return time.toMillis(true) - (long) (timezoneOffset * 15 * 60 * 1000);
+        return time.toMillis(true) - timezoneOffset * 15 * 60 * 1000;
     }
 
     /**
@@ -195,10 +195,12 @@ public class SmsCbEtwsInfo implements Parcelable {
 
     /** Creator for unparcelling objects. */
     public static final Creator<SmsCbEtwsInfo> CREATOR = new Creator<SmsCbEtwsInfo>() {
+        @Override
         public SmsCbEtwsInfo createFromParcel(Parcel in) {
             return new SmsCbEtwsInfo(in);
         }
 
+        @Override
         public SmsCbEtwsInfo[] newArray(int size) {
             return new SmsCbEtwsInfo[size];
         }
index 7b5bd0d..6eb72a8 100644 (file)
 
 package android.telephony;
 
-import android.os.Bundle;
 import android.os.Parcel;
 import android.os.Parcelable;
-import android.telephony.gsm.GsmCellLocation;
 
 /**
  * Represents the location and geographical scope of a cell broadcast message.
index c35f09e..e83d175 100644 (file)
@@ -22,7 +22,6 @@ import android.telephony.Rlog;
 import com.android.internal.telephony.GsmAlphabet;
 import com.android.internal.telephony.GsmAlphabet.TextEncodingDetails;
 import com.android.internal.telephony.SmsConstants;
-import com.android.internal.telephony.SmsHeader;
 import com.android.internal.telephony.SmsMessageBase;
 import com.android.internal.telephony.SmsMessageBase.SubmitPduBase;
 
@@ -37,7 +36,7 @@ import static android.telephony.TelephonyManager.PHONE_TYPE_CDMA;
  * A Short Message Service message.
  */
 public class SmsMessage {
-    private static final String LOG_TAG = "SMS";
+    private static final String LOG_TAG = "SmsMessage";
 
     /**
      * SMS Class enumeration.
@@ -101,6 +100,7 @@ public class SmsMessage {
         public byte[] encodedScAddress; // Null if not applicable.
         public byte[] encodedMessage;
 
+        @Override
         public String toString() {
             return "SubmitPdu: encodedScAddress = "
                     + Arrays.toString(encodedScAddress)
index 3b75298..5630678 100644 (file)
@@ -219,7 +219,7 @@ import java.util.ArrayList;
      */
     @Deprecated
     public final ArrayList<android.telephony.SmsMessage> getAllMessagesFromSim() {
-        return mSmsMgrProxy.getAllMessagesFromIcc();
+        return android.telephony.SmsManager.getAllMessagesFromIcc();
     }
 
     /** Free space (TS 51.011 10.5.3).
index 7a814c3..8d5dac7 100644 (file)
 
 package android.telephony.gsm;
 
-import android.os.Parcel;
 import android.telephony.TelephonyManager;
 
 import com.android.internal.telephony.GsmAlphabet;
-import com.android.internal.telephony.EncodeException;
 import com.android.internal.telephony.SmsHeader;
 import com.android.internal.telephony.SmsMessageBase;
 import com.android.internal.telephony.SmsMessageBase.SubmitPduBase;
@@ -36,9 +34,6 @@ import static android.telephony.TelephonyManager.PHONE_TYPE_CDMA;
  */
 @Deprecated
 public class SmsMessage {
-    private static final boolean LOCAL_DEBUG = true;
-    private static final String LOG_TAG = "SMS";
-
     /**
      * SMS Class enumeration.
      * See TS 23.038.
@@ -127,6 +122,7 @@ public class SmsMessage {
         }
 
         /** @deprecated Use android.telephony.SmsMessage. */
+        @Override
         @Deprecated
         public String toString() {
             return "SubmitPdu: encodedScAddress = "
@@ -617,6 +613,7 @@ public class SmsMessage {
      * @return Specific SmsMessage.
      * @deprecated Use android.telephony.SmsMessage.
      */
+    @Deprecated
     private static final SmsMessageBase getSmsFacility(){
         int activePhone = TelephonyManager.getDefault().getCurrentPhoneType();
         if (PHONE_TYPE_CDMA == activePhone) {
index d059931..8ce3873 100644 (file)
@@ -22,21 +22,12 @@ import android.os.RegistrantList;
 import android.os.Registrant;
 import android.os.Handler;
 import android.os.AsyncResult;
-import android.os.SystemProperties;
 import android.telephony.TelephonyManager;
-import android.telephony.Rlog;
-
-import java.io.FileInputStream;
-import java.io.IOException;
-import java.util.regex.Matcher;
-import java.util.regex.Pattern;
 
 /**
  * {@hide}
  */
 public abstract class BaseCommands implements CommandsInterface {
-    static final String LOG_TAG = "RILB";
-
     //***** Instance Variables
     protected Context mContext;
     protected RadioState mState = RadioState.RADIO_UNAVAILABLE;
@@ -107,10 +98,12 @@ public abstract class BaseCommands implements CommandsInterface {
 
     //***** CommandsInterface implementation
 
+    @Override
     public RadioState getRadioState() {
         return mState;
     }
 
+    @Override
     public void registerForRadioStateChanged(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
 
@@ -120,12 +113,14 @@ public abstract class BaseCommands implements CommandsInterface {
         }
     }
 
+    @Override
     public void unregisterForRadioStateChanged(Handler h) {
         synchronized (mStateMonitor) {
             mRadioStateChangedRegistrants.remove(h);
         }
     }
 
+    @Override
     public void registerForOn(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
 
@@ -137,6 +132,7 @@ public abstract class BaseCommands implements CommandsInterface {
             }
         }
     }
+    @Override
     public void unregisterForOn(Handler h) {
         synchronized (mStateMonitor) {
             mOnRegistrants.remove(h);
@@ -144,6 +140,7 @@ public abstract class BaseCommands implements CommandsInterface {
     }
 
 
+    @Override
     public void registerForAvailable(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
 
@@ -156,12 +153,14 @@ public abstract class BaseCommands implements CommandsInterface {
         }
     }
 
+    @Override
     public void unregisterForAvailable(Handler h) {
         synchronized(mStateMonitor) {
             mAvailRegistrants.remove(h);
         }
     }
 
+    @Override
     public void registerForNotAvailable(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
 
@@ -174,12 +173,14 @@ public abstract class BaseCommands implements CommandsInterface {
         }
     }
 
+    @Override
     public void unregisterForNotAvailable(Handler h) {
         synchronized (mStateMonitor) {
             mNotAvailRegistrants.remove(h);
         }
     }
 
+    @Override
     public void registerForOffOrNotAvailable(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
 
@@ -191,243 +192,300 @@ public abstract class BaseCommands implements CommandsInterface {
             }
         }
     }
+    @Override
     public void unregisterForOffOrNotAvailable(Handler h) {
         synchronized(mStateMonitor) {
             mOffOrNotAvailRegistrants.remove(h);
         }
     }
 
+    @Override
     public void registerForCallStateChanged(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
 
         mCallStateRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForCallStateChanged(Handler h) {
         mCallStateRegistrants.remove(h);
     }
 
+    @Override
     public void registerForVoiceNetworkStateChanged(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
 
         mVoiceNetworkStateRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForVoiceNetworkStateChanged(Handler h) {
         mVoiceNetworkStateRegistrants.remove(h);
     }
 
+    @Override
     public void registerForDataNetworkStateChanged(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
 
         mDataNetworkStateRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForDataNetworkStateChanged(Handler h) {
         mDataNetworkStateRegistrants.remove(h);
     }
 
+    @Override
     public void registerForVoiceRadioTechChanged(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mVoiceRadioTechChangedRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForVoiceRadioTechChanged(Handler h) {
         mVoiceRadioTechChangedRegistrants.remove(h);
     }
 
+    @Override
     public void registerForIccStatusChanged(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mIccStatusChangedRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForIccStatusChanged(Handler h) {
         mIccStatusChangedRegistrants.remove(h);
     }
 
+    @Override
     public void setOnNewGsmSms(Handler h, int what, Object obj) {
         mGsmSmsRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnNewGsmSms(Handler h) {
         mGsmSmsRegistrant.clear();
     }
 
+    @Override
     public void setOnNewCdmaSms(Handler h, int what, Object obj) {
         mCdmaSmsRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnNewCdmaSms(Handler h) {
         mCdmaSmsRegistrant.clear();
     }
 
+    @Override
     public void setOnNewGsmBroadcastSms(Handler h, int what, Object obj) {
         mGsmBroadcastSmsRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnNewGsmBroadcastSms(Handler h) {
         mGsmBroadcastSmsRegistrant.clear();
     }
 
+    @Override
     public void setOnSmsOnSim(Handler h, int what, Object obj) {
         mSmsOnSimRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnSmsOnSim(Handler h) {
         mSmsOnSimRegistrant.clear();
     }
 
+    @Override
     public void setOnSmsStatus(Handler h, int what, Object obj) {
         mSmsStatusRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnSmsStatus(Handler h) {
         mSmsStatusRegistrant.clear();
     }
 
+    @Override
     public void setOnSignalStrengthUpdate(Handler h, int what, Object obj) {
         mSignalStrengthRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnSignalStrengthUpdate(Handler h) {
         mSignalStrengthRegistrant.clear();
     }
 
+    @Override
     public void setOnNITZTime(Handler h, int what, Object obj) {
         mNITZTimeRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnNITZTime(Handler h) {
         mNITZTimeRegistrant.clear();
     }
 
+    @Override
     public void setOnUSSD(Handler h, int what, Object obj) {
         mUSSDRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnUSSD(Handler h) {
         mUSSDRegistrant.clear();
     }
 
+    @Override
     public void setOnSuppServiceNotification(Handler h, int what, Object obj) {
         mSsnRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnSuppServiceNotification(Handler h) {
         mSsnRegistrant.clear();
     }
 
+    @Override
     public void setOnCatSessionEnd(Handler h, int what, Object obj) {
         mCatSessionEndRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnCatSessionEnd(Handler h) {
         mCatSessionEndRegistrant.clear();
     }
 
+    @Override
     public void setOnCatProactiveCmd(Handler h, int what, Object obj) {
         mCatProCmdRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnCatProactiveCmd(Handler h) {
         mCatProCmdRegistrant.clear();
     }
 
+    @Override
     public void setOnCatEvent(Handler h, int what, Object obj) {
         mCatEventRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnCatEvent(Handler h) {
         mCatEventRegistrant.clear();
     }
 
+    @Override
     public void setOnCatCallSetUp(Handler h, int what, Object obj) {
         mCatCallSetUpRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnCatCallSetUp(Handler h) {
         mCatCallSetUpRegistrant.clear();
     }
 
+    @Override
     public void setOnIccSmsFull(Handler h, int what, Object obj) {
         mIccSmsFullRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnIccSmsFull(Handler h) {
         mIccSmsFullRegistrant.clear();
     }
 
+    @Override
     public void registerForIccRefresh(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mIccRefreshRegistrants.add(r);
     }
+    @Override
     public void setOnIccRefresh(Handler h, int what, Object obj) {
         registerForIccRefresh(h, what, obj);
     }
 
+    @Override
     public void setEmergencyCallbackMode(Handler h, int what, Object obj) {
         mEmergencyCallbackModeRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unregisterForIccRefresh(Handler h) {
         mIccRefreshRegistrants.remove(h);
     }
+    @Override
     public void unsetOnIccRefresh(Handler h) {
         unregisterForIccRefresh(h);
     }
 
+    @Override
     public void setOnCallRing(Handler h, int what, Object obj) {
         mRingRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnCallRing(Handler h) {
         mRingRegistrant.clear();
     }
 
+    @Override
     public void registerForInCallVoicePrivacyOn(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mVoicePrivacyOnRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForInCallVoicePrivacyOn(Handler h){
         mVoicePrivacyOnRegistrants.remove(h);
     }
 
+    @Override
     public void registerForInCallVoicePrivacyOff(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mVoicePrivacyOffRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForInCallVoicePrivacyOff(Handler h){
         mVoicePrivacyOffRegistrants.remove(h);
     }
 
+    @Override
     public void setOnRestrictedStateChanged(Handler h, int what, Object obj) {
         mRestrictedStateRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unSetOnRestrictedStateChanged(Handler h) {
         mRestrictedStateRegistrant.clear();
     }
 
+    @Override
     public void registerForDisplayInfo(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mDisplayInfoRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForDisplayInfo(Handler h) {
         mDisplayInfoRegistrants.remove(h);
     }
 
+    @Override
     public void registerForCallWaitingInfo(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mCallWaitingInfoRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForCallWaitingInfo(Handler h) {
         mCallWaitingInfoRegistrants.remove(h);
     }
 
+    @Override
     public void registerForSignalInfo(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mSignalInfoRegistrants.add(r);
@@ -441,78 +499,95 @@ public abstract class BaseCommands implements CommandsInterface {
         mUnsolOemHookRawRegistrant.clear();
     }
 
+    @Override
     public void unregisterForSignalInfo(Handler h) {
         mSignalInfoRegistrants.remove(h);
     }
 
+    @Override
     public void registerForCdmaOtaProvision(Handler h,int what, Object obj){
         Registrant r = new Registrant (h, what, obj);
         mOtaProvisionRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForCdmaOtaProvision(Handler h){
         mOtaProvisionRegistrants.remove(h);
     }
 
+    @Override
     public void registerForNumberInfo(Handler h,int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mNumberInfoRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForNumberInfo(Handler h){
         mNumberInfoRegistrants.remove(h);
     }
 
-     public void registerForRedirectedNumberInfo(Handler h,int what, Object obj) {
+     @Override
+    public void registerForRedirectedNumberInfo(Handler h,int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mRedirNumInfoRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForRedirectedNumberInfo(Handler h) {
         mRedirNumInfoRegistrants.remove(h);
     }
 
+    @Override
     public void registerForLineControlInfo(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mLineControlInfoRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForLineControlInfo(Handler h) {
         mLineControlInfoRegistrants.remove(h);
     }
 
+    @Override
     public void registerFoT53ClirlInfo(Handler h,int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mT53ClirInfoRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForT53ClirInfo(Handler h) {
         mT53ClirInfoRegistrants.remove(h);
     }
 
+    @Override
     public void registerForT53AudioControlInfo(Handler h,int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mT53AudCntrlInfoRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForT53AudioControlInfo(Handler h) {
         mT53AudCntrlInfoRegistrants.remove(h);
     }
 
+    @Override
     public void registerForRingbackTone(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mRingbackToneRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForRingbackTone(Handler h) {
         mRingbackToneRegistrants.remove(h);
     }
 
+    @Override
     public void registerForResendIncallMute(Handler h, int what, Object obj) {
         Registrant r = new Registrant (h, what, obj);
         mResendIncallMuteRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForResendIncallMute(Handler h) {
         mResendIncallMuteRegistrants.remove(h);
     }
@@ -555,11 +630,9 @@ public abstract class BaseCommands implements CommandsInterface {
      */
     @Override
     public void registerForRilConnected(Handler h, int what, Object obj) {
-        Rlog.d(LOG_TAG, "registerForRilConnected h=" + h + " w=" + what);
         Registrant r = new Registrant (h, what, obj);
         mRilConnectedRegistrants.add(r);
         if (mRilVersion != -1) {
-            Rlog.d(LOG_TAG, "Notifying: ril connected mRilVersion=" + mRilVersion);
             r.notifyRegistrant(new AsyncResult(null, new Integer(mRilVersion), null));
         }
     }
@@ -591,11 +664,6 @@ public abstract class BaseCommands implements CommandsInterface {
         RadioState oldState;
 
         synchronized (mStateMonitor) {
-            if (false) {
-                Rlog.v(LOG_TAG, "setRadioState old: " + mState
-                    + " new " + newState);
-            }
-
             oldState = mState;
             mState = newState;
 
@@ -607,25 +675,21 @@ public abstract class BaseCommands implements CommandsInterface {
             mRadioStateChangedRegistrants.notifyRegistrants();
 
             if (mState.isAvailable() && !oldState.isAvailable()) {
-                Rlog.d(LOG_TAG,"Notifying: radio available");
                 mAvailRegistrants.notifyRegistrants();
                 onRadioAvailable();
             }
 
             if (!mState.isAvailable() && oldState.isAvailable()) {
-                Rlog.d(LOG_TAG,"Notifying: radio not available");
                 mNotAvailRegistrants.notifyRegistrants();
             }
 
             if (mState.isOn() && !oldState.isOn()) {
-                Rlog.d(LOG_TAG,"Notifying: Radio On");
                 mOnRegistrants.notifyRegistrants();
             }
 
             if ((!mState.isOn() || !mState.isAvailable())
                 && !((!oldState.isOn() || !oldState.isAvailable()))
             ) {
-                Rlog.d(LOG_TAG,"Notifying: radio off or not available");
                 mOffOrNotAvailRegistrants.notifyRegistrants();
             }
         }
index 2127258..931bd17 100644 (file)
@@ -16,6 +16,7 @@
 
 package com.android.internal.telephony;
 
+import java.util.ArrayList;
 import java.util.List;
 
 import android.telephony.Rlog;
@@ -47,6 +48,7 @@ public abstract class Call {
 
     public State state = State.IDLE;
 
+    public ArrayList<Connection> connections = new ArrayList<Connection>();
 
     // Flag to indicate if the current calling/caller information
     // is accurate. If false the information is known to be accurate.
@@ -85,7 +87,7 @@ public abstract class Call {
      * @return true if the call contains one or more connections
      */
     public boolean hasConnections() {
-        List connections = getConnections();
+        List<Connection> connections = getConnections();
 
         if (connections == null) {
             return false;
@@ -118,7 +120,7 @@ public abstract class Call {
      */
     public Connection
     getEarliestConnection() {
-        List l;
+        List<Connection> l;
         long time = Long.MAX_VALUE;
         Connection c;
         Connection earliest = null;
@@ -130,7 +132,7 @@ public abstract class Call {
         }
 
         for (int i = 0, s = l.size() ; i < s ; i++) {
-            c = (Connection) l.get(i);
+            c = l.get(i);
             long t;
 
             t = c.getCreateTime();
@@ -146,7 +148,7 @@ public abstract class Call {
 
     public long
     getEarliestCreateTime() {
-        List l;
+        List<Connection> l;
         long time = Long.MAX_VALUE;
 
         l = getConnections();
@@ -156,7 +158,7 @@ public abstract class Call {
         }
 
         for (int i = 0, s = l.size() ; i < s ; i++) {
-            Connection c = (Connection) l.get(i);
+            Connection c = l.get(i);
             long t;
 
             t = c.getCreateTime();
@@ -170,14 +172,14 @@ public abstract class Call {
     public long
     getEarliestConnectTime() {
         long time = Long.MAX_VALUE;
-        List l = getConnections();
+        List<Connection> l = getConnections();
 
         if (l.size() == 0) {
             return 0;
         }
 
         for (int i = 0, s = l.size() ; i < s ; i++) {
-            Connection c = (Connection) l.get(i);
+            Connection c = l.get(i);
             long t;
 
             t = c.getConnectTime();
@@ -205,7 +207,7 @@ public abstract class Call {
      */
     public Connection
     getLatestConnection() {
-        List l = getConnections();
+        List<Connection> l = getConnections();
         if (l.size() == 0) {
             return null;
         }
@@ -213,7 +215,7 @@ public abstract class Call {
         long time = 0;
         Connection latest = null;
         for (int i = 0, s = l.size() ; i < s ; i++) {
-            Connection c = (Connection) l.get(i);
+            Connection c = l.get(i);
             long t = c.getCreateTime();
 
             if (t > time) {
index 8b853b0..320e786 100644 (file)
@@ -31,6 +31,7 @@ public class CallForwardInfo {
     public String          number;      /* "number" from TS 27.007 7.11 */
     public int             timeSeconds; /* for CF no reply only */
 
+    @Override
     public String toString() {
         return super.toString() + (status == 0 ? " not active " : " active ")
             + " reason: " + reason
index 1c1799f..49e6488 100644 (file)
@@ -1100,8 +1100,8 @@ public final class CallManager {
 
     /**
      * Register for getting notifications for change in the Call State {@link Call.State}
-     * This is called PreciseCallState because the call state is more precise than the
-     * {@link Phone.State} which can be obtained using the {@link PhoneStateListener}
+     * This is called PreciseCallState because the call state is more precise than what
+     * can be obtained using the {@link PhoneStateListener}
      *
      * Resulting events will have an AsyncResult in <code>Message.obj</code>.
      * AsyncResult.userData will be set to the obj argument here.
index efe53c9..57de316 100644 (file)
@@ -21,8 +21,6 @@ import android.os.Handler;
 import android.os.Message;
 import android.os.SystemProperties;
 import android.text.TextUtils;
-import android.telephony.Rlog;
-
 import com.android.internal.telephony.CommandException;
 
 import java.io.FileDescriptor;
@@ -139,7 +137,7 @@ public abstract class CallTracker extends Handler {
      * To test Dial 112 take call then hang up on MO device to enter ECM
      * see RIL#processSolicited RIL_REQUEST_HANGUP_FOREGROUND_RESUME_BACKGROUND
      *
-     * @param number to test if it should be remapped
+     * @param dialString to test if it should be remapped
      * @return the same number or the remapped number.
      */
     protected String checkForTestEmergencyNumber(String dialString) {
@@ -165,6 +163,7 @@ public abstract class CallTracker extends Handler {
     }
 
     //***** Overridden from Handler
+    @Override
     public abstract void handleMessage (Message msg);
     public abstract void registerForVoiceCallStarted(Handler h, int what, Object obj);
     public abstract void unregisterForVoiceCallStarted(Handler h);
index ab52cef..0ea8035 100644 (file)
@@ -22,7 +22,6 @@ import com.android.internal.telephony.uicc.IccCardStatus;
 
 import android.os.Message;
 import android.os.Handler;
-import android.telephony.Rlog;
 
 /**
  * {@hide}
@@ -1558,8 +1557,8 @@ public interface CommandsInterface {
      * is a tri-state return value as for a period of time
      * the mode may be unknown.
      *
-     * @return {@link Phone#LTE_ON_CDMA_UNKNOWN}, {@link Phone#LTE_ON_CDMA_FALSE}
-     * or {@link Phone#LTE_ON_CDMA_TRUE}
+     * @return {@link PhoneConstants#LTE_ON_CDMA_UNKNOWN}, {@link PhoneConstants#LTE_ON_CDMA_FALSE}
+     * or {@link PhoneConstants#LTE_ON_CDMA_TRUE}
      */
     public int getLteOnCdmaMode();
 
index 8e10af5..5bfea1c 100644 (file)
@@ -28,7 +28,7 @@ public abstract class Connection {
     protected String cnapName;
     protected int cnapNamePresentation  = PhoneConstants.PRESENTATION_ALLOWED;
 
-    private static String LOG_TAG = "TelephonyConnection";
+    private static String LOG_TAG = "Connection";
 
     public enum DisconnectCause {
         NOT_DISCONNECTED,               /* has not yet disconnected */
@@ -106,7 +106,7 @@ public abstract class Connection {
 
     public int getCnapNamePresentation() {
        return cnapNamePresentation;
-    };
+    }
 
     /**
      * @return Call that owns this Connection, or null if none
index 45e39a4..12ebc1a 100644 (file)
@@ -24,7 +24,7 @@ import android.net.RouteInfo;
 import android.os.SystemProperties;
 import android.telephony.Rlog;
 
-import com.android.internal.telephony.dataconnection.DataConnection.FailCause;
+import com.android.internal.telephony.dataconnection.DataConnectionBase.FailCause;
 
 import java.net.Inet4Address;
 import java.net.InetAddress;
@@ -36,7 +36,7 @@ import java.net.UnknownHostException;
  */
 public class DataCallState {
     private final boolean DBG = true;
-    private final String LOG_TAG = "GSM";
+    private final String LOG_TAG = "DataCallState";
 
     public int version = 0;
     public int status = 0;
index 8f4ae84..636ae39 100644 (file)
@@ -24,7 +24,6 @@ import android.os.ServiceManager;
 import android.telephony.CellInfo;
 import android.telephony.ServiceState;
 import android.telephony.TelephonyManager;
-import android.telephony.Rlog;
 
 import com.android.internal.telephony.ITelephonyRegistry;
 
@@ -35,8 +34,6 @@ import java.util.List;
  */
 public class DefaultPhoneNotifier implements PhoneNotifier {
 
-    static final String LOG_TAG = "GSM";
-    private static final boolean DBG = true;
     private ITelephonyRegistry mRegistry;
 
     /*package*/
@@ -45,6 +42,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
                     "telephony.registry"));
     }
 
+    @Override
     public void notifyPhoneState(Phone sender) {
         Call ringingCall = sender.getRingingCall();
         String incomingNumber = "";
@@ -58,6 +56,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
         }
     }
 
+    @Override
     public void notifyServiceState(Phone sender) {
         ServiceState ss = sender.getServiceState();
         if (ss == null) {
@@ -71,6 +70,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
         }
     }
 
+    @Override
     public void notifySignalStrength(Phone sender) {
         try {
             mRegistry.notifySignalStrength(sender.getSignalStrength());
@@ -79,6 +79,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
         }
     }
 
+    @Override
     public void notifyMessageWaitingChanged(Phone sender) {
         try {
             mRegistry.notifyMessageWaitingChanged(sender.getMessageWaitingIndicator());
@@ -87,6 +88,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
         }
     }
 
+    @Override
     public void notifyCallForwardingChanged(Phone sender) {
         try {
             mRegistry.notifyCallForwardingChanged(sender.getCallForwardingIndicator());
@@ -95,6 +97,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
         }
     }
 
+    @Override
     public void notifyDataActivity(Phone sender) {
         try {
             mRegistry.notifyDataActivity(convertDataActivityState(sender.getDataActivityState()));
@@ -103,6 +106,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
         }
     }
 
+    @Override
     public void notifyDataConnection(Phone sender, String reason, String apnType,
             PhoneConstants.DataState state) {
         doNotifyDataConnection(sender, reason, apnType, state);
@@ -141,6 +145,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
         }
     }
 
+    @Override
     public void notifyDataConnectionFailed(Phone sender, String reason, String apnType) {
         try {
             mRegistry.notifyDataConnectionFailed(reason, apnType);
@@ -149,6 +154,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
         }
     }
 
+    @Override
     public void notifyCellLocation(Phone sender) {
         Bundle data = new Bundle();
         sender.getCellLocation().fillInNotifierBundle(data);
@@ -159,6 +165,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
         }
     }
 
+    @Override
     public void notifyCellInfo(Phone sender, List<CellInfo> cellInfo) {
         try {
             mRegistry.notifyCellInfo(cellInfo);
@@ -167,6 +174,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
         }
     }
 
+    @Override
     public void notifyOtaspChanged(Phone sender, int otaspMode) {
         try {
             mRegistry.notifyOtaspChanged(otaspMode);
@@ -175,13 +183,9 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
         }
     }
 
-    private void log(String s) {
-        Rlog.d(LOG_TAG, "[PhoneNotifier] " + s);
-    }
-
     /**
-     * Convert the {@link State} enum into the TelephonyManager.CALL_STATE_* constants
-     * for the public API.
+     * Convert the {@link PhoneConstants.State} enum into the TelephonyManager.CALL_STATE_*
+     * constants for the public API.
      */
     public static int convertCallState(PhoneConstants.State state) {
         switch (state) {
@@ -195,8 +199,8 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
     }
 
     /**
-     * Convert the TelephonyManager.CALL_STATE_* constants into the {@link State} enum
-     * for the public API.
+     * Convert the TelephonyManager.CALL_STATE_* constants into the
+     * {@link PhoneConstants.State} enum for the public API.
      */
     public static PhoneConstants.State convertCallState(int state) {
         switch (state) {
@@ -210,7 +214,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
     }
 
     /**
-     * Convert the {@link DataState} enum into the TelephonyManager.DATA_* constants
+     * Convert the {@link PhoneConstants.DataState} enum into the TelephonyManager.DATA_* constants
      * for the public API.
      */
     public static int convertDataState(PhoneConstants.DataState state) {
@@ -227,7 +231,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
     }
 
     /**
-     * Convert the TelephonyManager.DATA_* constants into {@link DataState} enum
+     * Convert the TelephonyManager.DATA_* constants into {@link PhoneConstants.DataState} enum
      * for the public API.
      */
     public static PhoneConstants.DataState convertDataState(int state) {
@@ -244,7 +248,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
     }
 
     /**
-     * Convert the {@link DataState} enum into the TelephonyManager.DATA_* constants
+     * Convert the {@link Phone.DataActivityState} enum into the TelephonyManager.DATA_* constants
      * for the public API.
      */
     public static int convertDataActivityState(Phone.DataActivityState state) {
@@ -263,7 +267,7 @@ public class DefaultPhoneNotifier implements PhoneNotifier {
     }
 
     /**
-     * Convert the TelephonyManager.DATA_* constants into the {@link DataState} enum
+     * Convert the TelephonyManager.DATA_* constants into the {@link Phone.DataActivityState} enum
      * for the public API.
      */
     public static Phone.DataActivityState convertDataActivityState(int state) {
index c76be5f..a923d8f 100644 (file)
@@ -15,7 +15,7 @@
  */
 
 package com.android.internal.telephony;
-//import com.android.internal.telephony.*;
+
 import android.telephony.Rlog;
 import java.lang.Comparable;
 import android.telephony.PhoneNumberUtils;
@@ -23,8 +23,8 @@ import android.telephony.PhoneNumberUtils;
 /**
  * {@hide}
  */
-public class DriverCall implements Comparable {
-    static final String LOG_TAG = "RILB";
+public class DriverCall implements Comparable<DriverCall> {
+    static final String LOG_TAG = "DriverCall";
 
     public enum State {
         ACTIVE,
@@ -101,6 +101,7 @@ public class DriverCall implements Comparable {
     DriverCall() {
     }
 
+    @Override
     public String
     toString() {
         return "id=" + index + ","
@@ -145,11 +146,9 @@ public class DriverCall implements Comparable {
     //***** Comparable Implementation
 
     /** For sorting by index */
+    @Override
     public int
-    compareTo (Object o) {
-        DriverCall dc;
-
-        dc = (DriverCall)o;
+    compareTo(DriverCall dc) {
 
         if (index < dc.index) {
             return -1;
index 2bab4bb..b0ecda5 100644 (file)
@@ -25,7 +25,6 @@ import android.os.ServiceManager;
 
 import com.android.internal.telephony.uicc.AdnRecord;
 import com.android.internal.telephony.uicc.AdnRecordCache;
-import com.android.internal.telephony.uicc.IccCardApplicationStatus;
 import com.android.internal.telephony.uicc.IccCardApplicationStatus.AppType;
 import com.android.internal.telephony.uicc.IccConstants;
 import com.android.internal.telephony.uicc.IccRecords;
@@ -156,6 +155,7 @@ public abstract class IccPhoneBookInterfaceManager extends IIccPhoneBook.Stub {
      * @param pin2 required to update EF_FDN, otherwise must be null
      * @return true for success
      */
+    @Override
     public boolean
     updateAdnRecordsInEfBySearch (int efid,
             String oldTag, String oldPhoneNumber,
@@ -206,6 +206,7 @@ public abstract class IccPhoneBookInterfaceManager extends IIccPhoneBook.Stub {
      * @param pin2 required to update EF_FDN, otherwise must be null
      * @return true for success
      */
+    @Override
     public boolean
     updateAdnRecordsInEfByIndex(int efid, String newTag,
             String newPhoneNumber, int index, String pin2) {
@@ -241,6 +242,7 @@ public abstract class IccPhoneBookInterfaceManager extends IIccPhoneBook.Stub {
      *            recordSizes[1]  is the total length of the EF file
      *            recordSizes[2]  is the number of records in the EF file
      */
+    @Override
     public abstract int[] getAdnRecordsSize(int efid);
 
     /**
@@ -252,6 +254,7 @@ public abstract class IccPhoneBookInterfaceManager extends IIccPhoneBook.Stub {
      * @param efid the EF id of a ADN-like ICC
      * @return List of AdnRecord
      */
+    @Override
     public List<AdnRecord> getAdnRecordsInEf(int efid) {
 
         if (phone.getContext().checkCallingOrSelfPermission(
index 6810fb0..7aa630c 100644 (file)
 
 package com.android.internal.telephony;
 
-import android.content.pm.PackageManager;
-import android.os.AsyncResult;
-import android.os.Handler;
-import android.os.Looper;
-import android.os.Message;
 import android.os.ServiceManager;
-import android.telephony.PhoneNumberUtils;
-import android.telephony.Rlog;
-
 import com.android.internal.telephony.uicc.AdnRecord;
 
 
-import java.util.ArrayList;
 import java.util.List;
 
 
@@ -52,27 +43,31 @@ public class IccPhoneBookInterfaceManagerProxy extends IIccPhoneBook.Stub {
         this.mIccPhoneBookInterfaceManager = iccPhoneBookInterfaceManager;
     }
 
+    @Override
     public boolean
     updateAdnRecordsInEfBySearch (int efid,
             String oldTag, String oldPhoneNumber,
             String newTag, String newPhoneNumber,
-            String pin2) throws android.os.RemoteException {
+            String pin2) {
         return mIccPhoneBookInterfaceManager.updateAdnRecordsInEfBySearch(
                 efid, oldTag, oldPhoneNumber, newTag, newPhoneNumber, pin2);
     }
 
+    @Override
     public boolean
     updateAdnRecordsInEfByIndex(int efid, String newTag,
-            String newPhoneNumber, int index, String pin2) throws android.os.RemoteException {
+            String newPhoneNumber, int index, String pin2) {
         return mIccPhoneBookInterfaceManager.updateAdnRecordsInEfByIndex(efid,
                 newTag, newPhoneNumber, index, pin2);
     }
 
-    public int[] getAdnRecordsSize(int efid) throws android.os.RemoteException {
+    @Override
+    public int[] getAdnRecordsSize(int efid) {
         return mIccPhoneBookInterfaceManager.getAdnRecordsSize(efid);
     }
 
-    public List<AdnRecord> getAdnRecordsInEf(int efid) throws android.os.RemoteException {
+    @Override
+    public List<AdnRecord> getAdnRecordsInEf(int efid) {
         return mIccPhoneBookInterfaceManager.getAdnRecordsInEf(efid);
     }
 }
index e01b00e..03ed244 100644 (file)
@@ -44,7 +44,7 @@ import static android.telephony.SmsManager.STATUS_ON_ICC_UNREAD;
  * access Sms in Icc.
  */
 public abstract class IccSmsInterfaceManager extends ISms.Stub {
-    static final String LOG_TAG = "RIL_IccSms";
+    static final String LOG_TAG = "IccSmsInterfaceManager";
     static final boolean DBG = true;
 
     protected final Object mLock = new Object();
@@ -158,6 +158,7 @@ public abstract class IccSmsInterfaceManager extends ISms.Stub {
      * @return success or not
      *
      */
+    @Override
     public boolean
     updateMessageOnIccEf(String callingPackage, int index, int status, byte[] pdu) {
         if (DBG) log("updateMessageOnIccEf: index=" + index +
@@ -208,6 +209,7 @@ public abstract class IccSmsInterfaceManager extends ISms.Stub {
      * @return success or not
      *
      */
+    @Override
     public boolean copyMessageToIccEf(String callingPackage, int status, byte[] pdu, byte[] smsc) {
         //NOTE smsc not used in RUIM
         if (DBG) log("copyMessageToIccEf: status=" + status + " ==> " +
@@ -239,6 +241,7 @@ public abstract class IccSmsInterfaceManager extends ISms.Stub {
      *
      * @return list of SmsRawData of all sms on Icc
      */
+    @Override
     public List<SmsRawData> getAllMessagesFromIccEf(String callingPackage) {
         if (DBG) log("getAllMessagesFromEF");
 
@@ -297,6 +300,7 @@ public abstract class IccSmsInterfaceManager extends ISms.Stub {
      *  broadcast when the message is delivered to the recipient.  The
      *  raw pdu of the status report is in the extended data ("pdu").
      */
+    @Override
     public void sendData(String callingPackage, String destAddr, String scAddr, int destPort,
             byte[] data, PendingIntent sentIntent, PendingIntent deliveryIntent) {
         mPhone.getContext().enforceCallingPermission(
@@ -338,6 +342,7 @@ public abstract class IccSmsInterfaceManager extends ISms.Stub {
      *  broadcast when the message is delivered to the recipient.  The
      *  raw pdu of the status report is in the extended data ("pdu").
      */
+    @Override
     public void sendText(String callingPackage, String destAddr, String scAddr,
             String text, PendingIntent sentIntent, PendingIntent deliveryIntent) {
         mPhone.getContext().enforceCallingPermission(
@@ -380,6 +385,7 @@ public abstract class IccSmsInterfaceManager extends ISms.Stub {
      *   to the recipient.  The raw pdu of the status report is in the
      *   extended data ("pdu").
      */
+    @Override
     public void sendMultipartText(String callingPackage, String destAddr, String scAddr,
             List<String> parts, List<PendingIntent> sentIntents,
             List<PendingIntent> deliveryIntents) {
@@ -401,10 +407,12 @@ public abstract class IccSmsInterfaceManager extends ISms.Stub {
                 (ArrayList<PendingIntent>) sentIntents, (ArrayList<PendingIntent>) deliveryIntents);
     }
 
+    @Override
     public int getPremiumSmsPermission(String packageName) {
         return mDispatcher.getPremiumSmsPermission(packageName);
     }
 
+    @Override
     public void setPremiumSmsPermission(String packageName, int permission) {
         mDispatcher.setPremiumSmsPermission(packageName, permission);
     }
index 5b96455..ed55d99 100644 (file)
@@ -36,63 +36,73 @@ public class IccSmsInterfaceManagerProxy extends ISms.Stub {
         this.mIccSmsInterfaceManager = iccSmsInterfaceManager;
     }
 
+    @Override
     public boolean
-    updateMessageOnIccEf(String callingPackage, int index, int status, byte[] pdu)
-            throws android.os.RemoteException {
+    updateMessageOnIccEf(String callingPackage, int index, int status, byte[] pdu) {
          return mIccSmsInterfaceManager.updateMessageOnIccEf(callingPackage, index, status, pdu);
     }
 
+    @Override
     public boolean copyMessageToIccEf(String callingPackage, int status, byte[] pdu,
-            byte[] smsc) throws android.os.RemoteException {
+            byte[] smsc) {
         return mIccSmsInterfaceManager.copyMessageToIccEf(callingPackage, status, pdu, smsc);
     }
 
-    public List<SmsRawData> getAllMessagesFromIccEf(String callingPackage)
-            throws android.os.RemoteException {
+    @Override
+    public List<SmsRawData> getAllMessagesFromIccEf(String callingPackage) {
         return mIccSmsInterfaceManager.getAllMessagesFromIccEf(callingPackage);
     }
 
+    @Override
     public void sendData(String callingPackage, String destAddr, String scAddr, int destPort,
             byte[] data, PendingIntent sentIntent, PendingIntent deliveryIntent) {
         mIccSmsInterfaceManager.sendData(callingPackage, destAddr, scAddr, destPort, data,
                 sentIntent, deliveryIntent);
     }
 
+    @Override
     public void sendText(String callingPackage, String destAddr, String scAddr,
             String text, PendingIntent sentIntent, PendingIntent deliveryIntent) {
         mIccSmsInterfaceManager.sendText(callingPackage, destAddr, scAddr, text, sentIntent,
                 deliveryIntent);
     }
 
+    @Override
     public void sendMultipartText(String callingPackage, String destAddr, String scAddr,
             List<String> parts, List<PendingIntent> sentIntents,
-            List<PendingIntent> deliveryIntents) throws android.os.RemoteException {
+            List<PendingIntent> deliveryIntents) {
         mIccSmsInterfaceManager.sendMultipartText(callingPackage, destAddr, scAddr,
                 parts, sentIntents, deliveryIntents);
     }
 
+    @Override
     public boolean enableCellBroadcast(int messageIdentifier) throws android.os.RemoteException {
         return mIccSmsInterfaceManager.enableCellBroadcast(messageIdentifier);
     }
 
+    @Override
     public boolean disableCellBroadcast(int messageIdentifier) throws android.os.RemoteException {
         return mIccSmsInterfaceManager.disableCellBroadcast(messageIdentifier);
     }
 
+    @Override
     public boolean enableCellBroadcastRange(int startMessageId, int endMessageId)
             throws android.os.RemoteException {
         return mIccSmsInterfaceManager.enableCellBroadcastRange(startMessageId, endMessageId);
     }
 
+    @Override
     public boolean disableCellBroadcastRange(int startMessageId, int endMessageId)
             throws android.os.RemoteException {
         return mIccSmsInterfaceManager.disableCellBroadcastRange(startMessageId, endMessageId);
     }
 
+    @Override
     public int getPremiumSmsPermission(String packageName) {
         return mIccSmsInterfaceManager.getPremiumSmsPermission(packageName);
     }
 
+    @Override
     public void setPremiumSmsPermission(String packageName, int permission) {
         mIccSmsInterfaceManager.setPremiumSmsPermission(packageName, permission);
     }
index 795740c..2efb3ee 100644 (file)
@@ -24,8 +24,6 @@ import android.telephony.Rlog;
 
 import com.android.internal.telephony.GsmAlphabet;
 import java.io.UnsupportedEncodingException;
-import java.nio.ByteBuffer;
-import java.nio.charset.Charset;
 
 /**
  * Various methods, useful for dealing with SIM data.
@@ -118,7 +116,7 @@ public class IccUtils {
     }
 
     /**
-     * Decodes a CDMA style BCD byte like {@link gsmBcdByteToInt}, but
+     * Decodes a CDMA style BCD byte like {@link #gsmBcdByteToInt}, but
      * opposite nibble format. The least significant BCD digit
      * is in the least significant nibble and the most significant
      * is in the most significant nibble.
@@ -406,7 +404,7 @@ public class IccUtils {
                 bitIndex = 7;
             }
             pixels[pixelIndex++] = bitToRGB((currentByte >> bitIndex-- ) & 0x01);
-        };
+        }
 
         if (pixelIndex != numOfPixels) {
             Rlog.e(LOG_TAG, "parse end and size error");
index 7d5a4e7..aee471a 100644 (file)
@@ -63,6 +63,7 @@ public final class MccTable
         }
 
 
+        @Override
         public int compareTo(MccEntry o)
         {
             return mcc - o.mcc;
index 1999cb3..336d9af 100644 (file)
@@ -97,6 +97,7 @@ public class OperatorInfo implements Parcelable {
     }
 
 
+    @Override
     public String toString() {
         return "OperatorInfo " + operatorAlphaLong
                 + "/" + operatorAlphaShort
@@ -112,6 +113,7 @@ public class OperatorInfo implements Parcelable {
      * NetworkQueryService to fix 1128695.
      */
 
+    @Override
     public int describeContents() {
         return 0;
     }
@@ -120,6 +122,7 @@ public class OperatorInfo implements Parcelable {
      * Implement the Parcelable interface.
      * Method to serialize a OperatorInfo object.
      */
+    @Override
     public void writeToParcel(Parcel dest, int flags) {
         dest.writeString(operatorAlphaLong);
         dest.writeString(operatorAlphaShort);
@@ -133,6 +136,7 @@ public class OperatorInfo implements Parcelable {
      */
     public static final Creator<OperatorInfo> CREATOR =
         new Creator<OperatorInfo>() {
+            @Override
             public OperatorInfo createFromParcel(Parcel in) {
                 OperatorInfo opInfo = new OperatorInfo(
                         in.readString(), /*operatorAlphaLong*/
@@ -142,6 +146,7 @@ public class OperatorInfo implements Parcelable {
                 return opInfo;
             }
 
+            @Override
             public OperatorInfo[] newArray(int size) {
                 return new OperatorInfo[size];
             }
index 1404774..13c99ea 100644 (file)
@@ -60,11 +60,11 @@ public interface Phone {
          * </ul>
          */
         NONE, DATAIN, DATAOUT, DATAINANDOUT, DORMANT;
-    };
+    }
 
     enum SuppService {
       UNKNOWN, SWITCH, SEPARATE, TRANSFER, CONFERENCE, REJECT, HANGUP;
-    };
+    }
 
     // "Features" accessible through the connectivity manager
     static final String FEATURE_ENABLE_MMS = "enableMMS";
@@ -298,7 +298,7 @@ public interface Phone {
     /**
      * Register for getting notifications for change in the Call State {@link Call.State}
      * This is called PreciseCallState because the call state is more precise than the
-     * {@link Phone.State} which can be obtained using the {@link PhoneStateListener}
+     * {@link PhoneConstants.State} which can be obtained using the {@link PhoneStateListener}
      *
      * Resulting events will have an AsyncResult in <code>Message.obj</code>.
      * AsyncResult.userData will be set to the obj argument here.
@@ -541,7 +541,7 @@ public interface Phone {
     void registerForInCallVoicePrivacyOff(Handler h, int what, Object obj);
 
     /**
-     * Unegister for notifications when a sInCall VoicePrivacy is disabled
+     * Unregister for notifications when a sInCall VoicePrivacy is disabled
      *
      * @param h Handler to be removed from the registrant list.
      */
@@ -557,7 +557,7 @@ public interface Phone {
     void registerForCdmaOtaStatusChange(Handler h, int what, Object obj);
 
     /**
-     * Unegister for notifications when CDMA OTA Provision status change
+     * Unregister for notifications when CDMA OTA Provision status change
      * @param h Handler to be removed from the registrant list.
      */
     void unregisterForCdmaOtaStatusChange(Handler h);
@@ -1253,7 +1253,7 @@ public interface Phone {
 
     /**
      * If this is a simulated phone interface, returns a SimulatedRadioControl.
-     * @ return A SimulatedRadioControl if this is a simulated interface;
+     * @return SimulatedRadioControl if this is a simulated interface;
      * otherwise, null.
      */
     SimulatedRadioControl getSimulatedRadioControl();
@@ -1261,7 +1261,7 @@ public interface Phone {
     /**
      * Enables the specified APN type. Only works for "special" APN types,
      * i.e., not the default APN.
-     * @param type The desired APN type. Cannot be {@link #APN_TYPE_DEFAULT}.
+     * @param type The desired APN type. Cannot be {@link PhoneConstants#APN_TYPE_DEFAULT}.
      * @return <code>APN_ALREADY_ACTIVE</code> if the current APN
      * services the requested type.<br/>
      * <code>APN_TYPE_NOT_AVAILABLE</code> if the carrier does not
@@ -1276,10 +1276,10 @@ public interface Phone {
     /**
      * Disables the specified APN type, and switches back to the default APN,
      * if necessary. Switching to the default APN will not happen if default
-     * data traffic has been explicitly disabled via a call to {@link #disableDataConnectivity}.
+     * data traffic has been explicitly disabled via a call to ITelephony#disableDataConnectivity.
      * <p/>Only works for "special" APN types,
      * i.e., not the default APN.
-     * @param type The desired APN type. Cannot be {@link #APN_TYPE_DEFAULT}.
+     * @param type The desired APN type. Cannot be {@link PhoneConstants#APN_TYPE_DEFAULT}.
      * @return <code>APN_ALREADY_ACTIVE</code> if the default APN
      * is already active.<br/>
      * <code>APN_REQUEST_STARTED</code> if the request to switch to the default
@@ -1643,7 +1643,8 @@ public interface Phone {
      * is a tri-state return value as for a period of time
      * the mode may be unknown.
      *
-     * @return {@link #LTE_ON_CDMA_UNKNOWN}, {@link #LTE_ON_CDMA_FALSE} or {@link #LTE_ON_CDMA_TRUE}
+     * @return {@link PhoneConstants#LTE_ON_CDMA_UNKNOWN}, {@link PhoneConstants#LTE_ON_CDMA_FALSE}
+     * or {@link PhoneConstants#LTE_ON_CDMA_TRUE}
      */
     public int getLteOnCdmaMode();
 
index 6f24b49..7e8ef3f 100644 (file)
 
 package com.android.internal.telephony;
 
-import android.app.ActivityManagerNative;
-import android.app.IActivityManager;
 import android.content.Context;
-import android.content.res.Configuration;
 import android.content.SharedPreferences;
 import android.net.LinkCapabilities;
 import android.net.LinkProperties;
@@ -39,22 +36,18 @@ import android.text.TextUtils;
 import android.telephony.Rlog;
 
 import com.android.internal.R;
-import com.android.internal.telephony.dataconnection.DataConnectionTracker;
+import com.android.internal.telephony.dataconnection.DataConnectionTrackerBase;
 import com.android.internal.telephony.test.SimulatedRadioControl;
 import com.android.internal.telephony.uicc.IccCardApplicationStatus.AppType;
 import com.android.internal.telephony.uicc.IccFileHandler;
 import com.android.internal.telephony.uicc.IccRecords;
 import com.android.internal.telephony.uicc.IsimRecords;
-import com.android.internal.telephony.uicc.SIMRecords;
 import com.android.internal.telephony.uicc.UiccCardApplication;
 import com.android.internal.telephony.uicc.UiccController;
 import com.android.internal.telephony.uicc.UsimServiceTable;
-import com.android.internal.telephony.uicc.IccCardApplicationStatus.AppState;
-
 import java.io.FileDescriptor;
 import java.io.PrintWriter;
 import java.util.List;
-import java.util.Locale;
 import java.util.concurrent.atomic.AtomicReference;
 
 
@@ -71,9 +64,7 @@ import java.util.concurrent.atomic.AtomicReference;
  */
 
 public abstract class PhoneBase extends Handler implements Phone {
-    private static final String LOG_TAG = "PHONE";
-    private static final boolean LOCAL_DEBUG = true;
-
+    private static final String LOG_TAG = "PhoneBase";
     // Key used to read and write the saved network selection numeric value
     public static final String NETWORK_SELECTION_KEY = "network_selection_key";
     // Key used to read and write the saved network selection operator name
@@ -131,7 +122,7 @@ public abstract class PhoneBase extends Handler implements Phone {
     /* Instance Variables */
     public CommandsInterface mCM;
     boolean mDnsCheckDisabled;
-    public DataConnectionTracker mDataConnectionTracker;
+    public DataConnectionTrackerBase mDataConnectionTracker;
     boolean mDoesRilSendMultipleCallRing;
     int mCallRingContinueToken;
     int mCallRingDelay;
@@ -270,6 +261,7 @@ public abstract class PhoneBase extends Handler implements Phone {
         mUiccController.registerForIccChanged(this, EVENT_ICC_CHANGED, null);
     }
 
+    @Override
     public void dispose() {
         synchronized(PhoneProxy.lockForRadioTechnologyChange) {
             mCM.unSetOnCallRing(this);
@@ -283,6 +275,7 @@ public abstract class PhoneBase extends Handler implements Phone {
         }
     }
 
+    @Override
     public void removeReferences() {
         mSmsStorageMonitor = null;
         mSmsUsageMonitor = null;
@@ -338,6 +331,7 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public Context getContext() {
         return mContext;
     }
@@ -350,6 +344,7 @@ public abstract class PhoneBase extends Handler implements Phone {
      * Useful for lab testing environment.
      * @param b true disables the check, false enables.
      */
+    @Override
     public void disableDnsCheck(boolean b) {
         mDnsCheckDisabled = b;
         SharedPreferences sp = PreferenceManager.getDefaultSharedPreferences(getContext());
@@ -361,11 +356,13 @@ public abstract class PhoneBase extends Handler implements Phone {
     /**
      * Returns true if the DNS check is currently disabled.
      */
+    @Override
     public boolean isDnsCheckDisabled() {
         return mDnsCheckDisabled;
     }
 
     // Inherited documentation suffices.
+    @Override
     public void registerForPreciseCallStateChanged(Handler h, int what, Object obj) {
         checkCorrectThread(h);
 
@@ -373,6 +370,7 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public void unregisterForPreciseCallStateChanged(Handler h) {
         mPreciseCallStateRegistrants.remove(h);
     }
@@ -387,6 +385,7 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public void registerForUnknownConnection(Handler h, int what, Object obj) {
         checkCorrectThread(h);
 
@@ -394,11 +393,13 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public void unregisterForUnknownConnection(Handler h) {
         mUnknownConnectionRegistrants.remove(h);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void registerForNewRingingConnection(
             Handler h, int what, Object obj) {
         checkCorrectThread(h);
@@ -407,31 +408,37 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public void unregisterForNewRingingConnection(Handler h) {
         mNewRingingConnectionRegistrants.remove(h);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void registerForInCallVoicePrivacyOn(Handler h, int what, Object obj){
         mCM.registerForInCallVoicePrivacyOn(h,what,obj);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void unregisterForInCallVoicePrivacyOn(Handler h){
         mCM.unregisterForInCallVoicePrivacyOn(h);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void registerForInCallVoicePrivacyOff(Handler h, int what, Object obj){
         mCM.registerForInCallVoicePrivacyOff(h,what,obj);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void unregisterForInCallVoicePrivacyOff(Handler h){
         mCM.unregisterForInCallVoicePrivacyOff(h);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void registerForIncomingRing(
             Handler h, int what, Object obj) {
         checkCorrectThread(h);
@@ -440,11 +447,13 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public void unregisterForIncomingRing(Handler h) {
         mIncomingRingRegistrants.remove(h);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void registerForDisconnect(Handler h, int what, Object obj) {
         checkCorrectThread(h);
 
@@ -452,11 +461,13 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public void unregisterForDisconnect(Handler h) {
         mDisconnectRegistrants.remove(h);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void registerForSuppServiceFailed(Handler h, int what, Object obj) {
         checkCorrectThread(h);
 
@@ -464,11 +475,13 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public void unregisterForSuppServiceFailed(Handler h) {
         mSuppServiceFailedRegistrants.remove(h);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void registerForMmiInitiate(Handler h, int what, Object obj) {
         checkCorrectThread(h);
 
@@ -476,11 +489,13 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public void unregisterForMmiInitiate(Handler h) {
         mMmiRegistrants.remove(h);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void registerForMmiComplete(Handler h, int what, Object obj) {
         checkCorrectThread(h);
 
@@ -488,6 +503,7 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public void unregisterForMmiComplete(Handler h) {
         checkCorrectThread(h);
 
@@ -521,11 +537,13 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public void setUnitTestMode(boolean f) {
         mUnitTestMode = f;
     }
 
     // Inherited documentation suffices.
+    @Override
     public boolean getUnitTestMode() {
         return mUnitTestMode;
     }
@@ -542,6 +560,7 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public void registerForServiceStateChanged(
             Handler h, int what, Object obj) {
         checkCorrectThread(h);
@@ -550,30 +569,36 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public void unregisterForServiceStateChanged(Handler h) {
         mServiceStateRegistrants.remove(h);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void registerForRingbackTone(Handler h, int what, Object obj) {
         mCM.registerForRingbackTone(h,what,obj);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void unregisterForRingbackTone(Handler h) {
         mCM.unregisterForRingbackTone(h);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void registerForResendIncallMute(Handler h, int what, Object obj) {
         mCM.registerForResendIncallMute(h,what,obj);
     }
 
     // Inherited documentation suffices.
+    @Override
     public void unregisterForResendIncallMute(Handler h) {
         mCM.unregisterForResendIncallMute(h);
     }
 
+    @Override
     public void setEchoSuppressionEnabled(boolean enabled) {
         // no need for regular phone
     }
@@ -590,6 +615,7 @@ public abstract class PhoneBase extends Handler implements Phone {
     }
 
     // Inherited documentation suffices.
+    @Override
     public SimulatedRadioControl getSimulatedRadioControl() {
         return mSimulatedRadioControl;
     }
@@ -654,6 +680,7 @@ public abstract class PhoneBase extends Handler implements Phone {
     /**
      * Get state
      */
+    @Override
     public abstract PhoneConstants.State getState();
 
     /**
@@ -735,6 +762,7 @@ public abstract class PhoneBase extends Handler implements Phone {
     /**
      *  Query the status of the CDMA roaming preference
      */
+    @Override
     public void queryCdmaRoamingPreference(Message response) {
         mCM.queryCdmaRoamingPreference(response);
     }
@@ -755,6 +783,7 @@ public abstract class PhoneBase extends Handler implements Phone {
     /**
      *  Set the status of the CDMA roaming preference
      */
+    @Override
     public void setCdmaRoamingPreference(int cdmaRoamingType, Message response) {
         mCM.setCdmaRoamingPreference(cdmaRoamingType, response);
     }
@@ -762,6 +791,7 @@ public abstract class PhoneBase extends Handler implements Phone {
     /**
      *  Set the status of the CDMA subscription mode
      */
+    @Override
     public void setCdmaSubscription(int cdmaSubscriptionType, Message response) {
         mCM.setCdmaSubscriptionSource(cdmaSubscriptionType, response);
     }
@@ -769,56 +799,69 @@ public abstract class PhoneBase extends Handler implements Phone {
     /**
      *  Set the preferred Network Type: Global, CDMA only or GSM/UMTS only
      */
+    @Override
     public void setPreferredNetworkType(int networkType, Message response) {
         mCM.setPreferredNetworkType(networkType, response);
     }
 
+    @Override
     public void getPreferredNetworkType(Message response) {
         mCM.getPreferredNetworkType(response);
     }
 
+    @Override
     public void getSmscAddress(Message result) {
         mCM.getSmscAddress(result);
     }
 
+    @Override
     public void setSmscAddress(String address, Message result) {
         mCM.setSmscAddress(address, result);
     }
 
+    @Override
     public void setTTYMode(int ttyMode, Message onComplete) {
         mCM.setTTYMode(ttyMode, onComplete);
     }
 
+    @Override
     public void queryTTYMode(Message onComplete) {
         mCM.queryTTYMode(onComplete);
     }
 
+    @Override
     public void enableEnhancedVoicePrivacy(boolean enable, Message onComplete) {
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("enableEnhancedVoicePrivacy");
     }
 
+    @Override
     public void getEnhancedVoicePrivacy(Message onComplete) {
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("getEnhancedVoicePrivacy");
     }
 
+    @Override
     public void setBandMode(int bandMode, Message response) {
         mCM.setBandMode(bandMode, response);
     }
 
+    @Override
     public void queryAvailableBandMode(Message response) {
         mCM.queryAvailableBandMode(response);
     }
 
+    @Override
     public void invokeOemRilRequestRaw(byte[] data, Message response) {
         mCM.invokeOemRilRequestRaw(data, response);
     }
 
+    @Override
     public void invokeOemRilRequestStrings(String[] strings, Message response) {
         mCM.invokeOemRilRequestStrings(strings, response);
     }
 
+    @Override
     public void notifyDataActivity() {
         mNotifier.notifyDataActivity(this);
     }
@@ -876,11 +919,14 @@ public abstract class PhoneBase extends Handler implements Phone {
         return false;
     }
 
+    @Override
     public abstract String getPhoneName();
 
+    @Override
     public abstract int getPhoneType();
 
     /** @hide */
+    @Override
     public int getVoiceMessageCount(){
         return 0;
     }
@@ -888,6 +934,7 @@ public abstract class PhoneBase extends Handler implements Phone {
     /**
      * Returns the CDMA ERI icon index to display
      */
+    @Override
     public int getCdmaEriIconIndex() {
         logUnexpectedCdmaMethodCall("getCdmaEriIconIndex");
         return -1;
@@ -898,6 +945,7 @@ public abstract class PhoneBase extends Handler implements Phone {
      * 0 - ON
      * 1 - FLASHING
      */
+    @Override
     public int getCdmaEriIconMode() {
         logUnexpectedCdmaMethodCall("getCdmaEriIconMode");
         return -1;
@@ -906,54 +954,64 @@ public abstract class PhoneBase extends Handler implements Phone {
     /**
      * Returns the CDMA ERI text,
      */
+    @Override
     public String getCdmaEriText() {
         logUnexpectedCdmaMethodCall("getCdmaEriText");
         return "GSM nw, no ERI";
     }
 
+    @Override
     public String getCdmaMin() {
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("getCdmaMin");
         return null;
     }
 
+    @Override
     public boolean isMinInfoReady() {
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("isMinInfoReady");
         return false;
     }
 
+    @Override
     public String getCdmaPrlVersion(){
         //  This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("getCdmaPrlVersion");
         return null;
     }
 
+    @Override
     public void sendBurstDtmf(String dtmfString, int on, int off, Message onComplete) {
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("sendBurstDtmf");
     }
 
+    @Override
     public void exitEmergencyCallbackMode() {
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("exitEmergencyCallbackMode");
     }
 
+    @Override
     public void registerForCdmaOtaStatusChange(Handler h, int what, Object obj) {
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("registerForCdmaOtaStatusChange");
     }
 
+    @Override
     public void unregisterForCdmaOtaStatusChange(Handler h) {
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("unregisterForCdmaOtaStatusChange");
     }
 
+    @Override
     public void registerForSubscriptionInfoReady(Handler h, int what, Object obj) {
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("registerForSubscriptionInfoReady");
     }
 
+    @Override
     public void unregisterForSubscriptionInfoReady(Handler h) {
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("unregisterForSubscriptionInfoReady");
@@ -963,6 +1021,7 @@ public abstract class PhoneBase extends Handler implements Phone {
      * Returns true if OTA Service Provisioning needs to be performed.
      * If not overridden return false.
      */
+    @Override
     public boolean needsOtaServiceProvisioning() {
         return false;
     }
@@ -971,124 +1030,153 @@ public abstract class PhoneBase extends Handler implements Phone {
      * Return true if number is an OTASP number.
      * If not overridden return false.
      */
+    @Override
     public  boolean isOtaSpNumber(String dialStr) {
         return false;
     }
 
+    @Override
     public void registerForCallWaiting(Handler h, int what, Object obj){
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("registerForCallWaiting");
     }
 
+    @Override
     public void unregisterForCallWaiting(Handler h){
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("unregisterForCallWaiting");
     }
 
+    @Override
     public void registerForEcmTimerReset(Handler h, int what, Object obj) {
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("registerForEcmTimerReset");
     }
 
+    @Override
     public void unregisterForEcmTimerReset(Handler h) {
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
         logUnexpectedCdmaMethodCall("unregisterForEcmTimerReset");
     }
 
+    @Override
     public void registerForSignalInfo(Handler h, int what, Object obj) {
         mCM.registerForSignalInfo(h, what, obj);
     }
 
+    @Override
     public void unregisterForSignalInfo(Handler h) {
         mCM.unregisterForSignalInfo(h);
     }
 
+    @Override
     public void registerForDisplayInfo(Handler h, int what, Object obj) {
         mCM.registerForDisplayInfo(h, what, obj);
     }
 
-     public void unregisterForDisplayInfo(Handler h) {
+     @Override
+    public void unregisterForDisplayInfo(Handler h) {
          mCM.unregisterForDisplayInfo(h);
      }
 
+    @Override
     public void registerForNumberInfo(Handler h, int what, Object obj) {
         mCM.registerForNumberInfo(h, what, obj);
     }
 
+    @Override
     public void unregisterForNumberInfo(Handler h) {
         mCM.unregisterForNumberInfo(h);
     }
 
+    @Override
     public void registerForRedirectedNumberInfo(Handler h, int what, Object obj) {
         mCM.registerForRedirectedNumberInfo(h, what, obj);
     }
 
+    @Override
     public void unregisterForRedirectedNumberInfo(Handler h) {
         mCM.unregisterForRedirectedNumberInfo(h);
     }
 
+    @Override
     public void registerForLineControlInfo(Handler h, int what, Object obj) {
         mCM.registerForLineControlInfo( h, what, obj);
     }
 
+    @Override
     public void unregisterForLineControlInfo(Handler h) {
         mCM.unregisterForLineControlInfo(h);
     }
 
+    @Override
     public void registerFoT53ClirlInfo(Handler h, int what, Object obj) {
         mCM.registerFoT53ClirlInfo(h, what, obj);
     }
 
+    @Override
     public void unregisterForT53ClirInfo(Handler h) {
         mCM.unregisterForT53ClirInfo(h);
     }
 
+    @Override
     public void registerForT53AudioControlInfo(Handler h, int what, Object obj) {
         mCM.registerForT53AudioControlInfo( h, what, obj);
     }
 
+    @Override
     public void unregisterForT53AudioControlInfo(Handler h) {
         mCM.unregisterForT53AudioControlInfo(h);
     }
 
-     public void setOnEcbModeExitResponse(Handler h, int what, Object obj){
+     @Override
+    public void setOnEcbModeExitResponse(Handler h, int what, Object obj){
          // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
          logUnexpectedCdmaMethodCall("setOnEcbModeExitResponse");
      }
 
-     public void unsetOnEcbModeExitResponse(Handler h){
+     @Override
+    public void unsetOnEcbModeExitResponse(Handler h){
         // This function should be overridden by the class CDMAPhone. Not implemented in GSMPhone.
          logUnexpectedCdmaMethodCall("unsetOnEcbModeExitResponse");
      }
 
+    @Override
     public String[] getActiveApnTypes() {
         return mDataConnectionTracker.getActiveApnTypes();
     }
 
+    @Override
     public String getActiveApnHost(String apnType) {
         return mDataConnectionTracker.getActiveApnString(apnType);
     }
 
+    @Override
     public LinkProperties getLinkProperties(String apnType) {
         return mDataConnectionTracker.getLinkProperties(apnType);
     }
 
+    @Override
     public LinkCapabilities getLinkCapabilities(String apnType) {
         return mDataConnectionTracker.getLinkCapabilities(apnType);
     }
 
+    @Override
     public int enableApnType(String type) {
         return mDataConnectionTracker.enableApnType(type);
     }
 
+    @Override
     public int disableApnType(String type) {
         return mDataConnectionTracker.disableApnType(type);
     }
 
+    @Override
     public boolean isDataConnectivityPossible() {
         return isDataConnectivityPossible(PhoneConstants.APN_TYPE_DEFAULT);
     }
 
+    @Override
     public boolean isDataConnectivityPossible(String apnType) {
         return ((mDataConnectionTracker != null) &&
                 (mDataConnectionTracker.isDataPossible(apnType)));
@@ -1135,6 +1223,7 @@ public abstract class PhoneBase extends Handler implements Phone {
         }
     }
 
+    @Override
     public boolean isCspPlmnEnabled() {
         // This function should be overridden by the class GSMPhone.
         // Not implemented in CDMAPhone.
@@ -1142,15 +1231,18 @@ public abstract class PhoneBase extends Handler implements Phone {
         return false;
     }
 
+    @Override
     public IsimRecords getIsimRecords() {
         Rlog.e(LOG_TAG, "getIsimRecords() is only supported on LTE devices");
         return null;
     }
 
+    @Override
     public void requestIsimAuthentication(String nonce, Message result) {
         Rlog.e(LOG_TAG, "requestIsimAuthentication() is only supported on LTE devices");
     }
 
+    @Override
     public String getMsisdn() {
         logUnexpectedGsmMethodCall("getMsisdn");
         return null;
@@ -1165,6 +1257,7 @@ public abstract class PhoneBase extends Handler implements Phone {
                 "called, CDMAPhone inactive.");
     }
 
+    @Override
     public PhoneConstants.DataState getDataConnectionState() {
         return getDataConnectionState(PhoneConstants.APN_TYPE_DEFAULT);
     }
index c0641d5..f1bce68 100644 (file)
@@ -22,8 +22,6 @@ import android.os.Looper;
 import android.provider.Settings;
 import android.telephony.TelephonyManager;
 import android.telephony.Rlog;
-import android.os.SystemProperties;
-
 import com.android.internal.telephony.cdma.CDMAPhone;
 import com.android.internal.telephony.cdma.CDMALTEPhone;
 import com.android.internal.telephony.cdma.CdmaSubscriptionSourceManager;
@@ -36,7 +34,7 @@ import com.android.internal.telephony.uicc.UiccController;
  * {@hide}
  */
 public class PhoneFactory {
-    static final String LOG_TAG = "PHONE";
+    static final String LOG_TAG = "PhoneFactory";
     static final int SOCKET_OPEN_RETRY_MILLIS = 2 * 1000;
     static final int SOCKET_OPEN_MAX_RETRY = 3;
 
index 300529b..8a05ab1 100644 (file)
@@ -33,8 +33,6 @@ import android.telephony.ServiceState;
 import android.telephony.SignalStrength;
 import android.telephony.Rlog;
 
-import com.android.internal.telephony.cdma.CDMAPhone;
-import com.android.internal.telephony.gsm.GSMPhone;
 import com.android.internal.telephony.test.SimulatedRadioControl;
 import com.android.internal.telephony.uicc.IccCardProxy;
 import com.android.internal.telephony.uicc.IsimRecords;
@@ -62,7 +60,7 @@ public class PhoneProxy extends Handler implements Phone {
     private static final int EVENT_REQUEST_VOICE_RADIO_TECH_DONE = 3;
     private static final int EVENT_RIL_CONNECTED = 4;
 
-    private static final String LOG_TAG = "PHONE";
+    private static final String LOG_TAG = "PhoneProxy";
 
     //***** Class Methods
     public PhoneProxy(PhoneBase phone) {
@@ -135,10 +133,6 @@ public class PhoneProxy extends Handler implements Phone {
         Rlog.d(LOG_TAG, "[PhoneProxy] " + msg);
     }
 
-    private void logw(String msg) {
-        Rlog.w(LOG_TAG, "[PhoneProxy] " + msg);
-    }
-
     private void loge(String msg) {
         Rlog.e(LOG_TAG, "[PhoneProxy] " + msg);
     }
@@ -260,10 +254,12 @@ public class PhoneProxy extends Handler implements Phone {
         oldPhone = null;
     }
 
+    @Override
     public ServiceState getServiceState() {
         return mActivePhone.getServiceState();
     }
 
+    @Override
     public CellLocation getCellLocation() {
         return mActivePhone.getCellLocation();
     }
@@ -276,354 +272,441 @@ public class PhoneProxy extends Handler implements Phone {
         return mActivePhone.getAllCellInfo();
     }
 
+    @Override
     public PhoneConstants.DataState getDataConnectionState() {
         return mActivePhone.getDataConnectionState(PhoneConstants.APN_TYPE_DEFAULT);
     }
 
+    @Override
     public PhoneConstants.DataState getDataConnectionState(String apnType) {
         return mActivePhone.getDataConnectionState(apnType);
     }
 
+    @Override
     public DataActivityState getDataActivityState() {
         return mActivePhone.getDataActivityState();
     }
 
+    @Override
     public Context getContext() {
         return mActivePhone.getContext();
     }
 
+    @Override
     public void disableDnsCheck(boolean b) {
         mActivePhone.disableDnsCheck(b);
     }
 
+    @Override
     public boolean isDnsCheckDisabled() {
         return mActivePhone.isDnsCheckDisabled();
     }
 
+    @Override
     public PhoneConstants.State getState() {
         return mActivePhone.getState();
     }
 
+    @Override
     public String getPhoneName() {
         return mActivePhone.getPhoneName();
     }
 
+    @Override
     public int getPhoneType() {
         return mActivePhone.getPhoneType();
     }
 
+    @Override
     public String[] getActiveApnTypes() {
         return mActivePhone.getActiveApnTypes();
     }
 
+    @Override
     public String getActiveApnHost(String apnType) {
         return mActivePhone.getActiveApnHost(apnType);
     }
 
+    @Override
     public LinkProperties getLinkProperties(String apnType) {
         return mActivePhone.getLinkProperties(apnType);
     }
 
+    @Override
     public LinkCapabilities getLinkCapabilities(String apnType) {
         return mActivePhone.getLinkCapabilities(apnType);
     }
 
+    @Override
     public SignalStrength getSignalStrength() {
         return mActivePhone.getSignalStrength();
     }
 
+    @Override
     public void registerForUnknownConnection(Handler h, int what, Object obj) {
         mActivePhone.registerForUnknownConnection(h, what, obj);
     }
 
+    @Override
     public void unregisterForUnknownConnection(Handler h) {
         mActivePhone.unregisterForUnknownConnection(h);
     }
 
+    @Override
     public void registerForPreciseCallStateChanged(Handler h, int what, Object obj) {
         mActivePhone.registerForPreciseCallStateChanged(h, what, obj);
     }
 
+    @Override
     public void unregisterForPreciseCallStateChanged(Handler h) {
         mActivePhone.unregisterForPreciseCallStateChanged(h);
     }
 
+    @Override
     public void registerForNewRingingConnection(Handler h, int what, Object obj) {
         mActivePhone.registerForNewRingingConnection(h, what, obj);
     }
 
+    @Override
     public void unregisterForNewRingingConnection(Handler h) {
         mActivePhone.unregisterForNewRingingConnection(h);
     }
 
+    @Override
     public void registerForIncomingRing(Handler h, int what, Object obj) {
         mActivePhone.registerForIncomingRing(h, what, obj);
     }
 
+    @Override
     public void unregisterForIncomingRing(Handler h) {
         mActivePhone.unregisterForIncomingRing(h);
     }
 
+    @Override
     public void registerForDisconnect(Handler h, int what, Object obj) {
         mActivePhone.registerForDisconnect(h, what, obj);
     }
 
+    @Override
     public void unregisterForDisconnect(Handler h) {
         mActivePhone.unregisterForDisconnect(h);
     }
 
+    @Override
     public void registerForMmiInitiate(Handler h, int what, Object obj) {
         mActivePhone.registerForMmiInitiate(h, what, obj);
     }
 
+    @Override
     public void unregisterForMmiInitiate(Handler h) {
         mActivePhone.unregisterForMmiInitiate(h);
     }
 
+    @Override
     public void registerForMmiComplete(Handler h, int what, Object obj) {
         mActivePhone.registerForMmiComplete(h, what, obj);
     }
 
+    @Override
     public void unregisterForMmiComplete(Handler h) {
         mActivePhone.unregisterForMmiComplete(h);
     }
 
+    @Override
     public List<? extends MmiCode> getPendingMmiCodes() {
         return mActivePhone.getPendingMmiCodes();
     }
 
+    @Override
     public void sendUssdResponse(String ussdMessge) {
         mActivePhone.sendUssdResponse(ussdMessge);
     }
 
+    @Override
     public void registerForServiceStateChanged(Handler h, int what, Object obj) {
         mActivePhone.registerForServiceStateChanged(h, what, obj);
     }
 
+    @Override
     public void unregisterForServiceStateChanged(Handler h) {
         mActivePhone.unregisterForServiceStateChanged(h);
     }
 
+    @Override
     public void registerForSuppServiceNotification(Handler h, int what, Object obj) {
         mActivePhone.registerForSuppServiceNotification(h, what, obj);
     }
 
+    @Override
     public void unregisterForSuppServiceNotification(Handler h) {
         mActivePhone.unregisterForSuppServiceNotification(h);
     }
 
+    @Override
     public void registerForSuppServiceFailed(Handler h, int what, Object obj) {
         mActivePhone.registerForSuppServiceFailed(h, what, obj);
     }
 
+    @Override
     public void unregisterForSuppServiceFailed(Handler h) {
         mActivePhone.unregisterForSuppServiceFailed(h);
     }
 
+    @Override
     public void registerForInCallVoicePrivacyOn(Handler h, int what, Object obj){
         mActivePhone.registerForInCallVoicePrivacyOn(h,what,obj);
     }
 
+    @Override
     public void unregisterForInCallVoicePrivacyOn(Handler h){
         mActivePhone.unregisterForInCallVoicePrivacyOn(h);
     }
 
+    @Override
     public void registerForInCallVoicePrivacyOff(Handler h, int what, Object obj){
         mActivePhone.registerForInCallVoicePrivacyOff(h,what,obj);
     }
 
+    @Override
     public void unregisterForInCallVoicePrivacyOff(Handler h){
         mActivePhone.unregisterForInCallVoicePrivacyOff(h);
     }
 
+    @Override
     public void registerForCdmaOtaStatusChange(Handler h, int what, Object obj) {
         mActivePhone.registerForCdmaOtaStatusChange(h,what,obj);
     }
 
+    @Override
     public void unregisterForCdmaOtaStatusChange(Handler h) {
          mActivePhone.unregisterForCdmaOtaStatusChange(h);
     }
 
+    @Override
     public void registerForSubscriptionInfoReady(Handler h, int what, Object obj) {
         mActivePhone.registerForSubscriptionInfoReady(h, what, obj);
     }
 
+    @Override
     public void unregisterForSubscriptionInfoReady(Handler h) {
         mActivePhone.unregisterForSubscriptionInfoReady(h);
     }
 
+    @Override
     public void registerForEcmTimerReset(Handler h, int what, Object obj) {
         mActivePhone.registerForEcmTimerReset(h,what,obj);
     }
 
+    @Override
     public void unregisterForEcmTimerReset(Handler h) {
         mActivePhone.unregisterForEcmTimerReset(h);
     }
 
+    @Override
     public void registerForRingbackTone(Handler h, int what, Object obj) {
         mActivePhone.registerForRingbackTone(h,what,obj);
     }
 
+    @Override
     public void unregisterForRingbackTone(Handler h) {
         mActivePhone.unregisterForRingbackTone(h);
     }
 
+    @Override
     public void registerForResendIncallMute(Handler h, int what, Object obj) {
         mActivePhone.registerForResendIncallMute(h,what,obj);
     }
 
+    @Override
     public void unregisterForResendIncallMute(Handler h) {
         mActivePhone.unregisterForResendIncallMute(h);
     }
 
+    @Override
     public boolean getIccRecordsLoaded() {
         return mIccCardProxy.getIccRecordsLoaded();
     }
 
+    @Override
     public IccCard getIccCard() {
         return mIccCardProxy;
     }
 
+    @Override
     public void acceptCall() throws CallStateException {
         mActivePhone.acceptCall();
     }
 
+    @Override
     public void rejectCall() throws CallStateException {
         mActivePhone.rejectCall();
     }
 
+    @Override
     public void switchHoldingAndActive() throws CallStateException {
         mActivePhone.switchHoldingAndActive();
     }
 
+    @Override
     public boolean canConference() {
         return mActivePhone.canConference();
     }
 
+    @Override
     public void conference() throws CallStateException {
         mActivePhone.conference();
     }
 
+    @Override
     public void enableEnhancedVoicePrivacy(boolean enable, Message onComplete) {
         mActivePhone.enableEnhancedVoicePrivacy(enable, onComplete);
     }
 
+    @Override
     public void getEnhancedVoicePrivacy(Message onComplete) {
         mActivePhone.getEnhancedVoicePrivacy(onComplete);
     }
 
+    @Override
     public boolean canTransfer() {
         return mActivePhone.canTransfer();
     }
 
+    @Override
     public void explicitCallTransfer() throws CallStateException {
         mActivePhone.explicitCallTransfer();
     }
 
+    @Override
     public void clearDisconnected() {
         mActivePhone.clearDisconnected();
     }
 
+    @Override
     public Call getForegroundCall() {
         return mActivePhone.getForegroundCall();
     }
 
+    @Override
     public Call getBackgroundCall() {
         return mActivePhone.getBackgroundCall();
     }
 
+    @Override
     public Call getRingingCall() {
         return mActivePhone.getRingingCall();
     }
 
+    @Override
     public Connection dial(String dialString) throws CallStateException {
         return mActivePhone.dial(dialString);
     }
 
+    @Override
     public Connection dial(String dialString, UUSInfo uusInfo) throws CallStateException {
         return mActivePhone.dial(dialString, uusInfo);
     }
 
+    @Override
     public boolean handlePinMmi(String dialString) {
         return mActivePhone.handlePinMmi(dialString);
     }
 
+    @Override
     public boolean handleInCallMmiCommands(String command) throws CallStateException {
         return mActivePhone.handleInCallMmiCommands(command);
     }
 
+    @Override
     public void sendDtmf(char c) {
         mActivePhone.sendDtmf(c);
     }
 
+    @Override
     public void startDtmf(char c) {
         mActivePhone.startDtmf(c);
     }
 
+    @Override
     public void stopDtmf() {
         mActivePhone.stopDtmf();
     }
 
+    @Override
     public void setRadioPower(boolean power) {
         mActivePhone.setRadioPower(power);
     }
 
+    @Override
     public boolean getMessageWaitingIndicator() {
         return mActivePhone.getMessageWaitingIndicator();
     }
 
+    @Override
     public boolean getCallForwardingIndicator() {
         return mActivePhone.getCallForwardingIndicator();
     }
 
+    @Override
     public String getLine1Number() {
         return mActivePhone.getLine1Number();
     }
 
+    @Override
     public String getCdmaMin() {
         return mActivePhone.getCdmaMin();
     }
 
+    @Override
     public boolean isMinInfoReady() {
         return mActivePhone.isMinInfoReady();
     }
 
+    @Override
     public String getCdmaPrlVersion() {
         return mActivePhone.getCdmaPrlVersion();
     }
 
+    @Override
     public String getLine1AlphaTag() {
         return mActivePhone.getLine1AlphaTag();
     }
 
+    @Override
     public void setLine1Number(String alphaTag, String number, Message onComplete) {
         mActivePhone.setLine1Number(alphaTag, number, onComplete);
     }
 
+    @Override
     public String getVoiceMailNumber() {
         return mActivePhone.getVoiceMailNumber();
     }
 
      /** @hide */
+    @Override
     public int getVoiceMessageCount(){
         return mActivePhone.getVoiceMessageCount();
     }
 
+    @Override
     public String getVoiceMailAlphaTag() {
         return mActivePhone.getVoiceMailAlphaTag();
     }
 
+    @Override
     public void setVoiceMailNumber(String alphaTag,String voiceMailNumber,
             Message onComplete) {
         mActivePhone.setVoiceMailNumber(alphaTag, voiceMailNumber, onComplete);
     }
 
+    @Override
     public void getCallForwardingOption(int commandInterfaceCFReason,
             Message onComplete) {
         mActivePhone.getCallForwardingOption(commandInterfaceCFReason,
                 onComplete);
     }
 
+    @Override
     public void setCallForwardingOption(int commandInterfaceCFReason,
             int commandInterfaceCFAction, String dialingNumber,
             int timerSeconds, Message onComplete) {
@@ -631,228 +714,284 @@ public class PhoneProxy extends Handler implements Phone {
             commandInterfaceCFAction, dialingNumber, timerSeconds, onComplete);
     }
 
+    @Override
     public void getOutgoingCallerIdDisplay(Message onComplete) {
         mActivePhone.getOutgoingCallerIdDisplay(onComplete);
     }
 
+    @Override
     public void setOutgoingCallerIdDisplay(int commandInterfaceCLIRMode,
             Message onComplete) {
         mActivePhone.setOutgoingCallerIdDisplay(commandInterfaceCLIRMode,
                 onComplete);
     }
 
+    @Override
     public void getCallWaiting(Message onComplete) {
         mActivePhone.getCallWaiting(onComplete);
     }
 
+    @Override
     public void setCallWaiting(boolean enable, Message onComplete) {
         mActivePhone.setCallWaiting(enable, onComplete);
     }
 
+    @Override
     public void getAvailableNetworks(Message response) {
         mActivePhone.getAvailableNetworks(response);
     }
 
+    @Override
     public void setNetworkSelectionModeAutomatic(Message response) {
         mActivePhone.setNetworkSelectionModeAutomatic(response);
     }
 
+    @Override
     public void selectNetworkManually(OperatorInfo network, Message response) {
         mActivePhone.selectNetworkManually(network, response);
     }
 
+    @Override
     public void setPreferredNetworkType(int networkType, Message response) {
         mActivePhone.setPreferredNetworkType(networkType, response);
     }
 
+    @Override
     public void getPreferredNetworkType(Message response) {
         mActivePhone.getPreferredNetworkType(response);
     }
 
+    @Override
     public void getNeighboringCids(Message response) {
         mActivePhone.getNeighboringCids(response);
     }
 
+    @Override
     public void setOnPostDialCharacter(Handler h, int what, Object obj) {
         mActivePhone.setOnPostDialCharacter(h, what, obj);
     }
 
+    @Override
     public void setMute(boolean muted) {
         mActivePhone.setMute(muted);
     }
 
+    @Override
     public boolean getMute() {
         return mActivePhone.getMute();
     }
 
+    @Override
     public void setEchoSuppressionEnabled(boolean enabled) {
         mActivePhone.setEchoSuppressionEnabled(enabled);
     }
 
+    @Override
     public void invokeOemRilRequestRaw(byte[] data, Message response) {
         mActivePhone.invokeOemRilRequestRaw(data, response);
     }
 
+    @Override
     public void invokeOemRilRequestStrings(String[] strings, Message response) {
         mActivePhone.invokeOemRilRequestStrings(strings, response);
     }
 
+    @Override
     public void getDataCallList(Message response) {
         mActivePhone.getDataCallList(response);
     }
 
+    @Override
     public void updateServiceLocation() {
         mActivePhone.updateServiceLocation();
     }
 
+    @Override
     public void enableLocationUpdates() {
         mActivePhone.enableLocationUpdates();
     }
 
+    @Override
     public void disableLocationUpdates() {
         mActivePhone.disableLocationUpdates();
     }
 
+    @Override
     public void setUnitTestMode(boolean f) {
         mActivePhone.setUnitTestMode(f);
     }
 
+    @Override
     public boolean getUnitTestMode() {
         return mActivePhone.getUnitTestMode();
     }
 
+    @Override
     public void setBandMode(int bandMode, Message response) {
         mActivePhone.setBandMode(bandMode, response);
     }
 
+    @Override
     public void queryAvailableBandMode(Message response) {
         mActivePhone.queryAvailableBandMode(response);
     }
 
+    @Override
     public boolean getDataRoamingEnabled() {
         return mActivePhone.getDataRoamingEnabled();
     }
 
+    @Override
     public void setDataRoamingEnabled(boolean enable) {
         mActivePhone.setDataRoamingEnabled(enable);
     }
 
+    @Override
     public void queryCdmaRoamingPreference(Message response) {
         mActivePhone.queryCdmaRoamingPreference(response);
     }
 
+    @Override
     public void setCdmaRoamingPreference(int cdmaRoamingType, Message response) {
         mActivePhone.setCdmaRoamingPreference(cdmaRoamingType, response);
     }
 
+    @Override
     public void setCdmaSubscription(int cdmaSubscriptionType, Message response) {
         mActivePhone.setCdmaSubscription(cdmaSubscriptionType, response);
     }
 
+    @Override
     public SimulatedRadioControl getSimulatedRadioControl() {
         return mActivePhone.getSimulatedRadioControl();
     }
 
+    @Override
     public int enableApnType(String type) {
         return mActivePhone.enableApnType(type);
     }
 
+    @Override
     public int disableApnType(String type) {
         return mActivePhone.disableApnType(type);
     }
 
+    @Override
     public boolean isDataConnectivityPossible() {
         return mActivePhone.isDataConnectivityPossible(PhoneConstants.APN_TYPE_DEFAULT);
     }
 
+    @Override
     public boolean isDataConnectivityPossible(String apnType) {
         return mActivePhone.isDataConnectivityPossible(apnType);
     }
 
+    @Override
     public String getDeviceId() {
         return mActivePhone.getDeviceId();
     }
 
+    @Override
     public String getDeviceSvn() {
         return mActivePhone.getDeviceSvn();
     }
 
+    @Override
     public String getSubscriberId() {
         return mActivePhone.getSubscriberId();
     }
 
+    @Override
     public String getIccSerialNumber() {
         return mActivePhone.getIccSerialNumber();
     }
 
+    @Override
     public String getEsn() {
         return mActivePhone.getEsn();
     }
 
+    @Override
     public String getMeid() {
         return mActivePhone.getMeid();
     }
 
+    @Override
     public String getMsisdn() {
         return mActivePhone.getMsisdn();
     }
 
+    @Override
     public String getImei() {
         return mActivePhone.getImei();
     }
 
+    @Override
     public PhoneSubInfo getPhoneSubInfo(){
         return mActivePhone.getPhoneSubInfo();
     }
 
+    @Override
     public IccSmsInterfaceManager getIccSmsInterfaceManager(){
         return mActivePhone.getIccSmsInterfaceManager();
     }
 
+    @Override
     public IccPhoneBookInterfaceManager getIccPhoneBookInterfaceManager(){
         return mActivePhone.getIccPhoneBookInterfaceManager();
     }
 
+    @Override
     public void setTTYMode(int ttyMode, Message onComplete) {
         mActivePhone.setTTYMode(ttyMode, onComplete);
     }
 
+    @Override
     public void queryTTYMode(Message onComplete) {
         mActivePhone.queryTTYMode(onComplete);
     }
 
+    @Override
     public void activateCellBroadcastSms(int activate, Message response) {
         mActivePhone.activateCellBroadcastSms(activate, response);
     }
 
+    @Override
     public void getCellBroadcastSmsConfig(Message response) {
         mActivePhone.getCellBroadcastSmsConfig(response);
     }
 
+    @Override
     public void setCellBroadcastSmsConfig(int[] configValuesArray, Message response) {
         mActivePhone.setCellBroadcastSmsConfig(configValuesArray, response);
     }
 
+    @Override
     public void notifyDataActivity() {
          mActivePhone.notifyDataActivity();
     }
 
+    @Override
     public void getSmscAddress(Message result) {
         mActivePhone.getSmscAddress(result);
     }
 
+    @Override
     public void setSmscAddress(String address, Message result) {
         mActivePhone.setSmscAddress(address, result);
     }
 
+    @Override
     public int getCdmaEriIconIndex() {
         return mActivePhone.getCdmaEriIconIndex();
     }
 
+    @Override
     public String getCdmaEriText() {
         return mActivePhone.getCdmaEriText();
     }
 
+    @Override
     public int getCdmaEriIconMode() {
         return mActivePhone.getCdmaEriIconMode();
     }
@@ -861,102 +1000,127 @@ public class PhoneProxy extends Handler implements Phone {
         return mActivePhone;
     }
 
+    @Override
     public void sendBurstDtmf(String dtmfString, int on, int off, Message onComplete){
         mActivePhone.sendBurstDtmf(dtmfString, on, off, onComplete);
     }
 
+    @Override
     public void exitEmergencyCallbackMode(){
         mActivePhone.exitEmergencyCallbackMode();
     }
 
+    @Override
     public boolean needsOtaServiceProvisioning(){
         return mActivePhone.needsOtaServiceProvisioning();
     }
 
+    @Override
     public boolean isOtaSpNumber(String dialStr){
         return mActivePhone.isOtaSpNumber(dialStr);
     }
 
+    @Override
     public void registerForCallWaiting(Handler h, int what, Object obj){
         mActivePhone.registerForCallWaiting(h,what,obj);
     }
 
+    @Override
     public void unregisterForCallWaiting(Handler h){
         mActivePhone.unregisterForCallWaiting(h);
     }
 
+    @Override
     public void registerForSignalInfo(Handler h, int what, Object obj) {
         mActivePhone.registerForSignalInfo(h,what,obj);
     }
 
+    @Override
     public void unregisterForSignalInfo(Handler h) {
         mActivePhone.unregisterForSignalInfo(h);
     }
 
+    @Override
     public void registerForDisplayInfo(Handler h, int what, Object obj) {
         mActivePhone.registerForDisplayInfo(h,what,obj);
     }
 
+    @Override
     public void unregisterForDisplayInfo(Handler h) {
         mActivePhone.unregisterForDisplayInfo(h);
     }
 
+    @Override
     public void registerForNumberInfo(Handler h, int what, Object obj) {
         mActivePhone.registerForNumberInfo(h, what, obj);
     }
 
+    @Override
     public void unregisterForNumberInfo(Handler h) {
         mActivePhone.unregisterForNumberInfo(h);
     }
 
+    @Override
     public void registerForRedirectedNumberInfo(Handler h, int what, Object obj) {
         mActivePhone.registerForRedirectedNumberInfo(h, what, obj);
     }
 
+    @Override
     public void unregisterForRedirectedNumberInfo(Handler h) {
         mActivePhone.unregisterForRedirectedNumberInfo(h);
     }
 
+    @Override
     public void registerForLineControlInfo(Handler h, int what, Object obj) {
         mActivePhone.registerForLineControlInfo( h, what, obj);
     }
 
+    @Override
     public void unregisterForLineControlInfo(Handler h) {
         mActivePhone.unregisterForLineControlInfo(h);
     }
 
+    @Override
     public void registerFoT53ClirlInfo(Handler h, int what, Object obj) {
         mActivePhone.registerFoT53ClirlInfo(h, what, obj);
     }
 
+    @Override
     public void unregisterForT53ClirInfo(Handler h) {
         mActivePhone.unregisterForT53ClirInfo(h);
     }
 
+    @Override
     public void registerForT53AudioControlInfo(Handler h, int what, Object obj) {
         mActivePhone.registerForT53AudioControlInfo( h, what, obj);
     }
 
+    @Override
     public void unregisterForT53AudioControlInfo(Handler h) {
         mActivePhone.unregisterForT53AudioControlInfo(h);
     }
 
+    @Override
     public void setOnEcbModeExitResponse(Handler h, int what, Object obj){
         mActivePhone.setOnEcbModeExitResponse(h,what,obj);
     }
 
+    @Override
     public void unsetOnEcbModeExitResponse(Handler h){
         mActivePhone.unsetOnEcbModeExitResponse(h);
     }
 
+    @Override
     public boolean isCspPlmnEnabled() {
         return mActivePhone.isCspPlmnEnabled();
     }
 
+    @Override
     public IsimRecords getIsimRecords() {
         return mActivePhone.getIsimRecords();
     }
 
+    @Override
     public void requestIsimAuthentication(String nonce, Message response) {
         mActivePhone.requestIsimAuthentication(nonce, response);
     }
@@ -979,12 +1143,14 @@ public class PhoneProxy extends Handler implements Phone {
         return mActivePhone.getUsimServiceTable();
     }
 
+    @Override
     public void dispose() {
         mCommandsInterface.unregisterForOn(this);
         mCommandsInterface.unregisterForVoiceRadioTechChanged(this);
         mCommandsInterface.unregisterForRilConnected(this);
     }
 
+    @Override
     public void removeReferences() {
         mActivePhone = null;
         mCommandsInterface = null;
index 40f0c34..9dad3cc 100644 (file)
@@ -37,15 +37,13 @@ import android.telephony.Rlog;
  */
 @Deprecated
 public final class PhoneStateIntentReceiver extends BroadcastReceiver {
-    private static final String LOG_TAG = "PHONE";
+    private static final String LOG_TAG = "PhoneStatIntentReceiver";
     private static final boolean DBG = false;
 
     private static final int NOTIF_PHONE    = 1 << 0;
     private static final int NOTIF_SERVICE  = 1 << 1;
     private static final int NOTIF_SIGNAL   = 1 << 2;
 
-    private static final int NOTIF_MAX      = 1 << 5;
-
     PhoneConstants.State mPhoneState = PhoneConstants.State.IDLE;
     ServiceState mServiceState = new ServiceState();
     SignalStrength mSignalStrength = new SignalStrength();
@@ -56,7 +54,6 @@ public final class PhoneStateIntentReceiver extends BroadcastReceiver {
     private int mWants;
     private int mPhoneStateEventWhat;
     private int mServiceStateEventWhat;
-    private int mLocationEventWhat;
     private int mAsuEventWhat;
 
     public PhoneStateIntentReceiver() {
@@ -177,7 +174,7 @@ public final class PhoneStateIntentReceiver extends BroadcastReceiver {
                 if (DBG) Rlog.d(LOG_TAG, "onReceiveIntent: ACTION_PHONE_STATE_CHANGED, state="
                                + intent.getStringExtra(PhoneConstants.STATE_KEY));
                 String phoneState = intent.getStringExtra(PhoneConstants.STATE_KEY);
-                mPhoneState = (PhoneConstants.State) Enum.valueOf(
+                mPhoneState = Enum.valueOf(
                         PhoneConstants.State.class, phoneState);
 
                 if (mTarget != null && getNotifyPhoneCallState()) {
index cbab817..6effe76 100755 (executable)
@@ -27,7 +27,10 @@ import android.telephony.Rlog;
 import com.android.internal.telephony.uicc.IsimRecords;
 
 public class PhoneSubInfo extends IPhoneSubInfo.Stub {
-    static final String LOG_TAG = "PHONE";
+    static final String LOG_TAG = "PhoneSubInfo";
+    private static final boolean DBG = true;
+    private static final boolean VDBG = false; // STOPSHIP if true
+
     private Phone mPhone;
     private Context mContext;
     private static final String READ_PHONE_STATE =
@@ -46,18 +49,20 @@ public class PhoneSubInfo extends IPhoneSubInfo.Stub {
     public void dispose() {
     }
 
+    @Override
     protected void finalize() {
         try {
             super.finalize();
         } catch (Throwable throwable) {
-            Rlog.e(LOG_TAG, "Error while finalizing:", throwable);
+            loge("Error while finalizing:", throwable);
         }
-        Rlog.d(LOG_TAG, "PhoneSubInfo finalized");
+        if (DBG) log("PhoneSubInfo finalized");
     }
 
     /**
      * Retrieves the unique device ID, e.g., IMEI for GSM phones and MEID for CDMA phones.
      */
+    @Override
     public String getDeviceId() {
         mContext.enforceCallingOrSelfPermission(READ_PHONE_STATE, "Requires READ_PHONE_STATE");
         return mPhone.getDeviceId();
@@ -67,6 +72,7 @@ public class PhoneSubInfo extends IPhoneSubInfo.Stub {
      * Retrieves the software version number for the device, e.g., IMEI/SV
      * for GSM phones.
      */
+    @Override
     public String getDeviceSvn() {
         mContext.enforceCallingOrSelfPermission(READ_PHONE_STATE, "Requires READ_PHONE_STATE");
         return mPhone.getDeviceSvn();
@@ -75,6 +81,7 @@ public class PhoneSubInfo extends IPhoneSubInfo.Stub {
     /**
      * Retrieves the unique subscriber ID, e.g., IMSI for GSM phones.
      */
+    @Override
     public String getSubscriberId() {
         mContext.enforceCallingOrSelfPermission(READ_PHONE_STATE, "Requires READ_PHONE_STATE");
         return mPhone.getSubscriberId();
@@ -83,6 +90,7 @@ public class PhoneSubInfo extends IPhoneSubInfo.Stub {
     /**
      * Retrieves the serial number of the ICC, if applicable.
      */
+    @Override
     public String getIccSerialNumber() {
         mContext.enforceCallingOrSelfPermission(READ_PHONE_STATE, "Requires READ_PHONE_STATE");
         return mPhone.getIccSerialNumber();
@@ -91,6 +99,7 @@ public class PhoneSubInfo extends IPhoneSubInfo.Stub {
     /**
      * Retrieves the phone number string for line 1.
      */
+    @Override
     public String getLine1Number() {
         mContext.enforceCallingOrSelfPermission(READ_PHONE_STATE, "Requires READ_PHONE_STATE");
         return mPhone.getLine1Number();
@@ -99,14 +108,16 @@ public class PhoneSubInfo extends IPhoneSubInfo.Stub {
     /**
      * Retrieves the alpha identifier for line 1.
      */
+    @Override
     public String getLine1AlphaTag() {
         mContext.enforceCallingOrSelfPermission(READ_PHONE_STATE, "Requires READ_PHONE_STATE");
-        return (String) mPhone.getLine1AlphaTag();
+        return mPhone.getLine1AlphaTag();
     }
 
     /**
      * Retrieves the MSISDN string.
      */
+    @Override
     public String getMsisdn() {
         mContext.enforceCallingOrSelfPermission(READ_PHONE_STATE, "Requires READ_PHONE_STATE");
         return mPhone.getMsisdn();
@@ -115,10 +126,11 @@ public class PhoneSubInfo extends IPhoneSubInfo.Stub {
     /**
      * Retrieves the voice mail number.
      */
+    @Override
     public String getVoiceMailNumber() {
         mContext.enforceCallingOrSelfPermission(READ_PHONE_STATE, "Requires READ_PHONE_STATE");
         String number = PhoneNumberUtils.extractNetworkPortion(mPhone.getVoiceMailNumber());
-        Rlog.d(LOG_TAG, "VM: PhoneSubInfo.getVoiceMailNUmber: "); // + number);
+        if (VDBG) log("VM: PhoneSubInfo.getVoiceMailNUmber: " + number);
         return number;
     }
 
@@ -127,26 +139,29 @@ public class PhoneSubInfo extends IPhoneSubInfo.Stub {
      *
      * @hide
      */
+    @Override
     public String getCompleteVoiceMailNumber() {
         mContext.enforceCallingOrSelfPermission(CALL_PRIVILEGED,
                 "Requires CALL_PRIVILEGED");
         String number = mPhone.getVoiceMailNumber();
-        Rlog.d(LOG_TAG, "VM: PhoneSubInfo.getCompleteVoiceMailNUmber: "); // + number);
+        if (VDBG) log("VM: PhoneSubInfo.getCompleteVoiceMailNUmber: " + number);
         return number;
     }
 
     /**
      * Retrieves the alpha identifier associated with the voice mail number.
      */
+    @Override
     public String getVoiceMailAlphaTag() {
         mContext.enforceCallingOrSelfPermission(READ_PHONE_STATE, "Requires READ_PHONE_STATE");
-        return (String) mPhone.getVoiceMailAlphaTag();
+        return mPhone.getVoiceMailAlphaTag();
     }
 
     /**
      * Returns the IMS private user identity (IMPI) that was loaded from the ISIM.
      * @return the IMPI, or null if not present or not loaded
      */
+    @Override
     public String getIsimImpi() {
         mContext.enforceCallingOrSelfPermission(READ_PRIVILEGED_PHONE_STATE,
                 "Requires READ_PRIVILEGED_PHONE_STATE");
@@ -162,6 +177,7 @@ public class PhoneSubInfo extends IPhoneSubInfo.Stub {
      * Returns the IMS home network domain name that was loaded from the ISIM.
      * @return the IMS domain name, or null if not present or not loaded
      */
+    @Override
     public String getIsimDomain() {
         mContext.enforceCallingOrSelfPermission(READ_PRIVILEGED_PHONE_STATE,
                 "Requires READ_PRIVILEGED_PHONE_STATE");
@@ -178,6 +194,7 @@ public class PhoneSubInfo extends IPhoneSubInfo.Stub {
      * @return an array of IMPU strings, with one IMPU per string, or null if
      *      not present or not loaded
      */
+    @Override
     public String[] getIsimImpu() {
         mContext.enforceCallingOrSelfPermission(READ_PRIVILEGED_PHONE_STATE,
                 "Requires READ_PRIVILEGED_PHONE_STATE");
@@ -189,6 +206,15 @@ public class PhoneSubInfo extends IPhoneSubInfo.Stub {
         }
     }
 
+    private void log(String s) {
+        Rlog.d(LOG_TAG, s);
+    }
+
+    private void loge(String s, Throwable e) {
+        Rlog.e(LOG_TAG, s, e);
+    }
+
+    @Override
     protected void dump(FileDescriptor fd, PrintWriter pw, String[] args) {
         if (mContext.checkCallingOrSelfPermission(android.Manifest.permission.DUMP)
                 != PackageManager.PERMISSION_GRANTED) {
@@ -202,5 +228,4 @@ public class PhoneSubInfo extends IPhoneSubInfo.Stub {
         pw.println("  Phone Type = " + mPhone.getPhoneName());
         pw.println("  Device ID = " + mPhone.getDeviceId());
     }
-
 }
index bd130de..610f0c2 100755 (executable)
@@ -19,8 +19,6 @@ package com.android.internal.telephony;
 import java.io.FileDescriptor;
 import java.io.PrintWriter;
 
-import android.content.pm.PackageManager;
-import android.os.Binder;
 import android.os.ServiceManager;
 
 
@@ -38,10 +36,12 @@ public class PhoneSubInfoProxy extends IPhoneSubInfo.Stub {
         this.mPhoneSubInfo = phoneSubInfo;
     }
 
+    @Override
     public String getDeviceId() {
         return mPhoneSubInfo.getDeviceId();
     }
 
+    @Override
     public String getDeviceSvn() {
         return mPhoneSubInfo.getDeviceSvn();
     }
@@ -49,6 +49,7 @@ public class PhoneSubInfoProxy extends IPhoneSubInfo.Stub {
     /**
      * Retrieves the unique subscriber ID, e.g., IMSI for GSM phones.
      */
+    @Override
     public String getSubscriberId() {
         return mPhoneSubInfo.getSubscriberId();
     }
@@ -56,6 +57,7 @@ public class PhoneSubInfoProxy extends IPhoneSubInfo.Stub {
     /**
      * Retrieves the serial number of the ICC, if applicable.
      */
+    @Override
     public String getIccSerialNumber() {
         return mPhoneSubInfo.getIccSerialNumber();
     }
@@ -63,6 +65,7 @@ public class PhoneSubInfoProxy extends IPhoneSubInfo.Stub {
     /**
      * Retrieves the phone number string for line 1.
      */
+    @Override
     public String getLine1Number() {
         return mPhoneSubInfo.getLine1Number();
     }
@@ -70,6 +73,7 @@ public class PhoneSubInfoProxy extends IPhoneSubInfo.Stub {
     /**
      * Retrieves the alpha identifier for line 1.
      */
+    @Override
     public String getLine1AlphaTag() {
         return mPhoneSubInfo.getLine1AlphaTag();
     }
@@ -77,6 +81,7 @@ public class PhoneSubInfoProxy extends IPhoneSubInfo.Stub {
     /**
      * Retrieves the MSISDN Number.
      */
+    @Override
     public String getMsisdn() {
         return mPhoneSubInfo.getMsisdn();
     }
@@ -84,6 +89,7 @@ public class PhoneSubInfoProxy extends IPhoneSubInfo.Stub {
     /**
      * Retrieves the voice mail number.
      */
+    @Override
     public String getVoiceMailNumber() {
         return mPhoneSubInfo.getVoiceMailNumber();
     }
@@ -91,6 +97,7 @@ public class PhoneSubInfoProxy extends IPhoneSubInfo.Stub {
     /**
      * Retrieves the complete voice mail number.
      */
+    @Override
     public String getCompleteVoiceMailNumber() {
         return mPhoneSubInfo.getCompleteVoiceMailNumber();
     }
@@ -98,6 +105,7 @@ public class PhoneSubInfoProxy extends IPhoneSubInfo.Stub {
     /**
      * Retrieves the alpha identifier associated with the voice mail number.
      */
+    @Override
     public String getVoiceMailAlphaTag() {
         return mPhoneSubInfo.getVoiceMailAlphaTag();
     }
@@ -106,6 +114,7 @@ public class PhoneSubInfoProxy extends IPhoneSubInfo.Stub {
      * Returns the IMS private user identity (IMPI) that was loaded from the ISIM.
      * @return the IMPI, or null if not present or not loaded
      */
+    @Override
     public String getIsimImpi() {
         return mPhoneSubInfo.getIsimImpi();
     }
@@ -114,6 +123,7 @@ public class PhoneSubInfoProxy extends IPhoneSubInfo.Stub {
      * Returns the IMS home network domain name that was loaded from the ISIM.
      * @return the IMS domain name, or null if not present or not loaded
      */
+    @Override
     public String getIsimDomain() {
         return mPhoneSubInfo.getIsimDomain();
     }
@@ -123,10 +133,12 @@ public class PhoneSubInfoProxy extends IPhoneSubInfo.Stub {
      * @return an array of IMPU strings, with one IMPU per string, or null if
      *      not present or not loaded
      */
+    @Override
     public String[] getIsimImpu() {
         return mPhoneSubInfo.getIsimImpu();
     }
 
+    @Override
     protected void dump(FileDescriptor fd, PrintWriter pw, String[] args) {
         mPhoneSubInfo.dump(fd, pw, args);
     }
index c4dfcea..3301f44 100644 (file)
@@ -59,7 +59,7 @@ import com.android.internal.telephony.uicc.IccUtils;
 import com.android.internal.telephony.cdma.CdmaCallWaitingNotification;
 import com.android.internal.telephony.cdma.CdmaInformationRecords;
 import com.android.internal.telephony.cdma.CdmaSmsBroadcastConfigInfo;
-import com.android.internal.telephony.dataconnection.DataConnection;
+import com.android.internal.telephony.dataconnection.DataConnectionBase;
 
 import java.io.ByteArrayInputStream;
 import java.io.DataInputStream;
@@ -75,7 +75,7 @@ import java.util.concurrent.atomic.AtomicBoolean;
  * {@hide}
  */
 class RILRequest {
-    static final String LOG_TAG = "RILJ";
+    static final String LOG_TAG = "RilRequest";
 
     //***** Class Variables
     static int sNextSerial = 0;
@@ -209,7 +209,7 @@ class RILRequest {
 public final class RIL extends BaseCommands implements CommandsInterface {
     static final String RILJ_LOG_TAG = "RILJ";
     static final boolean RILJ_LOGD = true;
-    static final boolean RILJ_LOGV = false; // STOP SHIP if true
+    static final boolean RILJ_LOGV = false; // STOPSHIP if true
 
     /**
      * Wake lock timeout should be longer than the longest timeout in
@@ -288,6 +288,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         byte[] dataLength = new byte[4];
 
         //***** Runnable implementation
+        @Override
         public void
         run() {
             //setup if needed
@@ -505,6 +506,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
             buffer = new byte[RIL_MAX_COMMAND_BYTES];
         }
 
+        @Override
         public void
         run() {
             int retryCount = 0;
@@ -659,6 +661,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
 
     //***** CommandsInterface implementation
 
+    @Override
     public void getVoiceRadioTechnology(Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_VOICE_RADIO_TECH, result);
 
@@ -681,6 +684,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         }
     }
 
+    @Override
     public void
     getIccCardStatus(Message result) {
         //Note: This RIL request has not been renamed to ICC,
@@ -816,6 +820,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     changeBarringPassword(String facility, String oldPwd, String newPwd, Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_CHANGE_BARRING_PASSWORD, result);
@@ -830,6 +835,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     supplyNetworkDepersonalization(String netpin, Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_ENTER_NETWORK_DEPERSONALIZATION, result);
@@ -842,6 +848,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getCurrentCalls (Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_GET_CURRENT_CALLS, result);
@@ -851,11 +858,13 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     @Deprecated public void
     getPDPContextList(Message result) {
         getDataCallList(result);
     }
 
+    @Override
     public void
     getDataCallList(Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_DATA_CALL_LIST, result);
@@ -865,11 +874,13 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     dial (String address, int clirMode, Message result) {
         dial(address, clirMode, null, result);
     }
 
+    @Override
     public void
     dial(String address, int clirMode, UUSInfo uusInfo, Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_DIAL, result);
@@ -892,11 +903,13 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getIMSI(Message result) {
         getIMSIForApp(null, result);
     }
 
+    @Override
     public void
     getIMSIForApp(String aid, Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_GET_IMSI, result);
@@ -911,6 +924,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getIMEI(Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_GET_IMEI, result);
@@ -920,6 +934,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getIMEISV(Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_GET_IMEISV, result);
@@ -930,6 +945,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     }
 
 
+    @Override
     public void
     hangupConnection (int gsmIndex, Message result) {
         if (RILJ_LOGD) riljLog("hangupConnection: gsmIndex=" + gsmIndex);
@@ -945,6 +961,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     hangupWaitingOrBackground (Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_HANGUP_WAITING_OR_BACKGROUND,
@@ -955,6 +972,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     hangupForegroundResumeBackground (Message result) {
         RILRequest rr
@@ -966,6 +984,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     switchWaitingOrHoldingAndActive (Message result) {
         RILRequest rr
@@ -977,6 +996,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     conference (Message result) {
         RILRequest rr
@@ -988,6 +1008,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     }
 
 
+    @Override
     public void setPreferredVoicePrivacy(boolean enable, Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_CDMA_SET_PREFERRED_VOICE_PRIVACY_MODE,
                 result);
@@ -998,12 +1019,14 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void getPreferredVoicePrivacy(Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_CDMA_QUERY_PREFERRED_VOICE_PRIVACY_MODE,
                 result);
         send(rr);
     }
 
+    @Override
     public void
     separateConnection (int gsmIndex, Message result) {
         RILRequest rr
@@ -1018,6 +1041,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     acceptCall (Message result) {
         RILRequest rr
@@ -1028,6 +1052,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     rejectCall (Message result) {
         RILRequest rr
@@ -1038,6 +1063,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     explicitCallTransfer (Message result) {
         RILRequest rr
@@ -1048,6 +1074,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getLastCallFailCause (Message result) {
         RILRequest rr
@@ -1061,6 +1088,8 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * @deprecated
      */
+    @Deprecated
+    @Override
     public void
     getLastPdpFailCause (Message result) {
         getLastDataCallFailCause (result);
@@ -1069,6 +1098,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * The preferred new alternative to getLastPdpFailCause
      */
+    @Override
     public void
     getLastDataCallFailCause (Message result) {
         RILRequest rr
@@ -1079,6 +1109,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     setMute (boolean enableMute, Message response) {
         RILRequest rr
@@ -1093,6 +1124,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getMute (Message response) {
         RILRequest rr
@@ -1103,6 +1135,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getSignalStrength (Message result) {
         RILRequest rr
@@ -1113,6 +1146,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getVoiceRegistrationState (Message result) {
         RILRequest rr
@@ -1123,6 +1157,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getDataRegistrationState (Message result) {
         RILRequest rr
@@ -1133,6 +1168,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getOperator(Message result) {
         RILRequest rr
@@ -1143,6 +1179,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     sendDtmf(char c, Message result) {
         RILRequest rr
@@ -1155,6 +1192,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     startDtmf(char c, Message result) {
         RILRequest rr
@@ -1167,6 +1205,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     stopDtmf(Message result) {
         RILRequest rr
@@ -1177,6 +1216,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     sendBurstDtmf(String dtmfString, int on, int off, Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_CDMA_BURST_DTMF, result);
@@ -1192,6 +1232,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     sendSMS (String smscPDU, String pdu, Message result) {
         RILRequest rr
@@ -1206,6 +1247,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     sendCdmaSms(byte[] pdu, Message result) {
         int address_nbr_of_digits;
@@ -1253,6 +1295,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void deleteSmsOnSim(int index, Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_DELETE_SMS_ON_SIM,
                 response);
@@ -1260,15 +1303,14 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         rr.mp.writeInt(1);
         rr.mp.writeInt(index);
 
-        if (false) {
-            if (RILJ_LOGD) riljLog(rr.serialString() + "> "
-                    + requestToString(rr.mRequest)
-                    + " " + index);
-        }
+        if (RILJ_LOGV) riljLog(rr.serialString() + "> "
+                + requestToString(rr.mRequest)
+                + " " + index);
 
         send(rr);
     }
 
+    @Override
     public void deleteSmsOnRuim(int index, Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_CDMA_DELETE_SMS_ON_RUIM,
                 response);
@@ -1276,15 +1318,14 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         rr.mp.writeInt(1);
         rr.mp.writeInt(index);
 
-        if (false) {
-            if (RILJ_LOGD) riljLog(rr.serialString() + "> "
-                    + requestToString(rr.mRequest)
-                    + " " + index);
-        }
+        if (RILJ_LOGV) riljLog(rr.serialString() + "> "
+                + requestToString(rr.mRequest)
+                + " " + index);
 
         send(rr);
     }
 
+    @Override
     public void writeSmsToSim(int status, String smsc, String pdu, Message response) {
         status = translateStatus(status);
 
@@ -1295,15 +1336,14 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         rr.mp.writeString(pdu);
         rr.mp.writeString(smsc);
 
-        if (false) {
-            if (RILJ_LOGD) riljLog(rr.serialString() + "> "
-                    + requestToString(rr.mRequest)
-                    + " " + status);
-        }
+        if (RILJ_LOGV) riljLog(rr.serialString() + "> "
+                + requestToString(rr.mRequest)
+                + " " + status);
 
         send(rr);
     }
 
+    @Override
     public void writeSmsToRuim(int status, String pdu, Message response) {
         status = translateStatus(status);
 
@@ -1313,11 +1353,9 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         rr.mp.writeInt(status);
         rr.mp.writeString(pdu);
 
-        if (false) {
-            if (RILJ_LOGD) riljLog(rr.serialString() + "> "
-                    + requestToString(rr.mRequest)
-                    + " " + status);
-        }
+        if (RILJ_LOGV) riljLog(rr.serialString() + "> "
+                + requestToString(rr.mRequest)
+                + " " + status);
 
         send(rr);
     }
@@ -1342,6 +1380,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         return 1;
     }
 
+    @Override
     public void
     setupDataCall(String radioTechnology, String profile, String apn,
             String user, String password, String authType, String protocol,
@@ -1367,6 +1406,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     deactivateDataCall(int cid, int reason, Message result) {
         RILRequest rr
@@ -1382,6 +1422,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     setRadioPower(boolean on, Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_RADIO_POWER, result);
@@ -1397,6 +1438,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     setSuppServiceNotifications(boolean enable, Message result) {
         RILRequest rr
@@ -1411,6 +1453,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     acknowledgeLastIncomingGsmSms(boolean success, int cause, Message result) {
         RILRequest rr
@@ -1426,6 +1469,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     acknowledgeLastIncomingCdmaSms(boolean success, int cause, Message result) {
         RILRequest rr
@@ -1441,6 +1485,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     acknowledgeIncomingGsmSmsWithPdu(boolean success, String ackPdu, Message result) {
         RILRequest rr
@@ -1456,11 +1501,13 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     iccIO (int command, int fileid, String path, int p1, int p2, int p3,
             String data, String pin2, Message result) {
         iccIOForApp(command, fileid, path, p1, p2, p3, data, pin2, null, result);
     }
+    @Override
     public void
     iccIOForApp (int command, int fileid, String path, int p1, int p2, int p3,
             String data, String pin2, String aid, Message result) {
@@ -1490,6 +1537,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getCLIR(Message result) {
         RILRequest rr
@@ -1500,6 +1548,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     setCLIR(int clirMode, Message result) {
         RILRequest rr
@@ -1516,6 +1565,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     queryCallWaiting(int serviceClass, Message response) {
         RILRequest rr
@@ -1530,6 +1580,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     setCallWaiting(boolean enable, int serviceClass, Message response) {
         RILRequest rr
@@ -1545,6 +1596,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     setNetworkSelectionModeAutomatic(Message response) {
         RILRequest rr
@@ -1556,6 +1608,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     setNetworkSelectionModeManual(String operatorNumeric, Message response) {
         RILRequest rr
@@ -1570,6 +1623,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getNetworkSelectionMode(Message response) {
         RILRequest rr
@@ -1581,6 +1635,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getAvailableNetworks(Message response) {
         RILRequest rr
@@ -1592,6 +1647,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     setCallForward(int action, int cfReason, int serviceClass,
                 String number, int timeSeconds, Message response) {
@@ -1612,6 +1668,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     queryCallForwardStatus(int cfReason, int serviceClass,
                 String number, Message response) {
@@ -1631,6 +1688,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     queryCLIP(Message response) {
         RILRequest rr
@@ -1642,6 +1700,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     }
 
 
+    @Override
     public void
     getBasebandVersion (Message response) {
         RILRequest rr
@@ -1714,6 +1773,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
 
     }
 
+    @Override
     public void
     sendUSSD (String ussdString, Message response) {
         RILRequest rr
@@ -1732,6 +1792,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     }
 
     // inherited javadoc suffices
+    @Override
     public void cancelPendingUssd (Message response) {
         RILRequest rr
                 = RILRequest.obtain(RIL_REQUEST_CANCEL_USSD, response);
@@ -1743,6 +1804,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     }
 
 
+    @Override
     public void resetRadio(Message result) {
         RILRequest rr
                 = RILRequest.obtain(RIL_REQUEST_RESET_RADIO, result);
@@ -1752,6 +1814,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void invokeOemRilRequestRaw(byte[] data, Message response) {
         RILRequest rr
                 = RILRequest.obtain(RIL_REQUEST_OEM_HOOK_RAW, response);
@@ -1765,6 +1828,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
 
     }
 
+    @Override
     public void invokeOemRilRequestStrings(String[] strings, Message response) {
         RILRequest rr
                 = RILRequest.obtain(RIL_REQUEST_OEM_HOOK_STRINGS, response);
@@ -1782,6 +1846,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
      * @param bandMode one of BM_*_BAND
      * @param response is callback message
      */
+    @Override
     public void setBandMode (int bandMode, Message response) {
         RILRequest rr
                 = RILRequest.obtain(RIL_REQUEST_SET_BAND_MODE, response);
@@ -1802,6 +1867,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
      *        ((AsyncResult)response.obj).result  is an int[] with every
      *        element representing one avialable BM_*_BAND
      */
+    @Override
     public void queryAvailableBandMode (Message response) {
         RILRequest rr
                 = RILRequest.obtain(RIL_REQUEST_QUERY_AVAILABLE_BAND_MODE,
@@ -1815,6 +1881,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void sendTerminalResponse(String contents, Message response) {
         RILRequest rr = RILRequest.obtain(
                 RILConstants.RIL_REQUEST_STK_SEND_TERMINAL_RESPONSE, response);
@@ -1828,6 +1895,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void sendEnvelope(String contents, Message response) {
         RILRequest rr = RILRequest.obtain(
                 RILConstants.RIL_REQUEST_STK_SEND_ENVELOPE_COMMAND, response);
@@ -1841,6 +1909,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void sendEnvelopeWithStatus(String contents, Message response) {
         RILRequest rr = RILRequest.obtain(
                 RILConstants.RIL_REQUEST_STK_SEND_ENVELOPE_WITH_STATUS, response);
@@ -1855,6 +1924,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void handleCallSetupRequestFromSim(
             boolean accept, Message response) {
 
@@ -1883,6 +1953,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void setPreferredNetworkType(int networkType , Message response) {
         RILRequest rr = RILRequest.obtain(
                 RILConstants.RIL_REQUEST_SET_PREFERRED_NETWORK_TYPE, response);
@@ -1902,6 +1973,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void getPreferredNetworkType(Message response) {
         RILRequest rr = RILRequest.obtain(
                 RILConstants.RIL_REQUEST_GET_PREFERRED_NETWORK_TYPE, response);
@@ -1914,6 +1986,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void getNeighboringCids(Message response) {
         RILRequest rr = RILRequest.obtain(
                 RILConstants.RIL_REQUEST_GET_NEIGHBORING_CELL_IDS, response);
@@ -1926,6 +1999,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void setLocationUpdates(boolean enable, Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_SET_LOCATION_UPDATES, response);
         rr.mp.writeInt(1);
@@ -1940,6 +2014,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void getSmscAddress(Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_GET_SMSC_ADDRESS, result);
 
@@ -1951,6 +2026,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void setSmscAddress(String address, Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_SET_SMSC_ADDRESS, result);
 
@@ -1965,6 +2041,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void reportSmsMemoryStatus(boolean available, Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_REPORT_SMS_MEMORY_STATUS, result);
         rr.mp.writeInt(1);
@@ -1979,6 +2056,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void reportStkServiceIsRunning(Message result) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_REPORT_STK_SERVICE_IS_RUNNING, result);
 
@@ -1990,6 +2068,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void getGsmBroadcastConfig(Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_GSM_GET_BROADCAST_CONFIG, response);
 
@@ -2001,6 +2080,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void setGsmBroadcastConfig(SmsBroadcastConfigInfo[] config, Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_GSM_SET_BROADCAST_CONFIG, response);
 
@@ -2029,6 +2109,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void setGsmBroadcastActivation(boolean activate, Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_GSM_BROADCAST_ACTIVATION, response);
 
@@ -2053,6 +2134,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     protected void
     onRadioAvailable() {
         // In case screen state was lost (due to process crash),
@@ -2974,7 +3056,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
 
         response = p.readStringArray();
 
-        if (false) {
+        if (RILJ_LOGV) {
             num = p.readInt();
 
             response = new String[num];
@@ -3168,7 +3250,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
             dataCall.active = p.readInt();
             dataCall.type = p.readString();
             dataCall.ifname = p.readString();
-            if ((dataCall.status == DataConnection.FailCause.NONE.getErrorCode()) &&
+            if ((dataCall.status == DataConnectionBase.FailCause.NONE.getErrorCode()) &&
                     TextUtils.isEmpty(dataCall.ifname)) {
               throw new RuntimeException("getDataCallState, no ifname");
             }
@@ -3422,7 +3504,8 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         CdmaCallWaitingNotification notification = new CdmaCallWaitingNotification();
 
         notification.number = p.readString();
-        notification.numberPresentation = notification.presentationFromCLIP(p.readInt());
+        notification.numberPresentation =
+                CdmaCallWaitingNotification.presentationFromCLIP(p.readInt());
         notification.name = p.readString();
         notification.namePresentation = notification.numberPresentation;
         notification.isPresent = p.readInt();
@@ -3690,6 +3773,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
 
 
     // ***** Methods for CDMA support
+    @Override
     public void
     getDeviceIdentity(Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_DEVICE_IDENTITY, response);
@@ -3699,6 +3783,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void
     getCDMASubscription(Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_CDMA_SUBSCRIPTION, response);
@@ -3717,6 +3802,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void queryCdmaRoamingPreference(Message response) {
         RILRequest rr = RILRequest.obtain(
                 RILConstants.RIL_REQUEST_CDMA_QUERY_ROAMING_PREFERENCE, response);
@@ -3729,6 +3815,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void setCdmaRoamingPreference(int cdmaRoamingType, Message response) {
         RILRequest rr = RILRequest.obtain(
                 RILConstants.RIL_REQUEST_CDMA_SET_ROAMING_PREFERENCE, response);
@@ -3745,6 +3832,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void setCdmaSubscriptionSource(int cdmaSubscription , Message response) {
         RILRequest rr = RILRequest.obtain(
                 RILConstants.RIL_REQUEST_CDMA_SET_SUBSCRIPTION_SOURCE, response);
@@ -3774,6 +3862,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void queryTTYMode(Message response) {
         RILRequest rr = RILRequest.obtain(
                 RILConstants.RIL_REQUEST_QUERY_TTY_MODE, response);
@@ -3786,6 +3875,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void setTTYMode(int ttyMode, Message response) {
         RILRequest rr = RILRequest.obtain(
                 RILConstants.RIL_REQUEST_SET_TTY_MODE, response);
@@ -3802,6 +3892,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void
     sendCDMAFeatureCode(String FeatureCode, Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_CDMA_FLASH, response);
@@ -3814,12 +3905,14 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void getCdmaBroadcastConfig(Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_CDMA_GET_BROADCAST_CONFIG, response);
 
         send(rr);
     }
 
+    @Override
     public void setCdmaBroadcastConfig(CdmaSmsBroadcastConfigInfo[] configs, Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_CDMA_SET_BROADCAST_CONFIG, response);
 
@@ -3854,6 +3947,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void setCdmaBroadcastActivation(boolean activate, Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_CDMA_BROADCAST_ACTIVATION, response);
 
@@ -3868,6 +3962,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
     /**
      * {@inheritDoc}
      */
+    @Override
     public void exitEmergencyCallbackMode(Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_EXIT_EMERGENCY_CALLBACK_MODE, response);
 
@@ -3876,6 +3971,7 @@ public final class RIL extends BaseCommands implements CommandsInterface {
         send(rr);
     }
 
+    @Override
     public void requestIsimAuthentication(String nonce, Message response) {
         RILRequest rr = RILRequest.obtain(RIL_REQUEST_ISIM_AUTHENTICATION, response);
 
index ad2b88d..1c377e7 100644 (file)
@@ -16,8 +16,6 @@
 
 package com.android.internal.telephony;
 
-import android.telephony.ServiceState;
-
 public class RestrictedState {
 
     /**
index 7d9dc9c..81ad54d 100644 (file)
@@ -73,7 +73,7 @@ import java.util.ArrayList;
  * {@hide}
  */
 public class RetryManager {
-    static public final String LOG_TAG = "GSM";
+    static public final String LOG_TAG = "RetryManager";
     static public final boolean DBG = true;
     static public final boolean VDBG = false;
 
@@ -115,6 +115,7 @@ public class RetryManager {
         if (VDBG) log("constructor");
     }
 
+    @Override
     public String toString() {
         String ret = "RetryManager: forever=" + mRetryForever + ", maxRetry=" + mMaxRetryCount +
                 ", retry=" + mRetryCount + ",\n    " + mConfig;
index e781c3c..d804eeb 100644 (file)
@@ -61,7 +61,6 @@ import android.view.WindowManager;
 import android.widget.Button;
 import android.widget.CheckBox;
 import android.widget.CompoundButton;
-import android.widget.LinearLayout;
 import android.widget.TextView;
 
 import com.android.internal.R;
@@ -84,7 +83,8 @@ import static android.telephony.SmsManager.RESULT_ERROR_NULL_PDU;
 import static android.telephony.SmsManager.RESULT_ERROR_RADIO_OFF;
 
 public abstract class SMSDispatcher extends Handler {
-    static final String TAG = "SMS";    // accessed from inner class
+    static final String TAG = "SMSDispatcher";    // accessed from inner class
+    static final boolean DBG = false;
     private static final String SEND_NEXT_MSG_EXTRA = "SendNextMsg";
 
     /** Permission required to receive SMS and SMS-CB messages. */
@@ -298,9 +298,7 @@ public abstract class SMSDispatcher extends Handler {
         switch (msg.what) {
         case EVENT_NEW_SMS:
             // A new SMS has been received by the device
-            if (false) {
-                Rlog.d(TAG, "New SMS Message Received");
-            }
+            if (DBG) Rlog.d(TAG, "New SMS Message Received");
 
             SmsMessage sms;
 
@@ -431,10 +429,7 @@ public abstract class SMSDispatcher extends Handler {
         PendingIntent sentIntent = tracker.mSentIntent;
 
         if (ar.exception == null) {
-            if (false) {
-                Rlog.d(TAG, "SMS send complete. Broadcasting "
-                        + "intent: " + sentIntent);
-            }
+            if (DBG) Rlog.d(TAG, "SMS send complete. Broadcasting intent: " + sentIntent);
 
             if (tracker.mDeliveryIntent != null) {
                 // Expecting a status report.  Add it to the list.
@@ -459,9 +454,7 @@ public abstract class SMSDispatcher extends Handler {
                 } catch (CanceledException ex) {}
             }
         } else {
-            if (false) {
-                Rlog.d(TAG, "SMS send failed");
-            }
+            if (DBG) Rlog.d(TAG, "SMS send failed");
 
             int ss = mPhone.getServiceState().getState();
 
@@ -533,7 +526,7 @@ public abstract class SMSDispatcher extends Handler {
      * Dispatches an incoming SMS messages.
      *
      * @param sms the incoming message from the phone
-     * @return a result code from {@link Telephony.Sms.Intents}, or
+     * @return a result code from {@link android.provider.Telephony.Sms.Intents}, or
      *         {@link Activity#RESULT_OK} if the message has been broadcast
      *         to applications
      */
@@ -544,7 +537,7 @@ public abstract class SMSDispatcher extends Handler {
      * {@link #dispatchMessage(SmsMessageBase)} if no format-specific handling is required.
      *
      * @param sms
-     * @return
+     * @return {@link Activity#RESULT_OK} on success
      */
     protected int dispatchNormalMessage(SmsMessageBase sms) {
         SmsHeader smsHeader = sms.getUserDataHeader();
@@ -582,7 +575,7 @@ public abstract class SMSDispatcher extends Handler {
      * If this is the last part send the parts out to the application, otherwise
      * the part is stored for later processing. Handles both 3GPP concatenated messages
      * as well as 3GPP2 format WAP push messages processed by
-     * {@link com.android.internal.telephony.cdma.CdmaSMSDispatcher#processCdmaWapPdu}.
+     * com.android.internal.telephony.cdma.CdmaSMSDispatcher#processCdmaWapPdu.
      *
      * @param pdu the message PDU, or the datagram portion of a CDMA WDP datagram segment
      * @param address the originating address
@@ -594,7 +587,7 @@ public abstract class SMSDispatcher extends Handler {
      * @param destPort the destination port for the message, or -1 for no destination port
      * @param isCdmaWapPush true if pdu is a CDMA WDP datagram segment and not an SM PDU
      *
-     * @return a result code from {@link Telephony.Sms.Intents}, or
+     * @return a result code from {@link android.provider.Telephony.Sms.Intents}, or
      *         {@link Activity#RESULT_OK} if the message has been broadcast
      *         to applications
      */
@@ -1050,7 +1043,7 @@ public abstract class SMSDispatcher extends Handler {
                     networkCountryIso = mTelephonyManager.getSimCountryIso();
                 }
 
-                smsCategory = mUsageMonitor.mergeShortCodeCategories(smsCategory,
+                smsCategory = SmsUsageMonitor.mergeShortCodeCategories(smsCategory,
                         mUsageMonitor.checkDestination(tracker.mDestAddress, networkCountryIso));
             }
 
@@ -1343,7 +1336,7 @@ public abstract class SMSDispatcher extends Handler {
          * @return true if the tracker holds a multi-part SMS; false otherwise
          */
         protected boolean isMultipart() {
-            HashMap map = mData;
+            HashMap<String, Object> map = mData;
             return map.containsKey("parts");
         }
     }
index 5a37a48..60e73cf 100644 (file)
 
 package com.android.internal.telephony;
 
-import android.content.Context;
 import android.os.AsyncResult;
 import android.os.Handler;
-import android.os.Looper;
 import android.os.Message;
 import android.os.Registrant;
 import android.os.RegistrantList;
-import android.os.SystemClock;
 import android.telephony.CellInfo;
 import android.telephony.ServiceState;
 import android.telephony.SignalStrength;
@@ -33,7 +30,7 @@ import java.io.FileDescriptor;
 import java.io.PrintWriter;
 import java.util.List;
 
-import com.android.internal.telephony.dataconnection.DataConnectionTracker;
+import com.android.internal.telephony.dataconnection.DataConnectionTrackerBase;
 import com.android.internal.telephony.uicc.IccCardApplicationStatus.AppState;
 import com.android.internal.telephony.uicc.IccRecords;
 import com.android.internal.telephony.uicc.UiccCardApplication;
@@ -457,7 +454,7 @@ public abstract class ServiceStateTracker extends Handler {
      *
      * Hang up the existing voice calls to decrease call drop rate.
      */
-    public void powerOffRadioSafely(DataConnectionTracker dcTracker) {
+    public void powerOffRadioSafely(DataConnectionTrackerBase dcTracker) {
         synchronized (this) {
             if (!mPendingRadioPowerOffAfterDataOff) {
                 // To minimize race conditions we call cleanUpAllConnections on
index 22d8cd8..105566c 100644 (file)
@@ -27,8 +27,6 @@ import android.provider.Telephony;
  * {@hide}
  */
 public abstract class SmsMessageBase {
-    private static final String LOG_TAG = "SMS";
-
     /** {@hide} The address of the SMSC. May be null */
     protected String scAddress;
 
@@ -91,6 +89,7 @@ public abstract class SmsMessageBase {
         public byte[] encodedScAddress; // Null if not applicable.
         public byte[] encodedMessage;
 
+        @Override
         public String toString() {
             return "SubmitPdu: encodedScAddress = "
                     + Arrays.toString(encodedScAddress)
index a7c2840..633a53e 100644 (file)
@@ -38,6 +38,7 @@ public class SmsResponse {
         this.errorCode = errorCode;
     }
 
+    @Override
     public String toString() {
         String ret = "{ messageRef = " + messageRef
                         + ", errorCode = " + errorCode
index 359069c..aa9f4e5 100644 (file)
@@ -24,7 +24,6 @@ import android.content.res.XmlResourceParser;
 import android.database.ContentObserver;
 import android.os.Binder;
 import android.os.Handler;
-import android.os.Message;
 import android.os.Process;
 import android.os.RemoteException;
 import android.os.UserHandle;
@@ -39,7 +38,6 @@ import com.android.internal.util.XmlUtils;
 
 import org.xmlpull.v1.XmlPullParser;
 import org.xmlpull.v1.XmlPullParserException;
-import org.xmlpull.v1.XmlPullParserFactory;
 import org.xmlpull.v1.XmlSerializer;
 
 import java.io.File;
@@ -48,7 +46,6 @@ import java.io.FileNotFoundException;
 import java.io.FileOutputStream;
 import java.io.FileReader;
 import java.io.IOException;
-import java.io.StringReader;
 import java.util.ArrayList;
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.HashMap;
@@ -128,9 +125,6 @@ public class SmsUsageMonitor {
     /** Notice when the enabled setting changes - can be changed through gservices */
     private final AtomicBoolean mCheckEnabled = new AtomicBoolean(true);
 
-    /** Cached short code regex patterns from secure settings for {@link #mCurrentCountry}. */
-    private String mSettingsShortCodePatterns;
-
     /** Handler for responding to content observer updates. */
     private final SettingsObserverHandler mSettingsObserverHandler;
 
index f374f41..bbef33e 100644 (file)
@@ -163,7 +163,7 @@ public class TelephonyCapabilities {
      * (Abbreviated Dialing Numbers).
      *
      * Currently this returns true when the phone type is GSM
-     * ({@link Phone#PHONE_TYPE_GSM}).
+     * ({@link PhoneConstants#PHONE_TYPE_GSM}).
      *
      * This is using int for an argument for letting apps outside
      * Phone process access to it, while other methods in this class is
index 5a47822..8558b2e 100755 (executable)
 
 package com.android.internal.telephony;
 
-import android.Manifest;
 import android.app.Activity;
 import android.app.AppOpsManager;
 import android.content.Context;
 import android.content.ComponentName;
 import android.content.Intent;
 import android.content.ServiceConnection;
-import android.provider.Telephony;
 import android.provider.Telephony.Sms.Intents;
 import android.telephony.Rlog;
 import android.os.IBinder;
@@ -39,17 +37,12 @@ import com.android.internal.telephony.uicc.IccUtils;
  */
 public class WapPushOverSms {
     private static final String LOG_TAG = "WAP PUSH";
+    private static final boolean DBG = false;
 
     private final Context mContext;
     private WspTypeDecoder pduDecoder;
     private SMSDispatcher mSmsDispatcher;
 
-    /**
-     * Hold the wake lock for 5 seconds, which should be enough time for
-     * any receiver(s) to grab its own wake lock.
-     */
-    private final int WAKE_LOCK_TIMEOUT = 5000;
-
     private final int BIND_RETRY_INTERVAL = 1000;
     /**
      * A handle to WapPushManager interface
@@ -63,15 +56,17 @@ public class WapPushOverSms {
             mOwner = ownerContext;
         }
 
+        @Override
         public void onServiceConnected(ComponentName name, IBinder service) {
             mWapPushMan = IWapPushManager.Stub.asInterface(service);
-            if (false) Rlog.v(LOG_TAG, "wappush manager connected to " +
+            if (DBG) Rlog.v(LOG_TAG, "wappush manager connected to " +
                     mOwner.hashCode());
         }
 
+        @Override
         public void onServiceDisconnected(ComponentName name) {
             mWapPushMan = null;
-            if (false) Rlog.v(LOG_TAG, "wappush manager disconnected.");
+            if (DBG) Rlog.v(LOG_TAG, "wappush manager disconnected.");
             // Detach the previous binder
             mOwner.unbindService(mWapConn);
             // WapPushManager must be always attached.
@@ -99,6 +94,7 @@ public class WapPushOverSms {
 
             final ServiceConnection wapPushConnection = this;
             new Thread() {
+                @Override
                 public void run() {
                     while (mWapPushMan == null) {
                         mOwner.bindService(new Intent(IWapPushManager.class.getName()),
@@ -106,7 +102,7 @@ public class WapPushOverSms {
                         try {
                             Thread.sleep(BIND_RETRY_INTERVAL);
                         } catch (InterruptedException e) {
-                            if (false) Rlog.v(LOG_TAG, "sleep interrupted.");
+                            if (DBG) Rlog.v(LOG_TAG, "sleep interrupted.");
                         }
                     }
                 }
@@ -134,13 +130,13 @@ public class WapPushOverSms {
      * wap-230-wsp-20010705-a section 8 for details on the WAP PDU format.
      *
      * @param pdu The WAP PDU, made up of one or more SMS PDUs
-     * @return a result code from {@link Telephony.Sms.Intents}, or
+     * @return a result code from {@link android.provider.Telephony.Sms.Intents}, or
      *         {@link Activity#RESULT_OK} if the message has been broadcast
      *         to applications
      */
     public int dispatchWapPdu(byte[] pdu) {
 
-        if (false) Rlog.d(LOG_TAG, "Rx: " + IccUtils.bytesToHexString(pdu));
+        if (DBG) Rlog.d(LOG_TAG, "Rx: " + IccUtils.bytesToHexString(pdu));
 
         int index = 0;
         int transactionId = pdu[index++] & 0xFF;
@@ -149,7 +145,7 @@ public class WapPushOverSms {
 
         if ((pduType != WspTypeDecoder.PDU_TYPE_PUSH) &&
                 (pduType != WspTypeDecoder.PDU_TYPE_CONFIRMED_PUSH)) {
-            if (false) Rlog.w(LOG_TAG, "Received non-PUSH WAP PDU. Type = " + pduType);
+            if (DBG) Rlog.w(LOG_TAG, "Received non-PUSH WAP PDU. Type = " + pduType);
             return Intents.RESULT_SMS_HANDLED;
         }
 
@@ -162,7 +158,7 @@ public class WapPushOverSms {
          * So it will be encoded in no more than 5 octets.
          */
         if (pduDecoder.decodeUintvarInteger(index) == false) {
-            if (false) Rlog.w(LOG_TAG, "Received PDU. Header Length error.");
+            if (DBG) Rlog.w(LOG_TAG, "Received PDU. Header Length error.");
             return Intents.RESULT_SMS_GENERIC_ERROR;
         }
         headerLength = (int)pduDecoder.getValue32();
@@ -183,7 +179,7 @@ public class WapPushOverSms {
          * Length = Uintvar-integer
          */
         if (pduDecoder.decodeContentType(index) == false) {
-            if (false) Rlog.w(LOG_TAG, "Received PDU. Header Content-Type error.");
+            if (DBG) Rlog.w(LOG_TAG, "Received PDU. Header Content-Type error.");
             return Intents.RESULT_SMS_GENERIC_ERROR;
         }
 
@@ -220,14 +216,14 @@ public class WapPushOverSms {
 
             String contentType = ((mimeType == null) ?
                                   Long.toString(binaryContentType) : mimeType);
-            if (false) Rlog.v(LOG_TAG, "appid found: " + wapAppId + ":" + contentType);
+            if (DBG) Rlog.v(LOG_TAG, "appid found: " + wapAppId + ":" + contentType);
 
             try {
                 boolean processFurther = true;
                 IWapPushManager wapPushMan = mWapConn.getWapPushManager();
 
                 if (wapPushMan == null) {
-                    if (false) Rlog.w(LOG_TAG, "wap push manager not found!");
+                    if (DBG) Rlog.w(LOG_TAG, "wap push manager not found!");
                 } else {
                     Intent intent = new Intent();
                     intent.putExtra("transactionId", transactionId);
@@ -238,7 +234,7 @@ public class WapPushOverSms {
                             pduDecoder.getContentParameters());
 
                     int procRet = wapPushMan.processMessage(wapAppId, contentType, intent);
-                    if (false) Rlog.v(LOG_TAG, "procRet:" + procRet);
+                    if (DBG) Rlog.v(LOG_TAG, "procRet:" + procRet);
                     if ((procRet & WapPushManagerParams.MESSAGE_HANDLED) > 0
                         && (procRet & WapPushManagerParams.FURTHER_PROCESSING) == 0) {
                         processFurther = false;
@@ -248,13 +244,13 @@ public class WapPushOverSms {
                     return Intents.RESULT_SMS_HANDLED;
                 }
             } catch (RemoteException e) {
-                if (false) Rlog.w(LOG_TAG, "remote func failed...");
+                if (DBG) Rlog.w(LOG_TAG, "remote func failed...");
             }
         }
-        if (false) Rlog.v(LOG_TAG, "fall back to existing handler");
+        if (DBG) Rlog.v(LOG_TAG, "fall back to existing handler");
 
         if (mimeType == null) {
-            if (false) Rlog.w(LOG_TAG, "Header Content-Type error.");
+            if (DBG) Rlog.w(LOG_TAG, "Header Content-Type error.");
             return Intents.RESULT_SMS_GENERIC_ERROR;
         }
 
index 9284d03..8a1982f 100644 (file)
@@ -95,6 +95,11 @@ public class CatCmdMessage implements Parcelable {
             BIPClientParams param = (BIPClientParams) cmdParams;
             mTextMsg = param.textMsg;
             break;
+        case PROVIDE_LOCAL_INFORMATION:
+        case REFRESH:
+        case SET_UP_EVENT_LIST:
+        default:
+            break;
         }
     }
 
@@ -117,9 +122,12 @@ public class CatCmdMessage implements Parcelable {
             mCallSettings.confirmMsg = in.readParcelable(null);
             mCallSettings.callMsg = in.readParcelable(null);
             break;
+        default:
+            break;
         }
     }
 
+    @Override
     public void writeToParcel(Parcel dest, int flags) {
         dest.writeParcelable(mCmdDet, 0);
         dest.writeParcelable(mTextMsg, 0);
@@ -137,19 +145,24 @@ public class CatCmdMessage implements Parcelable {
             dest.writeParcelable(mCallSettings.confirmMsg, 0);
             dest.writeParcelable(mCallSettings.callMsg, 0);
             break;
+        default:
+            break;
         }
     }
 
     public static final Parcelable.Creator<CatCmdMessage> CREATOR = new Parcelable.Creator<CatCmdMessage>() {
+        @Override
         public CatCmdMessage createFromParcel(Parcel in) {
             return new CatCmdMessage(in);
         }
 
+        @Override
         public CatCmdMessage[] newArray(int size) {
             return new CatCmdMessage[size];
         }
     };
 
+    @Override
     public int describeContents() {
         return 0;
     }
index 4eacae3..a513c23 100644 (file)
@@ -62,6 +62,7 @@ class RilMessage {
  * {@hide}
  */
 public class CatService extends Handler implements AppInterface {
+    private static final boolean DBG = false;
 
     // Class members
     private static IccRecords mIccRecords;
@@ -94,12 +95,8 @@ public class CatService extends Handler implements AppInterface {
     private static final int MSG_ID_ICC_RECORDS_LOADED       = 20;
 
     private static final int DEV_ID_KEYPAD      = 0x01;
-    private static final int DEV_ID_DISPLAY     = 0x02;
-    private static final int DEV_ID_EARPIECE    = 0x03;
     private static final int DEV_ID_UICC        = 0x81;
     private static final int DEV_ID_TERMINAL    = 0x82;
-    private static final int DEV_ID_NETWORK     = 0x83;
-
     static final String STK_DEFAULT = "Defualt Message";
 
     /* Intentionally private for singleton */
@@ -146,6 +143,7 @@ public class CatService extends Handler implements AppInterface {
         this.removeCallbacksAndMessages(null);
     }
 
+    @Override
     protected void finalize() {
         CatLog.d(this, "Service finalized");
     }
@@ -405,7 +403,7 @@ public class CatService extends Handler implements AppInterface {
 
         byte[] rawData = buf.toByteArray();
         String hexString = IccUtils.bytesToHexString(rawData);
-        if (false) {
+        if (DBG) {
             CatLog.d(this, "TERMINAL RESPONSE: " + hexString);
         }
 
@@ -506,56 +504,11 @@ public class CatService extends Handler implements AppInterface {
         mCmdIf.sendEnvelope(hexString, null);
     }
 
-    private void eventDownload(int event, int sourceId, int destinationId,
-            byte[] additionalInfo, boolean oneShot) {
-
-        ByteArrayOutputStream buf = new ByteArrayOutputStream();
-
-        // tag
-        int tag = BerTlv.BER_EVENT_DOWNLOAD_TAG;
-        buf.write(tag);
-
-        // length
-        buf.write(0x00); // place holder, assume length < 128.
-
-        // event list
-        tag = 0x80 | ComprehensionTlvTag.EVENT_LIST.value();
-        buf.write(tag);
-        buf.write(0x01); // length
-        buf.write(event); // event value
-
-        // device identities
-        tag = 0x80 | ComprehensionTlvTag.DEVICE_IDENTITIES.value();
-        buf.write(tag);
-        buf.write(0x02); // length
-        buf.write(sourceId); // source device id
-        buf.write(destinationId); // destination device id
-
-        // additional information
-        if (additionalInfo != null) {
-            for (byte b : additionalInfo) {
-                buf.write(b);
-            }
-        }
-
-        byte[] rawData = buf.toByteArray();
-
-        // write real length
-        int len = rawData.length - 2; // minus (tag + length)
-        rawData[1] = (byte) len;
-
-        String hexString = IccUtils.bytesToHexString(rawData);
-
-        mCmdIf.sendEnvelope(hexString, null);
-    }
-
     /**
      * Used for instantiating/updating the Service from the GsmPhone or CdmaPhone constructor.
      *
      * @param ci CommandsInterface object
-     * @param ir IccRecords object
      * @param context phone app context
-     * @param fh Icc file handler
      * @param ic Icc card
      * @return The only Service object in the system
      */
@@ -659,6 +612,7 @@ public class CatService extends Handler implements AppInterface {
         }
     }
 
+    @Override
     public synchronized void onCmdResponse(CatResponseMessage resMsg) {
         if (resMsg == null) {
             return;
@@ -753,6 +707,8 @@ public class CatService extends Handler implements AppInterface {
                 // invoked by the CommandInterface call above.
                 mCurrntCmd = null;
                 return;
+            default:
+                break;
             }
             break;
         case NO_RESPONSE_FROM_USER:
index 3e7f722..aaa9d68 100644 (file)
@@ -24,7 +24,7 @@ abstract class ValueObject {
 }
 
 /**
- * Class for Command Detailes object of proactive commands from SIM.
+ * Class for Command Details object of proactive commands from SIM.
  * {@hide}
  */
 class CommandDetails extends ValueObject implements Parcelable {
@@ -33,6 +33,7 @@ class CommandDetails extends ValueObject implements Parcelable {
     public int typeOfCommand;
     public int commandQualifier;
 
+    @Override
     public ComprehensionTlvTag getTag() {
         return ComprehensionTlvTag.COMMAND_DETAILS;
     }
@@ -54,6 +55,7 @@ class CommandDetails extends ValueObject implements Parcelable {
         commandQualifier = in.readInt();
     }
 
+    @Override
     public void writeToParcel(Parcel dest, int flags) {
         dest.writeInt(compRequired ? 1 : 0);
         dest.writeInt(commandNumber);
@@ -63,15 +65,18 @@ class CommandDetails extends ValueObject implements Parcelable {
 
     public static final Parcelable.Creator<CommandDetails> CREATOR =
                                 new Parcelable.Creator<CommandDetails>() {
+        @Override
         public CommandDetails createFromParcel(Parcel in) {
             return new CommandDetails(in);
         }
 
+        @Override
         public CommandDetails[] newArray(int size) {
             return new CommandDetails[size];
         }
     };
 
+    @Override
     public int describeContents() {
         return 0;
     }
@@ -89,6 +94,7 @@ class DeviceIdentities extends ValueObject {
     public int sourceId;
     public int destinationId;
 
+    @Override
     ComprehensionTlvTag getTag() {
         return ComprehensionTlvTag.DEVICE_IDENTITIES;
     }
@@ -99,6 +105,7 @@ class IconId extends ValueObject {
     int recordNumber;
     boolean selfExplanatory;
 
+    @Override
     ComprehensionTlvTag getTag() {
         return ComprehensionTlvTag.ICON_ID;
     }
@@ -109,6 +116,7 @@ class ItemsIconId extends ValueObject {
     int [] recordNumbers;
     boolean selfExplanatory;
 
+    @Override
     ComprehensionTlvTag getTag() {
         return ComprehensionTlvTag.ITEM_ICON_ID_LIST;
     }
index 79f6ad2..8c04e37 100644 (file)
@@ -49,6 +49,7 @@ class DisplayTextParams extends CommandParams {
         this.textMsg = textMsg;
     }
 
+    @Override
     boolean setIcon(Bitmap icon) {
         if (icon != null && textMsg != null) {
             textMsg.icon = icon;
@@ -71,6 +72,7 @@ class LaunchBrowserParams extends CommandParams {
         this.url = url;
     }
 
+    @Override
     boolean setIcon(Bitmap icon) {
         if (icon != null && confirmMsg != null) {
             confirmMsg.icon = icon;
@@ -91,6 +93,7 @@ class PlayToneParams extends CommandParams {
         this.settings = new ToneSettings(duration, tone, vibrate);
     }
 
+    @Override
     boolean setIcon(Bitmap icon) {
         if (icon != null && textMsg != null) {
             textMsg.icon = icon;
@@ -111,6 +114,7 @@ class CallSetupParams extends CommandParams {
         this.callMsg = callMsg;
     }
 
+    @Override
     boolean setIcon(Bitmap icon) {
         if (icon == null) {
             return false;
@@ -136,6 +140,7 @@ class SelectItemParams extends CommandParams {
         this.loadTitleIcon = loadTitleIcon;
     }
 
+    @Override
     boolean setIcon(Bitmap icon) {
         if (icon != null && menu != null) {
             if (loadTitleIcon && menu.titleIcon == null) {
@@ -163,6 +168,7 @@ class GetInputParams extends CommandParams {
         this.input = input;
     }
 
+    @Override
     boolean setIcon(Bitmap icon) {
         if (icon != null && input != null) {
             input.icon = icon;
@@ -189,6 +195,7 @@ class BIPClientParams extends CommandParams {
         this.bHasAlphaId = has_alpha_id;
     }
 
+    @Override
     boolean setIcon(Bitmap icon) {
         if (icon != null && textMsg != null) {
             textMsg.icon = icon;
index 6e3b19f..75e8513 100644 (file)
@@ -663,33 +663,6 @@ class CommandParamsFactory extends Handler {
     }
 
     /**
-     * Processes SET_UP_EVENT_LIST proactive command from the SIM card.
-     *
-     * @param cmdDet Command Details object retrieved.
-     * @param ctlvs List of ComprehensionTlv objects following Command Details
-     *        object and Device Identities object within the proactive command
-     * @return true if the command is processing is pending and additional
-     *         asynchronous processing is required.
-     */
-    private boolean processSetUpEventList(CommandDetails cmdDet,
-            List<ComprehensionTlv> ctlvs) {
-
-        CatLog.d(this, "process SetUpEventList");
-        //
-        // ComprehensionTlv ctlv = searchForTag(ComprehensionTlvTag.EVENT_LIST,
-        // ctlvs);
-        // if (ctlv != null) {
-        // try {
-        // byte[] rawValue = ctlv.getRawValue();
-        // int valueIndex = ctlv.getValueIndex();
-        // int valueLen = ctlv.getLength();
-        //
-        // } catch (IndexOutOfBoundsException e) {}
-        // }
-        return true;
-    }
-
-    /**
      * Processes LAUNCH_BROWSER proactive command from the SIM card.
      *
      * @param cmdDet Command Details container object.
index e8cd404..e437f6e 100644 (file)
@@ -58,20 +58,24 @@ public class Duration implements Parcelable {
         timeUnit = TimeUnit.values()[in.readInt()];
     }
 
+    @Override
     public void writeToParcel(Parcel dest, int flags) {
         dest.writeInt(timeInterval);
         dest.writeInt(timeUnit.ordinal());
     }
 
+    @Override
     public int describeContents() {
         return 0;
     }
 
     public static final Parcelable.Creator<Duration> CREATOR = new Parcelable.Creator<Duration>() {
+        @Override
         public Duration createFromParcel(Parcel in) {
             return new Duration(in);
         }
 
+        @Override
         public Duration[] newArray(int size) {
             return new Duration[size];
         }
index c339f5e..e2f0172 100644 (file)
@@ -25,8 +25,6 @@ import android.os.Handler;
 import android.os.HandlerThread;
 import android.os.Looper;
 import android.os.Message;
-import android.telephony.Rlog;
-
 import java.util.HashMap;
 
 /**
@@ -173,7 +171,7 @@ class IconLoader extends Handler {
      * Handles Image descriptor parsing and required processing. This is the
      * first step required to handle retrieving icons from the SIM.
      *
-     * @param data byte [] containing Image Instance descriptor as defined in
+     * @param rawData byte [] containing Image Instance descriptor as defined in
      * TS 51.011.
      */
     private boolean handleImageDescriptor(byte[] rawData) {
@@ -184,7 +182,7 @@ class IconLoader extends Handler {
         return true;
     }
 
-    // Start reading colour lookup table from SIM card.
+    // Start reading color lookup table from SIM card.
     private void readClut() {
         int length = mIconData[3] * CLUT_ENTRY_SIZE;
         Message msg = this.obtainMessage(EVENT_READ_CLUT_DONE);
index 13a5ad4..48705d1 100644 (file)
@@ -68,10 +68,12 @@ public class Input implements Parcelable {
         duration = in.readParcelable(null);
     }
 
+    @Override
     public int describeContents() {
         return 0;
     }
 
+    @Override
     public void writeToParcel(Parcel dest, int flags) {
         dest.writeString(text);
         dest.writeString(defaultText);
@@ -88,10 +90,12 @@ public class Input implements Parcelable {
     }
 
     public static final Parcelable.Creator<Input> CREATOR = new Parcelable.Creator<Input>() {
+        @Override
         public Input createFromParcel(Parcel in) {
             return new Input(in);
         }
 
+        @Override
         public Input[] newArray(int size) {
             return new Input[size];
         }
index d4702bb..f2513b6 100644 (file)
@@ -45,10 +45,12 @@ public class Item implements Parcelable {
         icon = in.readParcelable(null);
     }
 
+    @Override
     public int describeContents() {
         return 0;
     }
 
+    @Override
     public void writeToParcel(Parcel dest, int flags) {
         dest.writeInt(id);
         dest.writeString(text);
@@ -56,15 +58,18 @@ public class Item implements Parcelable {
     }
 
     public static final Parcelable.Creator<Item> CREATOR = new Parcelable.Creator<Item>() {
+        @Override
         public Item createFromParcel(Parcel in) {
             return new Item(in);
         }
 
+        @Override
         public Item[] newArray(int size) {
             return new Item[size];
         }
     };
 
+    @Override
     public String toString() {
         return text;
     }
index 7bbae01..a3cbbdf 100644 (file)
@@ -72,10 +72,12 @@ public class Menu implements Parcelable {
         presentationType = PresentationType.values()[in.readInt()];
     }
 
+    @Override
     public int describeContents() {
         return 0;
     }
 
+    @Override
     public void writeToParcel(Parcel dest, int flags) {
         dest.writeString(title);
         dest.writeParcelable(titleIcon, flags);
@@ -94,10 +96,12 @@ public class Menu implements Parcelable {
     }
 
     public static final Parcelable.Creator<Menu> CREATOR = new Parcelable.Creator<Menu>() {
+        @Override
         public Menu createFromParcel(Parcel in) {
             return new Menu(in);
         }
 
+        @Override
         public Menu[] newArray(int size) {
             return new Menu[size];
         }
index 8544175..f001c7e 100644 (file)
@@ -19,7 +19,7 @@ package com.android.internal.telephony.cat;
 
 /**
  * Enumeration for the return code in TERMINAL RESPONSE.
- * To get the actual return code for each enum value, call {@link #code() code}
+ * To get the actual return code for each enum value, call {@link #value}
  * method.
  *
  * {@hide}
index 84879c2..5175b83 100644 (file)
@@ -45,6 +45,8 @@ public class ResultException extends CatException {
                 throw new AssertionError(
                         "For result code, " + result +
                         ", additional information must be given!");
+            default:
+                break;
         }
 
         mResult = result;
index 5ffd076..515f6c5 100644 (file)
@@ -44,10 +44,12 @@ public class TextMessage implements Parcelable {
         duration = in.readParcelable(null);
     }
 
+    @Override
     public int describeContents() {
         return 0;
     }
 
+    @Override
     public void writeToParcel(Parcel dest, int flags) {
         dest.writeString(title);
         dest.writeString(text);
@@ -60,10 +62,12 @@ public class TextMessage implements Parcelable {
     }
 
     public static final Parcelable.Creator<TextMessage> CREATOR = new Parcelable.Creator<TextMessage>() {
+        @Override
         public TextMessage createFromParcel(Parcel in) {
             return new TextMessage(in);
         }
 
+        @Override
         public TextMessage[] newArray(int size) {
             return new TextMessage[size];
         }
index 27b4489..eea7fed 100644 (file)
@@ -170,19 +170,23 @@ public enum Tone implements Parcelable {
         mValue = in.readInt();
     }
 
+    @Override
     public void writeToParcel(Parcel dest, int flags) {
         dest.writeInt(ordinal());
     }
 
+    @Override
     public int describeContents() {
         return 0;
     }
 
     public static final Parcelable.Creator<Tone> CREATOR = new Parcelable.Creator<Tone>() {
+        @Override
         public Tone createFromParcel(Parcel in) {
             return Tone.values()[in.readInt()];
         }
 
+        @Override
         public Tone[] newArray(int size) {
             return new Tone[size];
         }
index 6375afb..61c1573 100644 (file)
@@ -40,10 +40,12 @@ public class ToneSettings implements Parcelable {
         vibrate = in.readInt() == 1;
     }
 
+    @Override
     public int describeContents() {
         return 0;
     }
 
+    @Override
     public void writeToParcel(Parcel dest, int flags) {
         dest.writeParcelable(duration, 0);
         dest.writeParcelable(tone, 0);
@@ -51,10 +53,12 @@ public class ToneSettings implements Parcelable {
     }
 
     public static final Parcelable.Creator<ToneSettings> CREATOR = new Parcelable.Creator<ToneSettings>() {
+        @Override
         public ToneSettings createFromParcel(Parcel in) {
             return new ToneSettings(in);
         }
 
+        @Override
         public ToneSettings[] newArray(int size) {
             return new ToneSettings[size];
         }
index 5b1ae91..6f655ed 100644 (file)
@@ -29,7 +29,7 @@ abstract class ValueParser {
     /**
      * Search for a Command Details object from a list.
      *
-     * @param ctlvs List of ComprehensionTlv objects used for search
+     * @param ctlv List of ComprehensionTlv objects used for search
      * @return An CtlvCommandDetails object found from the objects. If no
      *         Command Details object is found, ResultException is thrown.
      * @throws ResultException
@@ -54,7 +54,7 @@ abstract class ValueParser {
     /**
      * Search for a Device Identities object from a list.
      *
-     * @param ctlvs List of ComprehensionTlv objects used for search
+     * @param ctlv List of ComprehensionTlv objects used for search
      * @return An CtlvDeviceIdentities object found from the objects. If no
      *         Command Details object is found, ResultException is thrown.
      * @throws ResultException
index 892c89c..97f3da2 100644 (file)
@@ -29,7 +29,6 @@ import android.telephony.Rlog;
 
 import com.android.internal.telephony.CommandsInterface;
 import com.android.internal.telephony.OperatorInfo;
-import com.android.internal.telephony.Phone;
 import com.android.internal.telephony.PhoneConstants;
 import com.android.internal.telephony.PhoneNotifier;
 import com.android.internal.telephony.PhoneProxy;
@@ -46,8 +45,7 @@ import java.io.FileDescriptor;
 import java.io.PrintWriter;
 
 public class CDMALTEPhone extends CDMAPhone {
-    static final String LOG_TAG = "CDMA";
-
+    static final String LOG_LTE_TAG = "CDMALTEPhone";
     private static final boolean DBG = true;
 
     /** Secondary SMSDispatcher for 3GPP format messages. */
@@ -176,7 +174,7 @@ public class CDMALTEPhone extends CDMAPhone {
         // look for our wrapper within the asyncresult, skip the rest if it
         // is null.
         if (!(ar.userObj instanceof NetworkSelectMessage)) {
-            Rlog.e(LOG_TAG, "unexpected result from user object.");
+            loge("unexpected result from user object.");
             return;
         }
 
@@ -199,7 +197,7 @@ public class CDMALTEPhone extends CDMAPhone {
 
         // commit and log the result.
         if (! editor.commit()) {
-            Rlog.e(LOG_TAG, "failed to commit network selection preference");
+            loge("failed to commit network selection preference");
         }
 
     }
@@ -217,7 +215,7 @@ public class CDMALTEPhone extends CDMAPhone {
                 mContext.getContentResolver().insert(uri, map);
                 return true;
             } catch (SQLException e) {
-                Rlog.e(LOG_TAG, "[CDMALTEPhone] Can't store current operator ret false", e);
+                loge("Can't store current operator ret false", e);
             }
         } else {
             if (DBG) log("updateCurrentCarrierInProvider mIccRecords == null ret false");
@@ -301,9 +299,17 @@ public class CDMALTEPhone extends CDMAPhone {
 
     @Override
     protected void log(String s) {
-            Rlog.d(LOG_TAG, "[CDMALTEPhone] " + s);
+            Rlog.d(LOG_LTE_TAG, s);
+    }
+
+    protected void loge(String s) {
+            Rlog.e(LOG_LTE_TAG, s);
     }
 
+    protected void loge(String s, Throwable e) {
+        Rlog.e(LOG_LTE_TAG, s, e);
+}
+
     @Override
     public void dump(FileDescriptor fd, PrintWriter pw, String[] args) {
         pw.println("CDMALTEPhone extends:");
index 3ab4780..7ae933c 100755 (executable)
@@ -37,23 +37,19 @@ import android.provider.Telephony;
 import android.telephony.CellLocation;
 import android.telephony.PhoneNumberUtils;
 import android.telephony.ServiceState;
-import android.telephony.SignalStrength;
 import android.text.TextUtils;
 import android.telephony.Rlog;
 
-import com.android.internal.telephony.Call;
 import com.android.internal.telephony.CallStateException;
 import com.android.internal.telephony.CallTracker;
 import com.android.internal.telephony.CommandException;
 import com.android.internal.telephony.CommandsInterface;
 import com.android.internal.telephony.Connection;
-import com.android.internal.telephony.IccCard;
 import com.android.internal.telephony.IccPhoneBookInterfaceManager;
 import com.android.internal.telephony.IccSmsInterfaceManager;
 import com.android.internal.telephony.MccTable;
 import com.android.internal.telephony.MmiCode;
 import com.android.internal.telephony.OperatorInfo;
-import com.android.internal.telephony.Phone;
 import com.android.internal.telephony.PhoneBase;
 import com.android.internal.telephony.PhoneConstants;
 import com.android.internal.telephony.PhoneNotifier;
@@ -63,17 +59,12 @@ import com.android.internal.telephony.ServiceStateTracker;
 import com.android.internal.telephony.TelephonyIntents;
 import com.android.internal.telephony.TelephonyProperties;
 import com.android.internal.telephony.UUSInfo;
-import com.android.internal.telephony.cat.CatService;
-import com.android.internal.telephony.dataconnection.GsmDataConnectionTracker;
+import com.android.internal.telephony.dataconnection.DataConnectionTracker;
 import com.android.internal.telephony.uicc.IccException;
-import com.android.internal.telephony.uicc.IccFileHandler;
 import com.android.internal.telephony.uicc.IccRecords;
 import com.android.internal.telephony.uicc.RuimRecords;
-import com.android.internal.telephony.uicc.UiccCard;
 import com.android.internal.telephony.uicc.UiccCardApplication;
 import com.android.internal.telephony.uicc.UiccController;
-import com.android.internal.telephony.uicc.IccCardApplicationStatus.AppState;
-
 import java.io.FileDescriptor;
 import java.io.PrintWriter;
 import java.util.ArrayList;
@@ -89,7 +80,7 @@ import static com.android.internal.telephony.TelephonyProperties.PROPERTY_ICC_OP
  * {@hide}
  */
 public class CDMAPhone extends PhoneBase {
-    static final String LOG_TAG = "CDMA";
+    static final String LOG_TAG = "CDMAPhone";
     private static final boolean DBG = true;
     private static final boolean VDBG = false; /* STOP SHIP if true */
 
@@ -169,7 +160,7 @@ public class CDMAPhone extends PhoneBase {
         mCdmaSSM = CdmaSubscriptionSourceManager.getInstance(context, mCM, this,
                 EVENT_CDMA_SUBSCRIPTION_SOURCE_CHANGED, null);
         mSMS = new CdmaSMSDispatcher(this, mSmsStorageMonitor, mSmsUsageMonitor);
-        mDataConnectionTracker = new GsmDataConnectionTracker (this);
+        mDataConnectionTracker = new DataConnectionTracker (this);
         mRuimPhoneBookInterfaceManager = new RuimPhoneBookInterfaceManager(this);
         mRuimSmsInterfaceManager = new RuimSmsInterfaceManager(this, mSMS);
         mSubInfo = new PhoneSubInfo(this);
@@ -274,64 +265,79 @@ public class CDMAPhone extends PhoneBase {
         }
     }
 
+    @Override
     public ServiceState getServiceState() {
         return mSST.ss;
     }
 
+    @Override
     public CallTracker getCallTracker() {
         return mCT;
     }
 
+    @Override
     public PhoneConstants.State getState() {
         return mCT.state;
     }
 
+    @Override
     public ServiceStateTracker getServiceStateTracker() {
         return mSST;
     }
 
+    @Override
     public String getPhoneName() {
         return "CDMA";
     }
 
+    @Override
     public int getPhoneType() {
         return PhoneConstants.PHONE_TYPE_CDMA;
     }
 
+    @Override
     public boolean canTransfer() {
         Rlog.e(LOG_TAG, "canTransfer: not possible in CDMA");
         return false;
     }
 
+    @Override
     public CdmaCall getRingingCall() {
         return mCT.ringingCall;
     }
 
+    @Override
     public void setMute(boolean muted) {
         mCT.setMute(muted);
     }
 
+    @Override
     public boolean getMute() {
         return mCT.getMute();
     }
 
-    public void conference() throws CallStateException {
+    @Override
+    public void conference() {
         // three way calls in CDMA will be handled by feature codes
         Rlog.e(LOG_TAG, "conference: not possible in CDMA");
     }
 
+    @Override
     public void enableEnhancedVoicePrivacy(boolean enable, Message onComplete) {
         this.mCM.setPreferredVoicePrivacy(enable, onComplete);
     }
 
+    @Override
     public void getEnhancedVoicePrivacy(Message onComplete) {
         this.mCM.getPreferredVoicePrivacy(onComplete);
     }
 
+    @Override
     public void clearDisconnected() {
         mCT.clearDisconnected();
     }
 
+    @Override
     public DataActivityState getDataActivityState() {
         DataActivityState ret = DataActivityState.NONE;
 
@@ -353,11 +359,16 @@ public class CDMAPhone extends PhoneBase {
                 case DORMANT:
                     ret = DataActivityState.DORMANT;
                 break;
+
+                default:
+                    ret = DataActivityState.NONE;
+                break;
             }
         }
         return ret;
     }
 
+    @Override
     public Connection
     dial (String dialString) throws CallStateException {
         // Need to make sure dialString gets parsed properly
@@ -365,29 +376,35 @@ public class CDMAPhone extends PhoneBase {
         return mCT.dial(newDialString);
     }
 
+    @Override
     public Connection dial(String dialString, UUSInfo uusInfo) throws CallStateException {
         throw new CallStateException("Sending UUS information NOT supported in CDMA!");
     }
 
+    @Override
     public boolean
     getMessageWaitingIndicator() {
         return (getVoiceMessageCount() > 0);
     }
 
+    @Override
     public List<? extends MmiCode>
     getPendingMmiCodes() {
         return mPendingMmis;
     }
 
+    @Override
     public void registerForSuppServiceNotification(
             Handler h, int what, Object obj) {
         Rlog.e(LOG_TAG, "method registerForSuppServiceNotification is NOT supported in CDMA!");
     }
 
+    @Override
     public CdmaCall getBackgroundCall() {
         return mCT.backgroundCall;
     }
 
+    @Override
     public boolean handleInCallMmiCommands(String dialString) {
         Rlog.e(LOG_TAG, "method handleInCallMmiCommands is NOT supported in CDMA!");
         return false;
@@ -402,64 +419,78 @@ public class CDMAPhone extends PhoneBase {
                 .isAlive());
     }
 
+    @Override
     public void
     setNetworkSelectionModeAutomatic(Message response) {
         Rlog.e(LOG_TAG, "method setNetworkSelectionModeAutomatic is NOT supported in CDMA!");
     }
 
+    @Override
     public void unregisterForSuppServiceNotification(Handler h) {
         Rlog.e(LOG_TAG, "method unregisterForSuppServiceNotification is NOT supported in CDMA!");
     }
 
+    @Override
     public void
     acceptCall() throws CallStateException {
         mCT.acceptCall();
     }
 
+    @Override
     public void
     rejectCall() throws CallStateException {
         mCT.rejectCall();
     }
 
+    @Override
     public void
     switchHoldingAndActive() throws CallStateException {
         mCT.switchWaitingOrHoldingAndActive();
     }
 
+    @Override
     public String getLine1Number() {
         return mSST.getMdnNumber();
     }
 
+    @Override
     public String getCdmaPrlVersion(){
         return mSST.getPrlVersion();
     }
 
+    @Override
     public String getCdmaMin() {
         return mSST.getCdmaMin();
     }
 
+    @Override
     public boolean isMinInfoReady() {
         return mSST.isMinInfoReady();
     }
 
+    @Override
     public void getCallWaiting(Message onComplete) {
         mCM.queryCallWaiting(CommandsInterface.SERVICE_CLASS_VOICE, onComplete);
     }
 
+    @Override
     public void
     setRadioPower(boolean power) {
         mSST.setRadioPower(power);
     }
 
+    @Override
     public String getEsn() {
         return mEsn;
     }
 
+    @Override
     public String getMeid() {
         return mMeid;
     }
 
     //returns MEID or ESN in CDMA
+    @Override
     public String getDeviceId() {
         String id = getMeid();
         if ((id == null) || id.matches("^0*$")) {
@@ -469,43 +500,52 @@ public class CDMAPhone extends PhoneBase {
         return id;
     }
 
+    @Override
     public String getDeviceSvn() {
         Rlog.d(LOG_TAG, "getDeviceSvn(): return 0");
         return "0";
     }
 
+    @Override
     public String getSubscriberId() {
         return mSST.getImsi();
     }
 
+    @Override
     public String getImei() {
         Rlog.e(LOG_TAG, "IMEI is not available in CDMA");
         return null;
     }
 
+    @Override
     public boolean canConference() {
         Rlog.e(LOG_TAG, "canConference: not possible in CDMA");
         return false;
     }
 
+    @Override
     public CellLocation getCellLocation() {
         return mSST.cellLoc;
     }
 
+    @Override
     public CdmaCall getForegroundCall() {
         return mCT.foregroundCall;
     }
 
+    @Override
     public void
     selectNetworkManually(OperatorInfo network,
             Message response) {
         Rlog.e(LOG_TAG, "selectNetworkManually: not possible in CDMA");
     }
 
+    @Override
     public void setOnPostDialCharacter(Handler h, int what, Object obj) {
         mPostDialHandler = new Registrant(h, what, obj);
     }
 
+    @Override
     public boolean handlePinMmi(String dialString) {
         CdmaMmiCode mmi = CdmaMmiCode.newFromDialString(dialString, this);
 
@@ -538,54 +578,67 @@ public class CDMAPhone extends PhoneBase {
         }
     }
 
+    @Override
     public void setLine1Number(String alphaTag, String number, Message onComplete) {
         Rlog.e(LOG_TAG, "setLine1Number: not possible in CDMA");
     }
 
+    @Override
     public void setCallWaiting(boolean enable, Message onComplete) {
         Rlog.e(LOG_TAG, "method setCallWaiting is NOT supported in CDMA!");
     }
 
+    @Override
     public void updateServiceLocation() {
         mSST.enableSingleLocationUpdate();
     }
 
+    @Override
     public void setDataRoamingEnabled(boolean enable) {
         mDataConnectionTracker.setDataOnRoamingEnabled(enable);
     }
 
+    @Override
     public void registerForCdmaOtaStatusChange(Handler h, int what, Object obj) {
         mCM.registerForCdmaOtaProvision(h, what, obj);
     }
 
+    @Override
     public void unregisterForCdmaOtaStatusChange(Handler h) {
         mCM.unregisterForCdmaOtaProvision(h);
     }
 
+    @Override
     public void registerForSubscriptionInfoReady(Handler h, int what, Object obj) {
         mSST.registerForSubscriptionInfoReady(h, what, obj);
     }
 
+    @Override
     public void unregisterForSubscriptionInfoReady(Handler h) {
         mSST.unregisterForSubscriptionInfoReady(h);
     }
 
+    @Override
     public void setOnEcbModeExitResponse(Handler h, int what, Object obj) {
         mEcmExitRespRegistrant = new Registrant (h, what, obj);
     }
 
+    @Override
     public void unsetOnEcbModeExitResponse(Handler h) {
         mEcmExitRespRegistrant.clear();
     }
 
+    @Override
     public void registerForCallWaiting(Handler h, int what, Object obj) {
         mCT.registerForCallWaiting(h, what, obj);
     }
 
+    @Override
     public void unregisterForCallWaiting(Handler h) {
         mCT.unregisterForCallWaiting(h);
     }
 
+    @Override
     public void
     getNeighboringCids(Message response) {
         /*
@@ -604,6 +657,7 @@ public class CDMAPhone extends PhoneBase {
         }
     }
 
+    @Override
     public PhoneConstants.DataState getDataConnectionState(String apnType) {
         PhoneConstants.DataState ret = PhoneConstants.DataState.DISCONNECTED;
 
@@ -647,10 +701,12 @@ public class CDMAPhone extends PhoneBase {
         return ret;
     }
 
+    @Override
     public void sendUssdResponse(String ussdMessge) {
         Rlog.e(LOG_TAG, "sendUssdResponse: not possible in CDMA");
     }
 
+    @Override
     public void sendDtmf(char c) {
         if (!PhoneNumberUtils.is12Key(c)) {
             Rlog.e(LOG_TAG,
@@ -662,6 +718,7 @@ public class CDMAPhone extends PhoneBase {
         }
     }
 
+    @Override
     public void startDtmf(char c) {
         if (!PhoneNumberUtils.is12Key(c)) {
             Rlog.e(LOG_TAG,
@@ -671,10 +728,12 @@ public class CDMAPhone extends PhoneBase {
         }
     }
 
+    @Override
     public void stopDtmf() {
         mCM.stopDtmf(null);
     }
 
+    @Override
     public void sendBurstDtmf(String dtmfString, int on, int off, Message onComplete) {
         boolean check = true;
         for (int itr = 0;itr < dtmfString.length(); itr++) {
@@ -690,30 +749,37 @@ public class CDMAPhone extends PhoneBase {
         }
      }
 
+    @Override
     public void getAvailableNetworks(Message response) {
         Rlog.e(LOG_TAG, "getAvailableNetworks: not possible in CDMA");
     }
 
+    @Override
     public void setOutgoingCallerIdDisplay(int commandInterfaceCLIRMode, Message onComplete) {
         Rlog.e(LOG_TAG, "setOutgoingCallerIdDisplay: not possible in CDMA");
     }
 
+    @Override
     public void enableLocationUpdates() {
         mSST.enableLocationUpdates();
     }
 
+    @Override
     public void disableLocationUpdates() {
         mSST.disableLocationUpdates();
     }
 
+    @Override
     public void getDataCallList(Message response) {
         mCM.getDataCallList(response);
     }
 
+    @Override
     public boolean getDataRoamingEnabled() {
         return mDataConnectionTracker.getDataOnRoamingEnabled();
     }
 
+    @Override
     public void setVoiceMailNumber(String alphaTag,
                                    String voiceMailNumber,
                                    Message onComplete) {
@@ -726,6 +792,7 @@ public class CDMAPhone extends PhoneBase {
         }
     }
 
+    @Override
     public String getVoiceMailNumber() {
         String number = null;
         SharedPreferences sp = PreferenceManager.getDefaultSharedPreferences(getContext());
@@ -744,6 +811,7 @@ public class CDMAPhone extends PhoneBase {
     /* Returns Number of Voicemails
      * @hide
      */
+    @Override
     public int getVoiceMessageCount() {
         IccRecords r = mIccRecords.get();
         int voicemailCount =  (r != null) ? r.getVoiceMessageCount() : 0;
@@ -757,6 +825,7 @@ public class CDMAPhone extends PhoneBase {
         return voicemailCount;
     }
 
+    @Override
     public String getVoiceMailAlphaTag() {
         // TODO: Where can we get this value has to be clarified with QC.
         String ret = "";//TODO: Remove = "", if we know where to get this value.
@@ -771,10 +840,12 @@ public class CDMAPhone extends PhoneBase {
         return ret;
     }
 
+    @Override
     public void getCallForwardingOption(int commandInterfaceCFReason, Message onComplete) {
         Rlog.e(LOG_TAG, "getCallForwardingOption: not possible in CDMA");
     }
 
+    @Override
     public void setCallForwardingOption(int commandInterfaceCFAction,
             int commandInterfaceCFReason,
             String dialingNumber,
@@ -783,36 +854,42 @@ public class CDMAPhone extends PhoneBase {
         Rlog.e(LOG_TAG, "setCallForwardingOption: not possible in CDMA");
     }
 
+    @Override
     public void
     getOutgoingCallerIdDisplay(Message onComplete) {
         Rlog.e(LOG_TAG, "getOutgoingCallerIdDisplay: not possible in CDMA");
     }
 
+    @Override
     public boolean
     getCallForwardingIndicator() {
         Rlog.e(LOG_TAG, "getCallForwardingIndicator: not possible in CDMA");
         return false;
     }
 
+    @Override
     public void explicitCallTransfer() {
         Rlog.e(LOG_TAG, "explicitCallTransfer: not possible in CDMA");
     }
 
+    @Override
     public String getLine1AlphaTag() {
         Rlog.e(LOG_TAG, "getLine1AlphaTag: not possible in CDMA");
         return null;
     }
 
     /**
-     * Notify any interested party of a Phone state change  {@link PhoneConstants.State}
+     * Notify any interested party of a Phone state change
+     * {@link com.android.internal.telephony.PhoneConstants.State}
      */
     /*package*/ void notifyPhoneStateChanged() {
         mNotifier.notifyPhoneState(this);
     }
 
     /**
-     * Notify registrants of a change in the call state. This notifies changes in {@link Call.State}
-     * Use this when changes in the precise call state are needed, else use notifyPhoneStateChanged.
+     * Notify registrants of a change in the call state. This notifies changes in
+     * {@link com.android.internal.telephony.Call.State}. Use this when changes
+     * in the precise call state are needed, else use notifyPhoneStateChanged.
      */
     /*package*/ void notifyPreciseCallStateChanged() {
         /* we'd love it if this was package-scoped*/
@@ -840,10 +917,12 @@ public class CDMAPhone extends PhoneBase {
         mUnknownConnectionRegistrants.notifyResult(this);
     }
 
+    @Override
     public boolean isInEmergencyCall() {
         return mCT.isInEmergencyCall();
     }
 
+    @Override
     public boolean isInEcm() {
         return mIsPhoneInEcmState;
     }
@@ -940,10 +1019,12 @@ public class CDMAPhone extends PhoneBase {
      * @param what User-defined message code
      * @param obj placed in Message.obj
      */
+    @Override
     public void registerForEcmTimerReset(Handler h, int what, Object obj) {
         mEcmTimerResetRegistrants.addUnique(h, what, obj);
     }
 
+    @Override
     public void unregisterForEcmTimerReset(Handler h) {
         mEcmTimerResetRegistrants.remove(h);
     }
@@ -1121,6 +1202,7 @@ public class CDMAPhone extends PhoneBase {
     /**
      * Retrieves the PhoneSubInfo of the CDMAPhone
      */
+    @Override
     public PhoneSubInfo getPhoneSubInfo() {
         return mSubInfo;
     }
@@ -1128,6 +1210,7 @@ public class CDMAPhone extends PhoneBase {
     /**
      * Retrieves the IccSmsInterfaceManager of the CDMAPhone
      */
+    @Override
     public IccSmsInterfaceManager getIccSmsInterfaceManager() {
         return mRuimSmsInterfaceManager;
     }
@@ -1135,6 +1218,7 @@ public class CDMAPhone extends PhoneBase {
     /**
      * Retrieves the IccPhoneBookInterfaceManager of the CDMAPhone
      */
+    @Override
     public IccPhoneBookInterfaceManager getIccPhoneBookInterfaceManager() {
         return mRuimPhoneBookInterfaceManager;
     }
@@ -1152,6 +1236,7 @@ public class CDMAPhone extends PhoneBase {
     /**
      * {@inheritDoc}
      */
+    @Override
     public final void setSystemProperty(String property, String value) {
         super.setSystemProperty(property, value);
     }
@@ -1162,6 +1247,7 @@ public class CDMAPhone extends PhoneBase {
      * @param activate 0 = activate, 1 = deactivate
      * @param response Callback message is empty on completion
      */
+    @Override
     public void activateCellBroadcastSms(int activate, Message response) {
         Rlog.e(LOG_TAG, "[CDMAPhone] activateCellBroadcastSms() is obsolete; use SmsManager");
         response.sendToTarget();
@@ -1172,6 +1258,7 @@ public class CDMAPhone extends PhoneBase {
      *
      * @param response Callback message is empty on completion
      */
+    @Override
     public void getCellBroadcastSmsConfig(Message response) {
         Rlog.e(LOG_TAG, "[CDMAPhone] getCellBroadcastSmsConfig() is obsolete; use SmsManager");
         response.sendToTarget();
@@ -1182,6 +1269,7 @@ public class CDMAPhone extends PhoneBase {
      *
      * @param response Callback message is empty on completion
      */
+    @Override
     public void setCellBroadcastSmsConfig(int[] configValuesArray, Message response) {
         Rlog.e(LOG_TAG, "[CDMAPhone] setCellBroadcastSmsConfig() is obsolete; use SmsManager");
         response.sendToTarget();
@@ -1269,11 +1357,11 @@ public class CDMAPhone extends PhoneBase {
         boolean isOtaSpNum = false;
         try {
             // Get how many number of system selection code ranges
-            int selRc = Integer.parseInt((String)sch[1]);
+            int selRc = Integer.parseInt(sch[1]);
             for (int i = 0; i < selRc; i++) {
                 if (!TextUtils.isEmpty(sch[i+2]) && !TextUtils.isEmpty(sch[i+3])) {
-                    int selMin = Integer.parseInt((String)sch[i+2]);
-                    int selMax = Integer.parseInt((String)sch[i+3]);
+                    int selMin = Integer.parseInt(sch[i+2]);
+                    int selMax = Integer.parseInt(sch[i+3]);
                     // Check if the selection code extracted from the dial string falls
                     // within any of the range pairs specified in the schema.
                     if ((sysSelCodeInt >= selMin) && (sysSelCodeInt <= selMax)) {
@@ -1337,8 +1425,8 @@ public class CDMAPhone extends PhoneBase {
                         }
                     }
                 } else if (!TextUtils.isEmpty(sch[0]) && sch[0].equals("FC")) {
-                    int fcLen =  Integer.parseInt((String)sch[1]);
-                    String fc = (String)sch[2];
+                    int fcLen =  Integer.parseInt(sch[1]);
+                    String fc = sch[2];
                     if (dialStr.regionMatches(0,fc,0,fcLen)) {
                         isOtaSpNum = true;
                     } else {
@@ -1509,7 +1597,7 @@ public class CDMAPhone extends PhoneBase {
 
     protected void log(String s) {
         if (DBG)
-            Rlog.d(LOG_TAG, "[CDMAPhone] " + s);
+            Rlog.d(LOG_TAG, s);
     }
 
     @Override
index 4ad61bb..c6a881f 100644 (file)
@@ -16,7 +16,6 @@
 
 package com.android.internal.telephony.cdma;
 
-import java.util.ArrayList;
 import java.util.List;
 
 import com.android.internal.telephony.Call;
@@ -24,7 +23,6 @@ import com.android.internal.telephony.CallStateException;
 import com.android.internal.telephony.Connection;
 import com.android.internal.telephony.DriverCall;
 import com.android.internal.telephony.Phone;
-import com.android.internal.telephony.Call.State;
 
 /**
  * {@hide}
@@ -32,8 +30,6 @@ import com.android.internal.telephony.Call.State;
 public final class CdmaCall extends Call {
     /*************************** Instance Variables **************************/
 
-    /*package*/ ArrayList<Connection> connections = new ArrayList<Connection>();
-    /*package*/ State state = State.IDLE;
     /*package*/ CdmaCallTracker owner;
 
     /***************************** Class Methods *****************************/
@@ -62,22 +58,20 @@ public final class CdmaCall extends Call {
     }
 
     /************************** Overridden from Call *************************/
+    @Override
     public List<Connection>
     getConnections() {
         // FIXME should return Collections.unmodifiableList();
         return connections;
     }
 
-    public State
-    getState() {
-        return state;
-    }
-
+    @Override
     public Phone
     getPhone() {
         return owner.phone;
     }
 
+    @Override
     public boolean isMultiparty() {
         return connections.size() > 1;
     }
@@ -86,11 +80,13 @@ public final class CdmaCall extends Call {
      *  background call exists, the background call will be resumed
      *  because an AT+CHLD=1 will be sent
      */
+    @Override
     public void
     hangup() throws CallStateException {
         owner.hangup(this);
     }
 
+    @Override
     public String
     toString() {
         return state.toString();
index 8ec5633..159cf77 100644 (file)
@@ -45,7 +45,7 @@ import java.util.List;
  * {@hide}
  */
 public final class CdmaCallTracker extends CallTracker {
-    static final String LOG_TAG = "CDMA";
+    static final String LOG_TAG = "CdmaCallTracker";
 
     private static final boolean REPEAT_POLLING = false;
 
@@ -134,6 +134,7 @@ public final class CdmaCallTracker extends CallTracker {
     //***** Instance Methods
 
     //***** Public Methods
+    @Override
     public void registerForVoiceCallStarted(Handler h, int what, Object obj) {
         Registrant r = new Registrant(h, what, obj);
         voiceCallStartedRegistrants.add(r);
@@ -142,15 +143,18 @@ public final class CdmaCallTracker extends CallTracker {
             r.notifyRegistrant(new AsyncResult(null, null, null));
         }
     }
+    @Override
     public void unregisterForVoiceCallStarted(Handler h) {
         voiceCallStartedRegistrants.remove(h);
     }
 
+    @Override
     public void registerForVoiceCallEnded(Handler h, int what, Object obj) {
         Registrant r = new Registrant(h, what, obj);
         voiceCallEndedRegistrants.add(r);
     }
 
+    @Override
     public void unregisterForVoiceCallEnded(Handler h) {
         voiceCallEndedRegistrants.remove(h);
     }
@@ -164,21 +168,6 @@ public final class CdmaCallTracker extends CallTracker {
         callWaitingRegistrants.remove(h);
     }
 
-    private void
-    fakeHoldForegroundBeforeDial() {
-        List<Connection> connCopy;
-
-        // We need to make a copy here, since fakeHoldBeforeDial()
-        // modifies the lists, and we don't want to reverse the order
-        connCopy = (List<Connection>) foregroundCall.connections.clone();
-
-        for (int i = 0, s = connCopy.size() ; i < s ; i++) {
-            CdmaConnection conn = (CdmaConnection)connCopy.get(i);
-
-            conn.fakeHoldBeforeDial();
-        }
-    }
-
     /**
      * clirMode is one of the CLIR_ constants
      */
@@ -198,7 +187,7 @@ public final class CdmaCallTracker extends CallTracker {
 
         // Cancel Ecm timer if a second emergency call is originating in Ecm mode
         if (isPhoneInEcmMode && isEmergencyCall) {
-            handleEcmTimer(phone.CANCEL_ECM_TIMER);
+            handleEcmTimer(CDMAPhone.CANCEL_ECM_TIMER);
         }
 
         // We are initiating a call therefore even if we previously
@@ -322,13 +311,13 @@ public final class CdmaCallTracker extends CallTracker {
     }
 
     void
-    conference() throws CallStateException {
+    conference() {
         // Should we be checking state?
         flashAndSetGenericTrue();
     }
 
     void
-    explicitCallTransfer() throws CallStateException {
+    explicitCallTransfer() {
         cm.explicitCallTransfer(obtainCompleteMessage(EVENT_ECT_RESULT));
     }
 
@@ -473,6 +462,7 @@ public final class CdmaCallTracker extends CallTracker {
 
     // ***** Overwritten from CallTracker
 
+    @Override
     protected void
     handlePollCalls(AsyncResult ar) {
         List polledCalls;
@@ -532,7 +522,7 @@ public final class CdmaCallTracker extends CallTracker {
                         hangupPendingMO = false;
                         // Re-start Ecm timer when an uncompleted emergency call ends
                         if (mIsEcmTimerCanceled) {
-                            handleEcmTimer(phone.RESTART_ECM_TIMER);
+                            handleEcmTimer(CDMAPhone.RESTART_ECM_TIMER);
                         }
 
                         try {
@@ -583,7 +573,7 @@ public final class CdmaCallTracker extends CallTracker {
 
                 // Re-start Ecm timer when the connected emergency call ends
                 if (mIsEcmTimerCanceled) {
-                    handleEcmTimer(phone.RESTART_ECM_TIMER);
+                    handleEcmTimer(CDMAPhone.RESTART_ECM_TIMER);
                 }
                 // If emergency call is not going through while dialing
                 checkAndEnableDataCallAfterEmergencyCallDropped();
@@ -859,7 +849,7 @@ public final class CdmaCallTracker extends CallTracker {
         throw new CallStateException("no gsm index found");
     }
 
-    void hangupAllConnections(CdmaCall call) throws CallStateException{
+    void hangupAllConnections(CdmaCall call) {
         try {
             int count = call.connections.size();