Hi Prasanta, > -----Original Message----- > From: Prasanta Sadhukhan > Sent: Monday, October 29, 2018 12:18 PM > To: Fairoz Matte <fairoz.ma...@oracle.com>; 2d-dev@openjdk.java.net > Subject: Re: [OpenJDK 2D-Dev] [8u] RFR: 8212914: Test > javax/imageio/plugins/bmp/BMP8BPPLoadTest.java fails > > Hi Fairoz, > > I do not see ImageIO.read and ImageIO.createImageInputStream throwing > IOB exception in the spec, it throws IOException so I guess there's no point > catching IOB.
This test case has been added as part of "JDK-8182461: IndexOutOfBoundsException when reading indexed color BMP" Due to missing "break" IOB exception was generated. > Also, you need to add this bugid to @bug tag and also remove @author tag > which we do not recommend now. Also, can you please indent byte[] data ? Yes updated all, here is the updated webrev. http://cr.openjdk.java.net/~fmatte/8212914/webrev.02/ Thanks, Fairoz > > Regards > Prasanta > On 26-Oct-18 10:03 PM, Fairoz Matte wrote: > > Hi Prasanta, > > > > Thanks for looking into it. > > > >> -----Original Message----- > >> From: Prasanta Sadhukhan > >> Sent: Friday, October 26, 2018 8:56 AM > >> To: Fairoz Matte <fairoz.ma...@oracle.com>; 2d-dev@openjdk.java.net > >> Subject: Re: [OpenJDK 2D-Dev] [8u] RFR: 8212914: Test > >> javax/imageio/plugins/bmp/BMP8BPPLoadTest.java fails > >> > >> Hi Fairoz, > >> > >> Do you know if the bmp image file has Oracle copyright? If not, you > >> cannot check it in. > > I was not aware of it. Yes image file is not Oracle copyright compliant. > > > >> Alternatively, you may get a hexdump of the bmp file and create a > >> byte[] array with that hex data and create ByteArrayInputStream with > >> that and use that for ImageIO as an ImageInputStream. > > Yes it is good approach, here is the updated webrev > > http://cr.openjdk.java.net/~fmatte/8212914/webrev.01/ > > > > Thanks, > > Fairoz > >> Regards > >> Prasanta > >> On 25-Oct-18 11:52 AM, Fairoz Matte wrote: > >>> Hi, > >>> > >>> Kindly review the small fix. > >>> > >>> Background: > >>> "javax/imageio/plugins/bmp/BMP8BPPLoadTest.java test case" has > been > >>> added part of JDK-8182461, Test case has a dependency on input file > >> "BMP8BPPLoadTest.PNG", during push this was missed. > >>> In this fix test case also modified to refer the input file from > >>> same directory > >>> > >>> JBS bug - https://bugs.openjdk.java.net/browse/JDK-8212914 > >>> Webrev - http://cr.openjdk.java.net/~fmatte/8212914/webrev.00/ > >>> > >>> Thanks, > >>> Fairoz > >>> > >>> >