RFR: 8187954 Update JAX-WS RI integration to latest version

Lance Andersen lance.andersen at oracle.com
Wed Oct 11 10:47:51 UTC 2017


Hi Jack,

I would prefer to see an updated webrev so that we do not inadvertently
push these changes.

Best
Lance
> On Oct 11, 2017, at 3:26 AM, Jack Li <zheng.jun.li at oracle.com> wrote:
> 
> Hi Lance
> 
> I will update them in Metro repository, do I need to regenerate webrev?
> or can you skip the files this time and I fix it in next integration?
> 
>> On Oct 9, 2017, at 19:35, Lance Andersen <lance.andersen at oracle.com <mailto:lance.andersen at oracle.com>> wrote:
>> 
>> Hi Jack,
>> 
>> UnMarshaller also has the same issue.  I would update the webrev given the number of places to help sanity check for omissions
>> 
>> Best
>> Lance
>>> On Oct 8, 2017, at 9:22 PM, Jack Li <zheng.jun.li at oracle.com <mailto:zheng.jun.li at oracle.com>> wrote:
>>> 
>>> Hi Lance,
>>> 
>>> the change is incorrect, it should be “javax/xml/bind”.
>>> thanks a lot for your finding, do you think I need to fix it and resubmit the webrev this time?
>>> or can you skip this file this time and I fix it in next integration?
>>> 
>>>> On Oct 4, 2017, at 02:09, Lance Andersen <Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>> wrote:
>>>> 
>>>> Hi Jack,
>>>> 
>>>> Is this change correct:
>>>> 
>>>> -------------
>>>> --- old/src/java.xml.bind/share/classes/javax/xml/bind/Marshaller.java	2017-09-29 13:58:31.968185273 +0100
>>>> +++ new/src/java.xml.bind/share/classes/javax/xml/bind/Marshaller.java	2017-09-29 13:58:31.676185267 +0100
>>>> @@ -373,7 +373,7 @@
>>>>       *      If the {@link ValidationEventHandler ValidationEventHandler}
>>>>       *      returns false from its {@code handleEvent} method or the
>>>>       *      {@code Marshaller} is unable to marshal {@code jaxbElement} (or any
>>>> -     *      object reachable from {@code jaxbElement}).  See <a href="{@docRoot}/javax/xml/bind/Marshaller.html#elementMarshalling">
>>>> +     *      object reachable from {@code jaxbElement}).  See <a href="{@docRoot}/java/xml/bind/Marshaller.html#elementMarshalling">
>>>>       *      Marshalling a JAXB element</a>.
>>>> 
>>>> ------------
>>>> 
>>>> The URL that is being changed currently works
>>>> 
>>>> Best
>>>> Lance
>>>> On Sep 29, 2017, at 10:55 PM, Jack Li <zheng.jun.li at oracle.com <mailto:zheng.jun.li at oracle.com>> wrote:
>>>> 
>>>>> Hi,
>>>>> 
>>>>> Please review standalone JAXB/JAXWS changes, synced to jdk/jaxws repo.
>>>>> 
>>>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8187954 <https://bugs.openjdk.java.net/browse/JDK-8187954> <https://bugs.openjdk.java.net/browse/JDK-8187954 <https://bugs.openjdk.java.net/browse/JDK-8187954>>
>>>>> Webrev: http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8187954/10/00/ <http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8187954/10/00/> <http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8187954/10/00/ <http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8187954/10/00/>>
>>>>> 
>>>>> Summary of changes:
>>>>> 
>>>>> jaxws/src/java.xml.bind/share/classes/javax/xml/bind/*
>>>>> JDK-8186946 - Fix accessibility and other issues in the java.xml.bind module
>>>>> 
>>>>> jaxws/src/java.xml.ws/share/classes/com/sun/xml/internal/messaging/saaj/**
>>>>> JDK-8186314 - code at c.s.x.i.m.saaj.soap.MessageImpl must be modified to avoid crash after javac change
>>>>> And also contains the fixes for importing nodes for SOAPDocumentFragment
>>>>> 
>>>>> 
>>>>> Patch also contains several small bugfixes, not tracked in JBS.
>>>>> 
>>>>> ----------------
>>>>> Best regards
>>>>> Jack Li
>>>>> 
>>>>> 
>>>>> 
>>>>> 
>>>>> 
>>>>> 
>>>> 
>>>> 
>>>> <oracle_sig_logo.gif>
>>>> 
>>>> Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
>>>> Oracle Java Engineering 
>>>> 1 Network Drive 
>>>> Burlington, MA 01803
>>>>  <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>>> 
>>> ----------------
>>> Best regards
>>> Jack Li
>>> 
>>> 
>>> 
>>> 
>>> 
>>> 
>> 
>> <oracle_sig_logo.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>  <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>  <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
>> Oracle Java Engineering 
>> 1 Network Drive 
>> Burlington, MA 01803
>> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>> 
>> 
>> 
> 
> 
> ----------------
> Best regards
> Jack Li

 <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>





More information about the core-libs-dev mailing list