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

Philip Race philip.race at oracle.com
Tue Nov 24 19:30:18 UTC 2015

On 11/24/15, 11:20 AM, Sergey Bylokhov wrote:
> On 23.11.15 23:20, Philip Race wrote:
>> Not sure if you are suggesting TK.createImage migrate to Image I/O to
>> keep them in sync ?
> Yes, I suggest to drop the old decoders and replace them by imageio.

As I mentioned in a separate thread there are some significant differences
in these two APIs so this is not a simple "swap".

>> Note that already BMP and WBMP are supported only via image I/O and
>> there is
>> no notion of required equivalence between these two image loading
>> mechanisms.
>> -phil.
>> On 11/23/15, 11:49 AM, Sergey Bylokhov wrote:
>>> 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

More information about the 2d-dev mailing list