On Mon, 20 May 2024 12:55:24 GMT, Alan Bateman <al...@openjdk.org> wrote:

>> src/java.xml/share/classes/module-info.java line 444:
>> 
>>> 442:  *
>>> 443:  * Deploying with this configuration prevents processors from 
>>> unknowingly making
>>> 444:  * outbound network connections to fetch DTDs, or process XML that 
>>> makes use of
>> 
>> s/process/processing/
>
> In XML parlance, a "processor" is an aggregation of parsers, serializers, and 
> other things that contribute to the processing. So I think it could be either 
> here, but you have a point and if it stays as "processor" then it should link 
> #FacPro where the term is defined.

It's the wording that doesn't sound right, before this you have "making" which 
doesn't sound right with "process". Maybe it needs two sentences.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18831#discussion_r1606754542

Reply via email to