Hello Sergey, Semyon,

Thanks!

Regards,
Alexander

On 1/22/2016 9:54 AM, Semyon Sadetsky wrote:
Looks good to me.

--Semyon

On 1/19/2016 7:17 PM, Alexander Stepanov wrote:
Hello Sergey,

please find the updated webrev:
http://cr.openjdk.java.net/~avstepan/8146881/webrev.01/index.html

Regards,
Alexander

On 1/18/2016 5:12 PM, Sergey Bylokhov wrote:
On 15/01/16 15:27, Alexander Stepanov wrote:
Hello Sergey,

 > Why this test do not cover the gif/bmp as well?
I can try to append the test, but it will fail for pair bmp +
TYPE_INT_ARGB due to
https://bugs.openjdk.java.net/browse/JDK-8147448

So we found a bug in jdk? I guess you can update the test to cover only working set of parameters and add a diff which should be made in the test to catch the problem. This is why, I always suggest[1] to validate all supported formats(spi).

[1] http://mail.openjdk.java.net/pipermail/2d-dev/2016-January/006168.html


Regards,
Alexander

On 1/12/2016 5:49 PM, Sergey Bylokhov wrote:
Hi, Alexander.
BitDepth.java the @summary should be updated. Why this test do not
cover the gif/bmp as well?
It is unclear why biRGBATypes are unused in the test.

On 12/01/16 17:40, Alexander Stepanov wrote:
Please ignore mistaken changes in ImageWriterCompressionTest
(re-uploading the webrev...)

Thanks,
Alexander

On 1/12/2016 5:28 PM, Alexander Stepanov wrote:
Hello,

Some imageio plugins reg. tests should probably be updated to reflect
the fact that TIFF format is supported now.

Could you please let me know if the following changes are appropriate?
http://cr.openjdk.java.net/~avstepan/8146881/webrev.00/index.html
(bug: https://bugs.openjdk.java.net/browse/JDK-8146881)

Thanks,
Alexander









Reply via email to