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

Commit b0800dce authored by Tobias Thierer's avatar Tobias Thierer
Browse files

Avoid instantiations of Boolean that are immediately unboxed. 

Replaced Boolean.valueOf() -> parseBoolean() across frameworks/base
(not including Am.java, which I'm leaving for a separate change).

Also fixed a couple of other valueOf()s in SyncStorageEngine.

Bug: 28289401
(cherry-picked from commit ca018207)
Change-Id: I2c6fb39a465885e7e625ed8894b1b9f8a30da5bc
parent 8a676470

core/java/android/service/notification/ZenModeConfig.java

+1 −1
Original line number Diff line number Diff line
@@ -570,7 +570,7 @@ public class ZenModeConfig implements Parcelable { 


    private static boolean safeBoolean(String val, boolean defValue) {

        if (TextUtils.isEmpty(val)) return defValue;

        return Boolean.valueOf(val);

        return Boolean.parseBoolean(val);

    }



    private static int safeInt(XmlPullParser parser, String att, int defValue) {

core/java/com/android/internal/net/VpnProfile.java

+1 −1
Original line number Diff line number Diff line
@@ -135,7 +135,7 @@ public class VpnProfile implements Cloneable, Parcelable { 
            profile.dnsServers = values[5];

            profile.searchDomains = values[6];

            profile.routes = values[7];

            profile.mppe = Boolean.valueOf(values[8]);

            profile.mppe = Boolean.parseBoolean(values[8]);

            profile.l2tpSecret = values[9];

            profile.ipsecIdentifier = values[10];

            profile.ipsecSecret = values[11];

core/java/com/android/internal/util/XmlUtils.java

+1 −1
Original line number Diff line number Diff line
@@ -1280,7 +1280,7 @@ public class XmlUtils { 
            if (eventType == parser.START_TAG) {

                if (parser.getName().equals("item")) {

                    try {

                        array[i] = Boolean.valueOf(parser.getAttributeValue(null, "value"));

                        array[i] = Boolean.parseBoolean(parser.getAttributeValue(null, "value"));

                    } catch (NullPointerException e) {

                        throw new XmlPullParserException("Need value attribute in item");

                    } catch (NumberFormatException e) {

services/core/java/com/android/server/am/ActivityRecord.java

+1 −1
Original line number Diff line number Diff line
@@ -1402,7 +1402,7 @@ final class ActivityRecord { 
            } else if (ATTR_RESOLVEDTYPE.equals(attrName)) {

                resolvedType = attrValue;

            } else if (ATTR_COMPONENTSPECIFIED.equals(attrName)) {

                componentSpecified = Boolean.valueOf(attrValue);

                componentSpecified = Boolean.parseBoolean(attrValue);

            } else if (ATTR_USERID.equals(attrName)) {

                userId = Integer.parseInt(attrValue);

            } else if (attrName.startsWith(TaskDescription.ATTR_TASKDESCRIPTION_PREFIX)) {

services/core/java/com/android/server/notification/RankingHelper.java

+1 −1
Original line number Diff line number Diff line
@@ -311,7 +311,7 @@ public class RankingHelper implements RankingConfig { 


    private static boolean tryParseBool(String value, boolean defValue) {

        if (TextUtils.isEmpty(value)) return defValue;

        return Boolean.valueOf(value);

        return Boolean.parseBoolean(value);

    }



    /**