OSDN Git Service

Update Docstring for SmsMessage.getOriginatingAddress
authorNathan Harold <nharold@google.com>
Mon, 29 Jan 2018 19:36:03 +0000 (11:36 -0800)
committerNathan Harold <nharold@google.com>
Tue, 30 Jan 2018 02:59:49 +0000 (18:59 -0800)
There was a public request for clarification on the address
format for getOriginatingAddress. I did a little research and
have added the answer to the docstring.

Bug: 64697463
Test: compilation
Change-Id: Icf37af0a5940a6fb7798d7c7cafe7b97683bb689

telephony/java/android/telephony/SmsMessage.java

index a6dbc06..577ea7d 100644 (file)
@@ -18,8 +18,8 @@ package android.telephony;
 
 import static android.telephony.TelephonyManager.PHONE_TYPE_CDMA;
 
+import android.annotation.Nullable;
 import android.annotation.StringDef;
-import android.app.PendingIntent;
 import android.content.res.Resources;
 import android.os.Binder;
 import android.text.TextUtils;
@@ -544,8 +544,16 @@ public class SmsMessage {
 
     /**
      * Returns the originating address (sender) of this SMS message in String
-     * form or null if unavailable
+     * form or null if unavailable.
+     *
+     * <p>If the address is a GSM-formatted address, it will be in a format specified by 3GPP
+     * 23.040 Sec 9.1.2.5. If it is a CDMA address, it will be a format specified by 3GPP2
+     * C.S005-D Table 2.7.1.3.2.4-2. The choice of format is carrier-specific, so callers of the
+     * should be careful to avoid assumptions about the returned content.
+     *
+     * @return a String representation of the address; null if unavailable.
      */
+    @Nullable
     public String getOriginatingAddress() {
         return mWrappedSmsMessage.getOriginatingAddress();
     }