<Swing Dev> 8138771: java.awt.image.AbstractMultiResolutionImage needs customized spec for methods of Image which it implements
Alexandr Scherbatiy
alexandr.scherbatiy at oracle.com
Thu Nov 3 10:48:43 UTC 2016
The fix looks good to me.
Thanks,
Alexandr.
On 10/31/2016 9:31 PM, Jim Graham wrote:
> Looks good. +1
>
> ...jim
>
> On 10/30/16 11:53 PM, Avik Niyogi wrote:
>> 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
>>> <mailto: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>
>>>>> <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/*>
>>>>>> <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