Miran, Did you use 'hg mv’? if so, then webrev ( without a file.list ) typically picks up that the file has moved and only highlights the differences. This typically helps with the review.
-Chris. On 24 Sep 2015, at 14:57, Miroslav Kos <miroslav....@oracle.com> wrote: > Hi everybody, > > I'd like to ask for review of following fix: > > RELAX NG API visible but not accessible > JBS: https://bugs.openjdk.java.net/browse/JDK-8061466 > webrev: http://cr.openjdk.java.net/~mkos/8061466/jaxws.01/index.html > > - org.relaxng.* classes in jdk.xml.bind module are not repackaged, but those > are not exported so they are not usable outside of module (jdk) anyway > - processing RelaxNG schemas with JAX-B's declared as > experimental/unsupported, but removing these classes is not easy since > several other packages within JAX-B depend on those ... > > This patch repackages these classes from org.relaxng.* to > com.sun.xml.internal.org.relaxng.* + removes loading RelaxNG runtime in xjc > (+ removing related messages). > > Testing: snadalone JAX-B/WS tests, JAX-WS JCK, jtreg, no regressions. > > Thanks > Miran > >