RFR: 8006421: GraphicsConfiguration of a frame is changed when the frame is moved to another screen [v5]
Tejesh R
tr at openjdk.org
Fri Jun 16 08:15:22 UTC 2023
On Thu, 15 Jun 2023 21:37:09 GMT, Alexander Zvegintsev <azvegint at openjdk.org> wrote:
>> Tejesh R has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Updated based on review comments
>
> src/java.desktop/macosx/classes/sun/lwawt/macosx/CFileDialog.java line 205:
>
>> 203: public GraphicsConfiguration getAppropriateGraphicsConfiguration(
>> 204: GraphicsConfiguration gc)
>> 205: {
>
> Could you please move the brace to the line 204 to match the overall style of the file?
Updated.
> src/java.desktop/unix/classes/sun/awt/X11/XWindowPeer.java line 240:
>
>> 238: XToolkit.awtUnlock();
>> 239: }
>> 240: }
>
> The same code is already in XCanvasPeer.
>
> XWindowPeer is a descendant of XCanvasPeer, there is no need to repeat the code here.
Updated.
> src/java.desktop/windows/classes/sun/awt/windows/WWindowPeer.java line 671:
>
>> 669: return gc;
>> 670: }
>> 671:
>
> Same here, WWindowPeer is a WCanvasPeer descendant.
Updated.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14246#discussion_r1231928415
PR Review Comment: https://git.openjdk.org/jdk/pull/14246#discussion_r1231928510
PR Review Comment: https://git.openjdk.org/jdk/pull/14246#discussion_r1231928769
More information about the client-libs-dev
mailing list