RFR: 8303530: Add system property for custom JAXP configuration file [v9]

Joe Wang joehw at openjdk.org
Tue May 2 19:30:19 UTC 2023


On Sun, 30 Apr 2023 07:53:06 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> src/java.xml/share/classes/module-info.java line 33:
>> 
>>> 31:  * JAXP supports the use of a configuration file for the
>>> 32:  * <a href="#LookupMechanism">Factory Lookup Mechanism</a> and
>>> 33:  * setting properties that have defined corresponding system properties.
>> 
>> The first sentence of the module description lists the 4 XML APIs and their acronyms. It is immediately followed by a section with title "JAXP Configuration File" which suggests that this is a configuration file for the first API that is listed (JAXP). As I understand it, properties for the streaming API can also be defined in this file. So it might be that a bit more setup is needed in the opening text to make it clearer that this is the configuration file for all of the APIs.
>
> One other thing to say on this is that jaxp.properties has "XML Library (java.xml) Configuration File" at the top of the file, maybe that could be used in the module description instead of "JAXP Configuration File".

Changed the opening statement and also the naming of config file as above.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/12985#discussion_r1182964917


More information about the core-libs-dev mailing list