On Fri, 12 Aug 2022 20:14:27 GMT, Phil Race <[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. Thanks for doing this. The changes look reasonable to me and the warning will help catch any attempts to use it in the JDK. ------------- Marked as reviewed by alanb (Reviewer). PR: https://git.openjdk.org/jdk/pull/9863
