Re: [OpenJDK 2D-Dev] RFR 8149028: [TEST] add test for TIFFDirectory

2016-04-01 Thread Yuri Nesterenko
+1 -yan On 03/31/2016 06:52 PM, Alexander Stepanov wrote: Please see http://cr.openjdk.java.net/~avstepan/8149028/webrev.02/ (one more test covering TIFFImageReadParam was added). Thanks, Alexander On 3/29/2016 8:21 PM, Alexander Stepanov wrote: Please see the updated webrev: http://cr.openj

Re: [OpenJDK 2D-Dev] RFR 8149028: [TEST] add test for TIFFDirectory

2016-04-01 Thread Alexander Stepanov
Thanks! On 3/31/2016 8:10 PM, Semyon Sadetsky wrote: Looks good. --Semyon On 3/31/2016 6:52 PM, Alexander Stepanov wrote: Please see http://cr.openjdk.java.net/~avstepan/8149028/webrev.02/ (one more test covering TIFFImageReadParam was added). Thanks, Alexander On 3/29/2016 8:21 PM, Alexand

Re: [OpenJDK 2D-Dev] RFR 8149028: [TEST] add test for TIFFDirectory

2016-03-31 Thread Semyon Sadetsky
Looks good. --Semyon On 3/31/2016 6:52 PM, Alexander Stepanov wrote: Please see http://cr.openjdk.java.net/~avstepan/8149028/webrev.02/ (one more test covering TIFFImageReadParam was added). Thanks, Alexander On 3/29/2016 8:21 PM, Alexander Stepanov wrote: Please see the updated webrev: http

[OpenJDK 2D-Dev] RFR 8149028: [TEST] add test for TIFFDirectory

2016-03-31 Thread Alexander Stepanov
Please see http://cr.openjdk.java.net/~avstepan/8149028/webrev.02/ (one more test covering TIFFImageReadParam was added). Thanks, Alexander On 3/29/2016 8:21 PM, Alexander Stepanov wrote: Please see the updated webrev: http://cr.openjdk.java.net/~avstepan/8149028/webrev.01/ (one more test was a

[OpenJDK 2D-Dev] RFR 8149028: [TEST] add test for TIFFDirectory

2016-03-29 Thread Alexander Stepanov
Please see the updated webrev: http://cr.openjdk.java.net/~avstepan/8149028/webrev.01/ (one more test was added). Thanks, Alexander On 3/25/2016 6:46 PM, Alexander Stepanov wrote: Hello, Could you please review the following fix http://cr.openjdk.java.net/~avstepan/8149028/webrev.00/ for https

[OpenJDK 2D-Dev] RFR 8149028: [TEST] add test for TIFFDirectory

2016-03-25 Thread Alexander Stepanov
Hello, Could you please review the following fix http://cr.openjdk.java.net/~avstepan/8149028/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8149028 Just a single test added. Thanks, Alexander