On Fri, 16 Jan 2026 01:08:27 GMT, Phil Race <[email protected]> wrote:
> This PR removes usage of AppContext from classes under javax/swing/text.
> Most of the uses were added in specific bug fixes 10-15 years ago but are now
> obsolete and the tests for those bugs need to be deleted too.
src/java.desktop/share/classes/javax/swing/text/LayoutQueue.java line 58:
> 56: public static LayoutQueue getDefaultQueue() {
> 57: if (defaultQueue == null) {
> 58: defaultQueue = new LayoutQueue();
synchronized not needed?
src/java.desktop/share/classes/javax/swing/text/html/HTMLEditorKit.java line
434:
> 432: */
> 433: public void setStyleSheet(StyleSheet s) {
> 434: defaultStyles = s;
Dont we need to add `synchronized` for setter and getter? getAppContext used to
use getAppContextLock!!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29259#discussion_r2703481945
PR Review Comment: https://git.openjdk.org/jdk/pull/29259#discussion_r2703478833
