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
