[OpenJDK 2D-Dev] RFR: 8149815: Misleading
Bluv Nallakaluva
bharath.nallakaluva at oracle.com
Thu May 19 18:06:44 UTC 2016
Hi Brian,
I think the change looks fine.
However I want confirm the reason for the check "type == TIFFTag.LONG" besides type == TIFFTag.TIFF_IFD_POINTER .
>From this http://partners.adobe.com/public/developer/en/tiff/TIFFPM6.pdf. , it is clear that IFD /LONG type can be used for the tag "SubIFD". (this entry points to a Thumbnail IFD)
Here the IFD entry corresponding to this tag should have the node name "TIFFIFD" which makes sense.
So there might be other IFD entries that would have a tag whose type can be LONG and still they have the value as the address to the actual IFD just like the above "SubIFD"
And for these entries the node name being "TIFFIFD" is valid.
Can you confirm if my understanding is correct ?
Also I think I caught one more place where this instanceof is used in TIFFFieldNode,
The following excerpt from TIFFFieldNode constructor. I believe this one should also be fixed.
isIFD = field.getData() instanceof TIFFDirectory;
Thanks,
Bharath N
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20160519/d232a4a9/attachment.html>
More information about the 2d-dev
mailing list