Merge "Prefer valueOf() to boxed primitive constructors" am: 773627cfd3 am: 2488ef52b4 am: f43a706e53

Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/2216444

Change-Id: I73085165b50ea9bdcc5b15359d899a72d034a251
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Orion Hodson
2022-09-20 12:34:17 +00:00
committed by Automerger Merge Worker

View File

@@ -268,22 +268,22 @@ public class TypedProperties extends HashMap<String, Object> {
if (value < Byte.MIN_VALUE || value > Byte.MAX_VALUE) { if (value < Byte.MIN_VALUE || value > Byte.MAX_VALUE) {
throw new ParseException(st, "8-bit integer constant"); throw new ParseException(st, "8-bit integer constant");
} }
return new Byte((byte)value); return Byte.valueOf((byte) value);
case 2: case 2:
if (value < Short.MIN_VALUE || value > Short.MAX_VALUE) { if (value < Short.MIN_VALUE || value > Short.MAX_VALUE) {
throw new ParseException(st, "16-bit integer constant"); throw new ParseException(st, "16-bit integer constant");
} }
return new Short((short)value); return Short.valueOf((short) value);
case 4: case 4:
if (value < Integer.MIN_VALUE || value > Integer.MAX_VALUE) { if (value < Integer.MIN_VALUE || value > Integer.MAX_VALUE) {
throw new ParseException(st, "32-bit integer constant"); throw new ParseException(st, "32-bit integer constant");
} }
return new Integer((int)value); return Integer.valueOf((int) value);
case 8: case 8:
if (value < Long.MIN_VALUE || value > Long.MAX_VALUE) { if (value < Long.MIN_VALUE || value > Long.MAX_VALUE) {
throw new ParseException(st, "64-bit integer constant"); throw new ParseException(st, "64-bit integer constant");
} }
return new Long(value); return Long.valueOf(value);
default: default:
throw new IllegalStateException( throw new IllegalStateException(
"Internal error; unexpected integer type width " + width); "Internal error; unexpected integer type width " + width);
@@ -317,10 +317,10 @@ public class TypedProperties extends HashMap<String, Object> {
throw new ParseException(st, "32-bit float constant"); throw new ParseException(st, "32-bit float constant");
} }
} }
return new Float((float)value); return Float.valueOf((float) value);
} else { } else {
// This property is a double; no need to truncate. // This property is a double; no need to truncate.
return new Double(value); return Double.valueOf(value);
} }
} else if (type == TYPE_STRING) { } else if (type == TYPE_STRING) {
// Expect a quoted string or the word "null". // Expect a quoted string or the word "null".