<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Barring objections to the contrary I am inclined to go with webrev.00. There is no precedent in the public javax.imageio.plugins classes for making getters and the like final. Note also that BMPImageWriteParam, JPEGImageReadParam, and JPEGImageWriteParam and not final either.<div><br></div><div>Thanks,</div><div><br></div><div>Brian<br><div><div><br><div><div>On Oct 18, 2016, at 2:24 PM, Brian Burkhalter <<a href="mailto:brian.burkhalter@oracle.com">brian.burkhalter@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;">One could potentially change all the public getters in non-final classes in javax.imageio.plugins.tiff to be final as in</span><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br></div><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><a href="http://cr.openjdk.java.net/~bpb/8165981/webrev.01/">http://cr.openjdk.java.net/~bpb/8165981/webrev.01/</a><br><div><br></div><div>or go even further and additionally refactor TIFFDirectory to be an interface implemented by the internal TIFFIFD class. In the interest of mitigating to an extent the difficulty encountered by any users potentially migrating from the original JAI Image I/O Tools package, I would however be more in favor of keeping the original proposal in this thread</div><div><br></div><div><a href="http://cr.openjdk.java.net/~bpb/8165981/webrev.00/">http://cr.openjdk.java.net/~bpb/8165981/webrev.00/</a></div><div><br></div><div>wherein only TIFFField and TIFFImageReadParam are made final.</div></div></blockquote></div><br></div></div></div></body></html>