On Sun, 14 Aug 2022 23:53:21 GMT, Iris Clark <[email protected]> wrote:
>> Thread.stop() and ThreadGroup.destroy() are called by >> sun.awt.AppContext.dispose() >> >> These should no longer be called. Both are deprecated for removal. >> ThreadGroup.destroy() does nothing. Thread.stop() throws >> UnsupportedOperationException >> for virtual threads, and is expected to do so for all threads if it is not >> removed before then. >> >> This fix stops calling these methods and prints a warning from dispose() to >> encourage >> callers to move away from AppContext which itself will likely be removed in >> a future release >> since the Webstart and Plugin scenarios that needed it are no longer >> supported. >> Although most tests that use AppContext are now irrelevant only one test >> actually fails >> and is removed so we are doing the least here and the big job of completely >> getting rid of >> AppContext is for another day. > > src/java.desktop/share/classes/sun/awt/AppContext.java line 406: > >> 404: Remove all uses of this internal class as soon as possible. >> 405: There is no replacement. >> 406: """); > > Should some portion of this warning be included in an `@implNote` in this > method's JavaDoc? Since it is internal API I think that unnecessarily generous. It shouldn't be called anyway. ------------- PR: https://git.openjdk.org/jdk/pull/9863
