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

Commit e3366ce3 authored by Wink Saville's avatar Wink Saville
Browse files

Fix SignalStrength response. 

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
parent f50b1bea

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

+3 −10
Original line number Diff line number Diff line
@@ -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;

    }