Changeset: c3ef78cd9081
Author:emc
Date: 2013-09-11 09:24 -0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c3ef78cd9081
6962494: Update documentation on Executable.getParameterAnnotations()
Summary: Update javadoc comments on getParameterAnnotations to correctly
describe its b
Thanks for the review.
On 09/10/13 17:31, Joe Darcy wrote:
> Hi Eric,
>
> Looks good, modulo some typos:
>
> 434 * this object. Synthetic and mandated parameters (see
> 435 * explanation below), such as the inner this parameter to an
>
> Inner class constructors have an *outer* thi
Hi Eric,
Looks good, modulo some typos:
434 * this object. Synthetic and mandated parameters (see
435 * explanation below), such as the inner this parameter to an
Inner class constructors have an *outer* this parameter.
437 * array. If the executable has no parameters (incl
A new webrev has been posted, with some improvements to the comment.
On 09/09/13 17:41, Eric McCorkle wrote:
> Hello,
>
> Please review this patch which updates the javadoc comments for
> java.lang.reflect.Executable.getParameterAnnotations(). The patch
> corrects the javadocs to describe the ac
Hello,
Please review this patch which updates the javadoc comments for
java.lang.reflect.Executable.getParameterAnnotations(). The patch
corrects the javadocs to describe the actual behavior of this method.
It also refers users to the new java.lang.reflect.Parameter API.
See comments on the bug
