<html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
I submitted <a class="moz-txt-link-freetext" href="https://bugs.openjdk.org/browse/JDK-8306320">https://bugs.openjdk.org/browse/JDK-8306320</a><br>
<br>
The PR should use that bug ID and the summary fom the bug.<br>
We will need a CSR.<br>
<br>
-phil.<br>
<br>
<div class="moz-cite-prefix">On 4/12/23 3:08 AM, Martin Desruisseaux
wrote:<br>
</div>
<blockquote type="cite" cite="mid:43709e12-9664-84ed-b4ef-348edd78f24d@geomatys.com">
<p>Hello all</p>
<p>BufferedImage implements WritableRenderedImage, but the javadoc
of all WritableRenderedImage methods are inconsistent with
implementation:</p>
<ul>
<li>getWritableTile(…) said "All registered TileObservers are
notified when a tile goes from having no writers to having one
writer" but the implementation doesn't do that.</li>
<li>releaseWritableTile(…) said "All registered TileObservers
are notified when a tile goes from having one writer to having
no writers" but the implementation does nothing.</li>
<li>isTileWritable(…) said "throws
ArrayIndexOutOfBoundsException if both tileX and tileY are not
equal to 0" but the implementation throws
IllegalArgumentException is *either* tileX or tileY is not 0.</li>
<li>addTileObserver(…) and removeTileObserver(…) are no-op, but
the javadoc don't said that.<br>
</li>
</ul>
<p>Should the javadoc be updated for saying that most methods are
actually no-op, or should the implementation be updated with
real support of listeners? Or should BufferedImage be considered
as permanently checkout out for writing, in which case the
current javadoc is not wrong (except above-cited throws clause)
but a note about the "permanently checkout-out" behavior would
be helpfui?<br>
</p>
<p> Martin</p>
<p><br>
</p>
</blockquote>
<br>
</body>
</html>