Re: [OpenJDK 2D-Dev] [12] RFR(XS) JDK-8214817: Bad links in ImageInputStream.java & ImageOutputStream.java

2018-12-10 Thread Prasanta Sadhukhan
: [OpenJDK 2D-Dev] [12] RFR(XS) JDK-8214817: Bad links in ImageInputStream.java & ImageOutputStream.java Hi Jay, "modified UTF-8" had space before but it doesn't. Otherwise, looks good. Regards, Alexey On 05/12/2018 12:35, Jayathirth D V wrote: Hello All, Please review the followin

Re: [OpenJDK 2D-Dev] [12] RFR(XS) JDK-8214817: Bad links in ImageInputStream.java & ImageOutputStream.java

2018-12-07 Thread Alexey Ivanov
: [OpenJDK 2D-Dev] [12] RFR(XS) JDK-8214817: Bad links in ImageInputStream.java & ImageOutputStream.java Hi Jay, "modified UTF-8" had space before but it doesn't. Otherwise, looks good. Regards, Alexey On 05/12/2018 12:35, Jayathirth D V wrote: Hello All, Please review the followin

Re: [OpenJDK 2D-Dev] [12] RFR(XS) JDK-8214817: Bad links in ImageInputStream.java & ImageOutputStream.java

2018-12-07 Thread Jayathirth D V
Hi Alexey, Thanks for the review. I will add space before pushing the change. Regards, Jay -Original Message- From: Alexey Ivanov Sent: Thursday, December 06, 2018 12:45 AM To: Jayathirth D V; 2d-dev Subject: Re: [OpenJDK 2D-Dev] [12] RFR(XS) JDK-8214817: Bad links

Re: [OpenJDK 2D-Dev] [12] RFR(XS) JDK-8214817: Bad links in ImageInputStream.java & ImageOutputStream.java

2018-12-05 Thread Alexey Ivanov
Hi Jay, “modified UTF-8” had space before but it doesn't. Otherwise, looks good. Regards, Alexey On 05/12/2018 12:35, Jayathirth D V wrote: Hello All, Please review the following fix in JDK12: Bug : https://bugs.openjdk.java.net/browse/JDK-8214817 Webrev :