Re: [9] Review request: 8091832: Provide API for getting the Screen scale on HiDPI screens

2016-03-30 Thread Jim Graham
Windows provides separate values for the X & Y direction. All of their system APIs that deal with screen and window resolutions provide both values... ...jim On 3/30/2016 12:12 AM, Dr. Michael Paus wrote: Hi, I'd like to know on which systems this distinction between X- and Y

Re: review: remove a cygwin dependancy

2016-03-30 Thread David Hill
On 3/30/16, 1:34 PM, David Hill wrote: Kevin, I have been running with this for a couple of weeks now. remove a cygwin dependancy https://bugs.openjdk.java.net/browse/JDK-8151747 http://cr.openjdk.java.net/~ddhill/8151747/ I updated the webrev in place, removing the unneeded ByteArrayOutpu

review: Add a command line toggle for worker.debug

2016-03-30 Thread David Hill
Kevin, https://bugs.openjdk.java.net/browse/JDK-8153110 Add a command line toggle for worker.debug There is a diff inline in the bug that works for toggling worker.debug from the gradle command line -- David Hill Java Embedded Development "A man's feet should be planted in his country, but h

review: javafx.swing system test RT30650Test fails in jigsaw mode

2016-03-30 Thread David Hill
Kevin, you were right about: https://bugs.openjdk.java.net/browse/JDK-8152898 just need an add exports: diff -r 1a6124ade5b4 tests/system/src/test/addExports --- a/tests/system/src/test/addExports Wed Mar 30 13:58:30 2016 -0400 +++ b/tests/system/src/test/addExports Wed Mar 30 14:03:58 2016 -04

Re: review: remove a cygwin dependancy

2016-03-30 Thread David Hill
On 3/30/16, 2:02 PM, Ali Ebrahimi wrote: Hi, line ByteArrayOutputStream out = new ByteArrayOutputStream(); can be removed! Good catch! thanks On Wed, Mar 30, 2016 at 10:04 PM, David Hill mailto:david.h...@oracle.com>> wrote: Kevin, I have been running with this for a couple of w

Re: review: remove a cygwin dependancy

2016-03-30 Thread Ali Ebrahimi
Hi, line ByteArrayOutputStream out = new ByteArrayOutputStream(); can be removed! On Wed, Mar 30, 2016 at 10:04 PM, David Hill wrote: > > Kevin, >I have been running with this for a couple of weeks now. > > remove a cygwin dependancy > https://bugs.openjdk.java.net/browse/JDK-8151747 > h

review: Conditionally save policy file in Sandbox tests

2016-03-30 Thread David Hill
Kevin, Conditionally save policy file in Sandbox tests https://bugs.openjdk.java.net/browse/JDK-8153086 http://cr.openjdk.java.net/~ddhill/8153086/ a small change, but very helpful when debugging the Sandbox test apps -- David Hill Java Embedded Development "A man's feet should be planted in

review: remove a cygwin dependancy

2016-03-30 Thread David Hill
Kevin, I have been running with this for a couple of weeks now. remove a cygwin dependancy https://bugs.openjdk.java.net/browse/JDK-8151747 http://cr.openjdk.java.net/~ddhill/8151747/ -- David Hill Java Embedded Development "A man's feet should be planted in his country, but his eyes should

Re: [9] Review request: 8091832: Provide API for getting the Screen scale on HiDPI screens

2016-03-30 Thread Dr. Michael Paus
Hi, I'd like to know on which systems this distinction between X- and Y-direction is actually relevant. I've never seen such a system before. Michael Am 29.03.16 um 03:25 schrieb Jim Graham: bug: https://bugs.openjdk.java.net/browse/JDK-8091832 webrev: http://cr.openjdk.java.net/~flar/JDK-8091

Re: [9] Review request for 8152315 : Don't server / request the Cached resource for if its Main resource is explicitly cancelled

2016-03-30 Thread Guru Hb
Updated webrev : http://cr.openjdk.java.net/~ghb/8152315/webrev.01/ Which address heap corruption detected by using GFlag On 22/3/16 12:54 AM, Guru Hb wrote: Hi Alexander, Arunprasad & Kevin, JBS : https://bugs.openjdk.java.net/browse/JDK-8152315 Webrev : http://cr.openjdk.java.net/~ghb/8152315