On Mon, 23 Jun 2025 18:58:32 GMT, Phil Race <p...@openjdk.org> wrote:
>> This is the implementation of JEP 504 - Remove the Applet API. >> API changes are >> - Remove the entire java.applet package >> - Remove the javax/swing/JApplet class >> - Remove applet related APIs in java.beans >> - Update javadoc referring to applets, including one gif image - now changed >> to an svg image >> - >> Other changes are >> - Remove references to the removed classes >> - Remove obsolete tests >> - Update obsolete code comments >> >> sun.awt.AppContext is even more obsolete now than it was before, but >> eliminating uses of that will be is not required, >> and will be follow-on internal clean up, at a later date, under unrelated >> bug ids, and likely not completed in the same >> release as this JEP is integrated. >> >> I have extensively tested this - running all the automated tests used by CI >> tiers 1 to 8. > > Phil Race has updated the pull request incrementally with one additional > commit since the last revision: > > 8359053 Marked as reviewed by aivanov (Reviewer). 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. ------------- PR Review: https://git.openjdk.org/jdk/pull/25698#pullrequestreview-2951339948 PR Review Comment: https://git.openjdk.org/jdk/pull/25698#discussion_r2162407407
