On Tue, 27 Jan 2026 06:06:46 GMT, Phil Race <[email protected]> wrote:

> Remove AppContext from ImageFetcher

Looks good to me.

You should bump the copyright year.

src/java.desktop/share/classes/sun/awt/image/ImageFetcher.java line 320:

> 318:     int numFetchers;
> 319:     int numWaiting;
> 320:     Vector<ImageFetchable> waitList;

I'd mark the `fetchers` and `waitList` fields as final, however, it should 
rather be done under a separate PR.

-------------

Marked as reviewed by aivanov (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/29441#pullrequestreview-3717775577
PR Review Comment: https://git.openjdk.org/jdk/pull/29441#discussion_r2737538043

Reply via email to