<Swing Dev> 8138771: java.awt.image.AbstractMultiResolutionImage needs customized spec for methods of Image which it implements
Jim Graham
james.graham at oracle.com
Thu Oct 27 19:48:20 UTC 2016
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
>
More information about the swing-dev
mailing list