Re: [OpenJDK 2D-Dev] [9] RFR JDK-8168609: No link to BMP specification in javax.imageio package documentation

2016-10-27 Thread Phil Race
+1 -phil On 10/27/2016 08:16 AM, Brian Burkhalter wrote: Hi Jay, It looks OK to me. It would be good also to see what Phil or others think. Thanks, Brian On Oct 27, 2016, at 12:08 AM, Jayathirth D V mailto:jayathirth@oracle.com>> wrote: Thanks for your inputs. I have updated the web

Re: [OpenJDK 2D-Dev] [9] RFR JDK-8168609: No link to BMP specification in javax.imageio package documentation

2016-10-27 Thread Brian Burkhalter
Hi Jay, It looks OK to me. It would be good also to see what Phil or others think. Thanks, Brian On Oct 27, 2016, at 12:08 AM, Jayathirth D V wrote: > Thanks for your inputs. > I have updated the webrev to include updated hyperlink and copyright year. > Please review it at your convenience: >

Re: [OpenJDK 2D-Dev] [9] RFR JDK-8168609: No link to BMP specification in javax.imageio package documentation

2016-10-27 Thread Jayathirth D V
/~jdv/8168609/javadoc.01/package.html Thanks, Jay From: Brian Burkhalter Sent: Wednesday, October 26, 2016 8:44 PM To: Jayathirth D V Cc: Philip Race; Vadim Pakhnushev; 2d-dev Subject: Re: [OpenJDK 2D-Dev] [9] RFR JDK-8168609: No link to BMP specification in javax.imageio package

Re: [OpenJDK 2D-Dev] [9] RFR JDK-8168609: No link to BMP specification in javax.imageio package documentation

2016-10-26 Thread Brian Burkhalter
Hi Jay, Note that [1] use the same link minus the version number [2]. I wonder whether it would be better to do that? Also I noticed that I forgot to update the copyright year when I checked in the same file yesterday so you might want to update it now. Thanks, Brian [1] http://www.digitalpr

[OpenJDK 2D-Dev] [9] RFR JDK-8168609: No link to BMP specification in javax.imageio package documentation

2016-10-26 Thread Jayathirth D V
Hello All, Please review the following api-doc-only change to the javax.imageio package summary in JDK9: Bug : https://bugs.openjdk.java.net/browse/JDK-8168609 Webrev : http://cr.openjdk.java.net/~jdv/8168609/webrev.00/