RFR: 8307160: Fix AWT/2D/A11Y to support the permissive- flag on the Microsoft Visual C compiler [v14]

Julian Waters jwaters at openjdk.org
Fri Nov 3 02:43:16 UTC 2023


On Thu, 2 Nov 2023 21:10:45 GMT, Phil Race <prr at openjdk.org> wrote:

>> Julian Waters has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Changes awt_Frame.cpp
>
> src/java.desktop/windows/native/libawt/windows/awt_PrintJob.cpp line 61:
> 
>> 59: 
>> 60: jfieldID AwtPrintDialog::pageID;
>> 61: 
> 
> where and why did this come from ?

This came from below, all I did was move it up and out of the extern "C" block. This cannot be inside extern "C" because it is a static class member and has permanent C++ name mangling

> src/java.desktop/windows/native/libawt/windows/awt_PrintJob.cpp line 533:
> 
>> 531: 
>> 532:     {
>> 533:         HWND parentID = AwtPrintControl::getParentID(env, self);
> 
> diff is making this really hard to compare line by line
> So here you've just put the code in { .. } and indented ? 
> What is the warning that motivates this ?
> I don't see where you describe it.

I haven't gotten around to converting this to the new approach of splitting declaration and assignment, but yes, that is all I've done. I've described the problem here above

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15096#discussion_r1381112565
PR Review Comment: https://git.openjdk.org/jdk/pull/15096#discussion_r1381113001


More information about the client-libs-dev mailing list