RFR: 5108778 Too many instances of java.lang.Boolean created in Java application(core-libs)

[Stuart Marks]

Hi Fabian,

As you might have seen from my message to Sebastian, I agree that the Boolean 
constructors shouldn't be deprecated -- at least not with deprecation in its 
current form.

The idea is eventually to mark them somehow, in a way that warns developers that 
other APIs are preferable, but without any implication that they will eventually 
be removed. There are no plans to remove the Boolean constructors from the API.

s'marks

On 10/8/15 10:30 AM, Fabian Lange wrote:
> Hi Sebastian,
>
> I am all for micro optimizations that do make sense, even if their impact
> might be low, but I disagree with deprecating the Boolean constructor.
> Using it, while not being the most efficient, is not a problem, and I would
> say in almost all application and workloads not an issue worth discussing.
> What I consider a problem is making the Wrapper types asymetric by removing
> the constructor (that is the ultimate goal of deprecation).
> I do realize Dr. Deprecator suggested doing this, but I do not agree this
> time-
>
> Also this would be a spec change, right?
>
> Fabian
>
>
>> Hello,
>>
>> Actually I am searching through the JBS for low hanging fruits.
>> Right now i am looking through the openjdk-sources and try to evaluate
>> if i can make something about JDK-5108778.
>>
>> Please find my webrevs for the jdk, jaxp, jaxws and corba repos at:
>>
>> http://cr.openjdk.java.net/~sebastian/5108778/core-libs/webrev.00/
>> http://cr.openjdk.java.net/~sebastian/5108778/jaxp/webrev.00/
>> http://cr.openjdk.java.net/~sebastian/5108778/jaxws/webrev.00/
>> http://cr.openjdk.java.net/~sebastian/5108778/corba/webrev.00/
>>
>> I hope for jaxp,jaxws and corba this mailinglist is also the right place.
>>
>> jdk:
>> The Boolean constructors are @Deprecated now so that we get
>> compile-warnings for the uses. See also [0] and [1]
>>
>> jaxp:
>> For the change in XBoolean (jaxp) i thought it would be more readable
>> than the autoboxing solution.
>>
>> jaxws:
>> The changes in jaxws are in a generated class. I would love to fix this
>> in the source, but i have no clue where the real source could be.
>>
>> corba:
>> For all calls against
>> com.sun.tools.corba.se.idl.constExpr.Expression.value(Object) i used the
>> valueOf solution instead of the autoboxing for better readability.
>>
>> For some general discussion on regression-tests for this please find the
>> thread in discuss[0][1] and for the general suggestion to make more
>> wrapper-type-constructors deprecated find [2] at core-libs-dev.
>>
>> [0]
>> http://mail.openjdk.java.net/pipermail/discuss/2015-September/003804.html
>> [1]
>> http://mail.openjdk.java.net/pipermail/discuss/2015-October/003805.html
>> [2]
>>
>> http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-October/035642.html
>>
>> -- Sebastian