This seems fine to me. Perhaps the test could be reworked to
look for any plugin that reports a duplicate compression type ?
That may be more useful as a test ..
Is this the only plugin that exhibits such duplication ?
-phil.
On 10/04/2016 04:39 AM, Jayathirth D V wrote:
Hi,
Please review the following fix in JDK9 at your convenience:
Bug : https://bugs.openjdk.java.net/browse/JDK-6294607
Webrev : http://cr.openjdk.java.net/~jdv/6294607/webrev.00/
<http://cr.openjdk.java.net/%7Ejdv/6294607/webrev.00/>
Issue : When we call ImageWriteParam.getCompressionTypes() for GIF it
returns two compression types “LZW” & “lzw”.
Root cause : Basically we support only LZW compression for GIF. In
ImageWriteParam, there are many places in specification where we
mention about multiple compression types. In case of GIF, since it
returns same compression type twice it would be misleading.
Solution : Update GIFImageWriter to return only one compression type
string.
Thanks,
Jay