[OpenJDK 2D-Dev] Updated Review Request for JDK-6588417:

Phil Race philip.race at oracle.com
Sun Sep 14 14:19:08 UTC 2014


Hi,

MemoryCache doesn't really have I/O Exceptions but does
synthesise have to them  so perhaps wording in MemoryCache.java like

148      * @throws IOException if there is an I/O exception while writing to the
  149      * stream

isn't 100% descriptive of what actually happens but is probably OK per the contract.


So this change looks OK to me.

BTW
 >  Here is the updated Webrev for JDK-6588417:
I don't remember seeing any previous version and so initially thought it 
was going to be a followup to
http://mail.openjdk.java.net/pipermail/2d-dev/2014-August/004755.html 
which is still outstanding ...

-phil.

On 9/12/14 6:47 PM, Anisha Nagarajan wrote:
> Hi,
>
> Here is the updated Webrev for JDK-6588417:
> http://cr.openjdk.java.net/~ssides/6588417/6588417.1/ 
> <http://cr.openjdk.java.net/%7Essides/6588417/6588417.1/>
>
> Thanks,
> Anisha

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20140914/748ea77a/attachment.html>


More information about the 2d-dev mailing list