Hi, i wanted to start a discussion/review-process for JDK5108778 some time ago. Please see a previous try below. The patches are now pretty old, and I will rebase and update them to the created sub-issues
- JDK-8143008 (jaxp) - JDK-8143009 (jaxws) - JDK-8143010 (corba) for a sub-repo-discussion/review/push-process after initial discussion has started. For now the webrev for the initial discussion i would love to use 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/ -- Sebastian On 10/27/2015 05:32 AM, Sebastian Sickelmann wrote: > 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 jaxp, jaxws and corba repos at: > > 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. > 8 > 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. > > -- Sebastian
