<Swing Dev> 8138771: java.awt.image.AbstractMultiResolutionImage needs customized spec for methods of Image which it implements

Avik Niyogi avik.niyogi at oracle.com
Mon Oct 31 06:53:21 UTC 2016


Hi All,
Please review the proposed specification for JDK9 including inputs from reviewer reviews.
cr.openjdk.java.net/~aniyogi/8138771/webrev.05/ <http://cr.openjdk.java.net/~aniyogi/8138771/webrev.05/>
Thank you in advance.

With Regards,
Avik Niyogi
> On 28-Oct-2016, at 1:18 am, Jim Graham <james.graham at oracle.com> wrote:
> 
> Hi Avik,
> 
> My suggestion about adding a word "the" was not taken and a couple of other changes were made to the @return statements which are not optimal.  Let's reset and use the following @return statements for each of the methods (to mirror the way these are described in the Image base class):
> 
> getWidth() - @return the width of the base image, or -1 if the width is not yet known
> getHeight() - @return the height of the base image, or -1 if the height is not yet known
> getGraphics() - @return throws {@code UnsupportedOperationException}
> getSource() - @return the image producer that produces the pixels for the base image
> getProperty() - @return the value of the named property in the base image
> 
> (It would also be nice if the blank lines were the same in all of the doc comments.  Some comments have a couple of blank lines to separate the javadoc sections and others have no blank lines.  But, that doesn't affect correctness, it is just an easthetic issue...)
> 
> 			...jim
> 
> On 10/26/16 11:51 PM, Avik Niyogi wrote:
>> Hi All,
>> 
>> Please review the proposed specification for JDK9 including inputs from reviewer reviews.
>> *http://cr.openjdk.java.net/~aniyogi/8138771/webrev.04/*
>> Thank you in advance.
>> 
>> With Regards,
>> Avik Niyogi
>> 
>>> On 27-Oct-2016, at 2:33 am, Jim Graham <james.graham at oracle.com <mailto:james.graham at oracle.com>> wrote:
>>> 
>>> The "@return" tags should not start with "returns" in the text.
>>> 
>>> Also, in the @return for getProperty(), insert a word "the" as "the property of the base image"...
>>> 
>>> ...jim
>>> 
>>> On 10/26/16 12:36 AM, Avik Niyogi wrote:
>>>> Hi All,
>>>> 
>>>> Please review the proposed specification for JDK9 including inputs from reviver reviews.
>>>> 
>>>> *cr.openjdk.java.net/~aniyogi/8138771/webrev.03/* <http://cr.openjdk.java.net/~aniyogi/8138771/webrev.03/*>
>>>> <http://cr.openjdk.java.net/~aniyogi/8138771/webrev.03/>
>>>> 
>>>> 
>>>> Thank you in advance.
>>>> 
>>>> With Regards,
>>>> Avik Niyogi
>> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20161031/e62542e5/attachment.html>


More information about the swing-dev mailing list