<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
One "PS" <br>
why say
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
<span class="changed">4294967295</span> in the spec where 0xFFFFFFFF
is probably<br>
more immediately to most programmers that it is not a random<br>
choice of number ?<br>
<br>
-phil<br>
<br>
<br>
On 12/7/16, 4:56 PM, Philip Race wrote:
<blockquote cite="mid:5848AFCD.6010304@oracle.com" type="cite">+1
<br>
<br>
Yes, it needs a CCC update. A quick "bug fix" one.
<br>
<br>
-phil.
<br>
<br>
On 12/6/16, 2:03 PM, Brian Burkhalter wrote:
<br>
<blockquote type="cite">Please review at your convenience:
<br>
<br>
Issue: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8169728">https://bugs.openjdk.java.net/browse/JDK-8169728</a>
<br>
Patch: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~bpb/8169728/webrev.00/">http://cr.openjdk.java.net/~bpb/8169728/webrev.00/</a>
<br>
<br>
To the primary constructor TIFFField(TIFFTag,int,int,Object),
add some range checks for TIFF_LONG, TIFF_RATIONAL, and
TIFF_IFD_POINTER, and some array size checks for TIFF_RATIONAL
and TIFF_SRATIONAL.
<br>
<br>
Does the new IllegalArgumentException case warrant a CCC review?
<br>
<br>
Thanks,
<br>
<br>
Brian
<br>
</blockquote>
</blockquote>
</body>
</html>