Fix SignalStrength response.
Wink Saville [Mon, 8 Apr 2013 18:42:08 +0000 (11:42 -0700)]
Use the new SignalStrength constructor with gsmFlag.

Also, remove code in responseStrings which would cause
it to return an empty array if RILJ_LOGV was true.

Bug: 8561333
Change-Id: Ie6eb3b0335e8d7ea114ec70b3fb72e917094632e

src/java/com/android/internal/telephony/RIL.java

index 3885dbe..425da2a 100644 (file)
@@ -3056,15 +3056,6 @@ public final class RIL extends BaseCommands implements CommandsInterface {
 
         response = p.readStringArray();
 
-        if (RILJ_LOGV) {
-            num = p.readInt();
-
-            response = new String[num];
-            for (int i = 0; i < num; i++) {
-                response[i] = p.readString();
-            }
-        }
-
         return response;
     }
 
@@ -3475,7 +3466,9 @@ public final class RIL extends BaseCommands implements CommandsInterface {
 
     private Object
     responseSignalStrength(Parcel p) {
-        SignalStrength signalStrength = new SignalStrength(p);
+        // Assume this is gsm, but doesn't matter as ServiceStateTracker
+        // sets the proper value.
+        SignalStrength signalStrength = SignalStrength.makeSignalStrengthFromRilParcel(p);
         return signalStrength;
     }