On Sun, 30 Apr 2023 07:53:06 GMT, Alan Bateman <al...@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
