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
