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

[Miroslav Kos]

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