RFR: 8163232: CatalogResolver to extend XMLResolver and LSResourceResolver

Daniel Fuchs daniel.fuchs at oracle.com
Fri Aug 26 19:01:52 UTC 2016


Hi Joe,

Looks good to me!

-- daniel

On 26/08/16 18:06, Joe Wang wrote:
> Hi,
>
> Please review a consolidation of resolver support to CatalogResolver.
> The purpose is to simplify the interface and usage scenarios so that the
> one CatalogResolver is usable for all JDK XML processors. As with the
> existing resolver interfaces, the change makes it no distinction between
> system identifiers and URIs, which would make it easier for users to
> create catalogs (without worrying about when and why to use what type of
> catalog entries). The implementation is almost identical for all JDK XML
> resolver interfaces.
>
> New tests: added to CatalogTest.java to cover URIResolver (Transform),
> XMLResolver (StAX) and LSResourceResolver (Validation).
>                  new CatalogSupport5.java is added to cover error
> handling. When there is no match found, a processor shall return with
> the specified checked exception rather than CatalogException.
>
> Existing tests: mainly replaced CatalogUriResolver with CatalogResolver.
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8163232
> webrevs: http://cr.openjdk.java.net/~joehw/jdk9/8163232/webrev/
>
> Thanks,
> Joe



More information about the core-libs-dev mailing list