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

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: 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