Hi, postponed this, for a later try.
I created 3 subtasks: https://bugs.openjdk.java.net/browse/JDK-8143008 (jaxp) https://bugs.openjdk.java.net/browse/JDK-8143009 (jaxws) https://bugs.openjdk.java.net/browse/JDK-8143010 (corba) and documented my patches and linked the related discussion-threads in the subtask. Please cc me, if someone wants to start discussion/review on those. -- Sebastian On 11/08/2015 01:50 PM, Sebastian Sickelmann wrote: > Hi, > > i wanted to start a discussion/review-process some time ago, see the > second-try below. > > Is there someone who wants to discuss/review/sponsor these changes? > > Else, should i link my result as reference into the JBS? > > For the two other affected mailing-lists/repos > (macos-port-dev[0],net-dev[1]) i created a subtask for each. If this > three patches here cannot go into one review, i would create subtasks > for those as well, to handle the different state of theses changes . > > -- Sebastian > > [0] > http://mail.openjdk.java.net/pipermail/macosx-port-dev/2015-October/006985.html > [1] http://mail.openjdk.java.net/pipermail/net-dev/2015-November/009308.html > > 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. >> >> 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 >> >>
