On Mon, 23 Jun 2025 19:42:10 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> Phil Race has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   8359053
>
> src/java.desktop/share/classes/java/awt/doc-files/Modality.html line 352:
> 
>> 350:       <code>Dialog(owner, true)</code>, etc. Prior to JDK 6
>> 351:       the default type was toolkit-modal,
>> 352:       and now with single application per-VM there is no
> 
> Why is it “per-VM” instead of “per VM”?
> 
> “…Single application per VM…”, in this sentence “per” is a preposition and 
> “VM” is a noun, don't you agree? There should be a space, not a hyphen.
> 
> If it were “per-VM application”, then it would be spelt with a hyphen.

Right, "per VM" means "for each VM". This is a minor mistake I've made many 
times
Suggestion:

      and now with single application per VM, there is no

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25698#discussion_r2197900689

Reply via email to