RFR: 8307160: Fix AWT/2D/A11Y to support the permissive- flag on the Microsoft Visual C compiler [v46]
    Magnus Ihse Bursie 
    ihse at openjdk.org
       
    Fri Mar 22 12:31:48 UTC 2024
    
    
  
On Sat, 20 Jan 2024 00:32:19 GMT, Phil Race <prr at openjdk.org> wrote:
>> Julian Waters has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 79 commits:
>> 
>>  - Merge branch 'openjdk:master' into patch-10
>>  - Merge branch 'openjdk:master' into patch-10
>>  - Fix awt_Window.cpp
>>  - Fix awt_PrintJob.cpp
>>  - -Zc:stringStrings no longer needed with -permissive- flags-cflags.m4
>>  - Fix awt_Window.cpp
>>  - awt_Window.cpp
>>  - awt_PrintJob.cpp
>>  - awt_Frame.cpp
>>  - Whitespace awt_Component.cpp
>>  - ... and 69 more: https://git.openjdk.org/jdk/compare/35e96627...cbfbaee6
>
> src/java.desktop/windows/native/libawt/windows/awt_Frame.cpp line 1365:
> 
>> 1363:     f = (AwtFrame *) pData;
>> 1364:     HWND hwnd = f->GetHWnd();
>> 1365:     if (::IsWindow(hwnd)) {
> 
> more formatting to revert
Suggestion:
    if (::IsWindow(hwnd))
    {
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15096#discussion_r1535491578
    
    
More information about the build-dev
mailing list