8200364: Remove unnecessary boxing via primitive wrapper valueOf(String) methods

Martin Buchholz · Martin Buchholz · commit 79fffd05ef30 · 2018-04-02T17:09:29.000-07:00
Reviewed-by: dfuchs, alanb
diff --git a/src/java.base/share/classes/java/text/ChoiceFormat.java b/src/java.base/share/classes/java/text/ChoiceFormat.java
@@ -212,7 +212,7 @@ public void applyPattern(String newPattern) {
                 } else if (tempBuffer.equals("-\u221E")) {
                     startValue = Double.NEGATIVE_INFINITY;
                 } else {
-                    startValue = Double.valueOf(tempBuffer);
+                    startValue = Double.parseDouble(tempBuffer);
                 }
 
                 if (ch == '<' && startValue != Double.POSITIVE_INFINITY &&
diff --git a/src/java.logging/share/classes/java/util/logging/Logger.java b/src/java.logging/share/classes/java/util/logging/Logger.java
@@ -640,7 +640,7 @@ public String run() {
                     return System.getProperty(key);
                 }
             });
-            return Boolean.valueOf(s);
+            return Boolean.parseBoolean(s);
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -212,7 +212,7 @@ public void applyPattern(String newPattern) {`
`212`	`212`	`} else if (tempBuffer.equals("-\u221E")) {`
`213`	`213`	`startValue = Double.NEGATIVE_INFINITY;`
`214`	`214`	`} else {`
`215`		`- startValue = Double.valueOf(tempBuffer);`
	`215`	`+ startValue = Double.parseDouble(tempBuffer);`
`216`	`216`	`}`
`217`	`217`
`218`	`218`	`if (ch == '<' && startValue != Double.POSITIVE_INFINITY &&`
Original file line number	Diff line number	Diff line change
`@@ -640,7 +640,7 @@ public String run() {`
`640`	`640`	`return System.getProperty(key);`
`641`	`641`	`}`
`642`	`642`	`});`
`643`		`- return Boolean.valueOf(s);`
	`643`	`+ return Boolean.parseBoolean(s);`
`644`	`644`	`}`
`645`	`645`	`}`
`646`	`646`