Re: [10] Review Request: JDK-8145795 : [PIT] java/awt/Window/ScreenLocation/ScreenLocationTest.java fails (can assign Integer.MAX_VALUE to Window dimensions)

2017-10-17 Thread Sergey Bylokhov
On 17/10/2017 04:48, Pankaj Bansal wrote: I can see that XDragSourceContextPeer.java also contains ScaleUp and ScaleDown functions, but I don't see them being called from anywhere. So have not made changes in the same. Looks like XDragSourceContextPeer.scaleUp() is unused, but scaleDown() is

Re: [10] Review request for 8148619: Select the closest resolution variant in BaseMultiResolutionImage

2017-10-17 Thread Phil Race
This class which is concrete - not abstract - would have been better called "BasicMultiResolutionImage" I do think I have issues with the documentation on this class and some aspects of its behaviour that should be documented and implemented, but I don't think we should try to make it somethi

Re: [10] Review request for 8182043: Access to Windows Large Icons

2017-10-17 Thread Alexey Ivanov
Hi Semyon, On 06/10/2017 21:33, Semyon Sadetsky wrote: Hi Alexey, On 10/06/2017 11:42 AM, Alexey Ivanov wrote: Hi Semyon, Sergey, On 30/09/2017 00:08, Semyon Sadetsky wrote: On 9/29/2017 3:15 PM, Sergey Bylokhov wrote: On 9/29/17 12:39, Semyon Sadetsky wrote: Why 128 pixels? Windows shell u

Re: [10] Review request for 8182043: Access to Windows Large Icons

2017-10-17 Thread Alexey Ivanov
Hi Sergey, Semyon, On 12/10/2017 21:39, Sergey Bylokhov wrote: On 06/10/2017 17:16, Semyon Sadetsky wrote: The maximum icon which we used before you fix is 32pixel's icon(yes it is a large icon), and 128 is a size of this icon on 4k monitor( The windows can return a 128pixel's icon on 4k monit

Re: [10] Review request for 8074824: Resolve disabled warnings for libawt_xawt

2017-10-17 Thread Semyon Sadetsky
Hi Ajit, Thank you for the information. I ran SwingSet under GTK2. No issues found. --Semyon On 10/16/2017 09:41 PM, Ajit Ghaisas wrote: Hi Semyon, I had attempted this fix some time back and had run into some regression for SwingSet2. There was a crash while using SwingSet2 with G

Re: [10] Review request for 8074824: Resolve disabled warnings for libawt_xawt

2017-10-17 Thread Semyon Sadetsky
Hi Sergey, On 10/16/2017 04:45 PM, Sergey Bylokhov wrote: Hi, Semyon. A few notes:> I ran OGL tests and found a typo. Below the update webrev> http://cr.openjdk.java.net/~ssadetsky/8074824/webrev.01/  - What is a typo? It seems the v0 and v1 are identical. It's in line 516 of the OGLPaints.c

Re: [10] Review Request: JDK-8145795 : [PIT] java/awt/Window/ScreenLocation/ScreenLocationTest.java fails (can assign Integer.MAX_VALUE to Window dimensions)

2017-10-17 Thread Pankaj Bansal
Hi Sergey, Yes, similar changes have to be made in XToolkit also. What happens in XToolkit is that after the int overflow, when garbage width and height is passed to XCreateWindow, it handles it properly at its own end. So the test passes without any fix also. But the into overflow should not h

Re: [10] Review request for JDK-8163265: [macosx] numpad 0 instead of VK_0

2017-10-17 Thread Manajit Halder
Thank you Sergey and Alex, I will update @bugid in the test before push. Regards, Manajit > On 17-Oct-2017, at 2:37 PM, Alexander Zvegintsev > wrote: > > +1 > > Thanks, > Alexander. > > On 17/10/2017 03:12, Sergey Bylokhov wrote: >> Looks fine. >> Please update the @bugid in the test befor

Re: [10] Review request for JDK-8163265: [macosx] numpad 0 instead of VK_0

2017-10-17 Thread Alexander Zvegintsev
+1 Thanks, Alexander. On 17/10/2017 03:12, Sergey Bylokhov wrote: Looks fine. Please update the @bugid in the test before the push. On 16/10/2017 06:38, Manajit Halder wrote: Hi All, Kindly review the fix for JDK10. Bug: https://bugs.openjdk.java.net/browse/JDK-8163265 Webrev: http://cr.op