RFR: 8065870 Update JAX-WS RI integration to latest version (2.2.11-b141124.1933)

[Stuart Marks]

Great, this sounds fine.

s'marks

On 12/2/14 6:03 AM, Miroslav Kos wrote:
> Hi Stuart,
> minor stuff like invalid characters and copyright years I can fix before push,
> the rest will leave for next integration which should be soon.
>
> Thanks!
> Miran
>
>
>
> On 01/12/14 20:35, Stuart Marks wrote:
>> Hi Miran,
>>
>> I'm pretty distant from the JAX-WS code, but I looked through all of the files
>> and most of the changes seem sensible. There are a few things that are
>> questionable though.
>>
>> **
>> src/java.xml.ws/share/classes/com/sun/xml/internal/ws/api/streaming/XMLStreamReaderFactory.java
>>
>>
>> The catch-and-ignore of Throwable at line 565 seems questionable. Wouldn't it
>> be better to catch a few specific exception types that might be thrown from
>> setProperty()?
>>
>> **
>> src/java.xml.ws/share/classes/com/sun/xml/internal/ws/util/resources/Messages_en.properties
>>
>>
>> The copyright year is changed from 2013 to 2012.
>>
>> The unknown character replacement (line 277) is replaced with a '?', though
>> I'm not sure what's really happening here since webrev might be mishandling
>> non-ascii characters. If this is intended to be an ascii file, shouldn't the
>> replacement be a plain single quote (') ?
>>
>> **
>> src/jdk.xml.bind/share/classes/com/sun/tools/internal/jxc/MessageBundle.properties
>>
>>
>> The version numbers in this file seem to be moved forward, but the copyright
>> is updated from 2014 to 2012. The same appears to be true of the localized
>> versions of this file.
>>
>> **
>> src/jdk.xml.bind/share/classes/com/sun/tools/internal/xjc/MessageBundle.properties
>>
>>
>> Copyright years 2014 => 2012 again. Also check localized versions of this file.
>>
>> **
>> src/jdk.xml.bind/share/classes/com/sun/tools/internal/xjc/generator/bean/MessageBundle.properties
>>
>>
>> Copyright years 2013 => 2012. Possibly incorrect replacement "???" for unknown
>> character in original file.
>>
>> ** src/jdk.xml.ws/share/classes/com/sun/tools/internal/ws/version.properties
>>
>> Copyright years 2014 => 2013.
>>
>> ==========
>>
>> Nothing earth-shattering here. If you want to push this changeset and fix up
>> these issues later (if indeed they need to be fixed up), I'd be fine with that.
>>
>> s'marks
>>
>>
>>
>>
>>
>> On 11/27/14 3:27 AM, Miroslav Kos wrote:
>>> Hi,
>>> there is a bulk update of JAX-B/WS from upstream projects -
>>> webrev: http://cr.openjdk.java.net/~mkos/8065870/jaxws.00/
>>> more details in issue desc: https://bugs.openjdk.java.net/browse/JDK-8065870
>>>
>>> Could I ask for a review?
>>>
>>> It seems quite big (1126 lines changed) but there are just minor changes/fixes.
>>>
>>> Thanks
>>> Miran
>