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/

Thanks,
Joe

Reply via email to