archive/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2025-08-27 23:04:50 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

8276904: Optional.toString() is unnecessarily expensive

Browse source 
Reviewed-by: rriggs, smarks
This commit is contained in:
Eamonn McManus 2021-12-08 21:24:43 +00:00
parent 6e7b7f352e
commit fe2ae8e38b
4 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/java.base/share/classes/java/util/Optional.java
View file

@ -454,7 +454,7 @@ public final class Optional<T> {
@Override
public String toString() {
return value != null
? String.format("Optional[%s]", value)
? ("Optional[" + value + "]")
: "Optional.empty";
}
}

2
src/java.base/share/classes/java/util/OptionalDouble.java
View file

@ -328,7 +328,7 @@ public final class OptionalDouble {
@Override
public String toString() {
return isPresent
? String.format("OptionalDouble[%s]", value)
? ("OptionalDouble[" + value + "]")
: "OptionalDouble.empty";
}
}

2
src/java.base/share/classes/java/util/OptionalInt.java
View file

@ -326,7 +326,7 @@ public final class OptionalInt {
@Override
public String toString() {
return isPresent
? String.format("OptionalInt[%s]", value)
? ("OptionalInt[" + value + "]")
: "OptionalInt.empty";
}
}

2
src/java.base/share/classes/java/util/OptionalLong.java
View file

@ -326,7 +326,7 @@ public final class OptionalLong {
@Override
public String toString() {
return isPresent
? String.format("OptionalLong[%s]", value)
? ("OptionalLong[" + value + "]")
: "OptionalLong.empty";
}
}