[OpenJDK 2D-Dev] RFR: 8143342: Integrate Java Image I/O support for TIFF per JEP 262

[Sergey Bylokhov]

The fix looks fine.
Note that this change implement the new SPI readers&writers for ImageIO. 
But as far as I understand it does not add support of the tiff format to 
the Toolkit.createImage which use old decoders. I think it is necessary 
to delete support of old decoders and to replace it with spi? Any 
objections?

On 20.11.15 6:41, Philip Race wrote:
> The changes to pre-existing files look good although it is not apparent
> to me why you deleted this in iio-plugin.properties
>
> SimpleRenderedImage0=The provided region doesn't intersect with the
> image bounds
>
> is it just not used ?
>
> The rest I barely "skimmed" but I give it a go.
>
> -phil.
>
> On 11/19/15, 1:48 PM, Brian Burkhalter wrote:
>> Please review at your convenience:
>>
>> Issue:	https://bugs.openjdk.java.net/browse/JDK-8143342
>> Webrev:	http://cr.openjdk.java.net/~bpb/8143342/8143342.00/
>>
>> This review request pertains to the changes required to integrate Java Image I/O support for TIFF into JDK 9. The two branches of the webrev, 8143342.00-jdk and 8143342.00-jdkforest, are for the JDK repository and the top level JDK forest repository, respectively. The latter only adds the new package javax.imageio.plugins.tiff to the list of packages which form the Java API Specification.
>>
>> The 8143342.00-jdk branch contains the actual source code and HTML changes. There are three subsets of this branch: 1) changes to existing files, 2) new files in the public API, and 3) new files in the non-public implementation.
>>
>> The portions of the changes to existing files which affect visible API or the behavior of existing API are to add the TIFF reader and writer plugin SPIs to the IIORegistry and to link the TIFF Metadata Format Specification and Usage Notes document from the javax.imageio.metadata package summary.
>>
>> The new files in the public API are the classes in the javax.imageio.plugins.tiff package, the package summary, and the aforementioned metadata and usage document tiff_metadata.html. The new API is principally concerned with a simplified approach to handling TIFF image metadata. If metadata are not of concern then there should be no need to use the classes in this package.
>>
>> The new files in the non-public API are the classes in the com.sun.imageio.plugins.tiff package. These classes handle the reading and writing of TIFF image data and metadata.
>>
>> Thanks,
>>
>> Brian


-- 
Best regards, Sergey.