Thanks Lance!
Joe
On 8/31/16, 1:14 PM, Lance Andersen wrote:
Looks OK Joe
Best
Lance
On Aug 29, 2016, at 3:18 PM, Joe Wang <[email protected]
<mailto:[email protected]>> wrote:
Hi,
Please review a patch that fixes an issue where circular references
were not caught if catalogs are pre-loaded. Note that to trigger the
issue (circular references), the Catalog must be instructed to load
additional catalogs. This can be done by either setting DEFER to
false, or providing an uri that does not have a match in the current
catalog (thus causes the CatalogResolver to load alternative
catalogs). When the uri is empty as in the original JCK test, the
CatalogResolver will skip the resolution and therefore no additional
catalogs will be loaded. In the unit test, the test (copied from the
JCK test) is fixed by adding a bogus uri 'anyuri'. The JCK test will
need a similar fix.
JBS: https://bugs.openjdk.java.net/browse/JDK-8162431
webrevs: http://cr.openjdk.java.net/~joehw/jdk9/8162431/webrev/
<http://cr.openjdk.java.net/%7Ejoehw/jdk9/8162431/webrev/>
Thanks,
Joe
<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
[email protected] <mailto:[email protected]>
