On Sat, 11 Mar 2023 05:34:39 GMT, Joe Wang <jo...@openjdk.org> wrote:

>> Add a system property, jdk.xml.config.file, to return the path to a custom 
>> JAXP configuration file. The current configuration file, jaxp.properties, 
>> that the JDK supports will become the default configuration file.
>> 
>> CSR: https://bugs.openjdk.org/browse/JDK-8303531
>> 
>> Tests: XML SQE and JCK tests passed.
>
> Joe Wang has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   clean up tests; fix copy&paste error.

src/java.xml/share/classes/javax/xml/stream/XMLEventFactory.java line 139:

> 137:    *   Use the value of the property {@code factoryId} set in the
> 138:    * <a 
> href="../../../module-summary.html#ConfigurationFile">configuration file</a>,
> 139:    * stax.properties or jaxp.properties by default. If the file exists 
> and

Is it time to drop all references to stax.properties from the API docs?

src/java.xml/share/classes/module-info.java line 89:

> 87:  * the StAX factories for the <a href="#LookupMechanism">Factory Lookup 
> Mechanism</a>
> 88:  * only. The files, if exist, are typically located in the {@code conf}
> 89:  * directory of the Java installation.

If jdk.xml.config.file is not set, should the reader assume it's default value 
is ${java.home}/conf/jaxp.properties? If so, they this is an implementation 
specific default, right?

-------------

PR: https://git.openjdk.org/jdk/pull/12985

Reply via email to