RFR: 8065870 Update JAX-WS RI integration to latest version (2.2.11-b141124.1933)
stuart.marks at oracle.com
Mon Dec 1 19:35:38 UTC 2014
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
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
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 (') ?
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.
Copyright years 2014 => 2012 again. Also check localized versions of this file.
Copyright years 2013 => 2012. Possibly incorrect replacement "???" for unknown
character in original file.
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.
On 11/27/14 3:27 AM, Miroslav Kos wrote:
> 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.
More information about the core-libs-dev