Just a nit:

> +    private boolean useCatalogResolver = false;
> +    private EntityResolver entityResolver = null;
> +    private URIResolver uriResolver = null;

Those fields are being initialized to their default values. The Java
Language Specification states [1] that every field must be initialized
with a default value, basically 0 for numbers, false for booleans, and
null for objects. So explicitly initializing them with their default
values is just noise.
I’d like to suggest everyone to remove those unnecessary initializations
in the future.



Reply via email to