On Mon, 9 Feb 2026 06:26:56 GMT, Damon Nguyen <[email protected]> wrote:

> In awt_Frame.cpp, there is a conditional that includes a "not-null" check in 
> a scenario where null is impossible. Starting on line 1589, each conditional 
> ends in a return statement. The only possibility in awt_Frame.cpp's line 1635 
> is peer is not null and f is not null. So, checking for `f != NULL` is 
> redundant and unnecessary. And the fix is simply to remove this extraneous 
> check in the conditional.

src/java.desktop/windows/native/libawt/windows/awt_Frame.cpp line 1635:

> 1633:     }
> 1634: 
> 1635:     if (::IsWindow(f->GetHWnd()))

please update the copyright year

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29625#discussion_r2784189094

Reply via email to