On 15.10.2015 21:32, Alan Bateman wrote:
On 15/10/2015 16:55, Jaroslav Bachorik wrote:
Any objections to changing the annotation name to @ConstructorMapping
to make it better distinguishable from
@java.beans.ConstructorProperties ?
Not from me. Do you mind updating the webrev so that we can see the
updated javadoc?
I have decided for @ConstructorParameters - it is rather close to the
original @ConstructorProperties and corresponds to the annotation purpose.
I tried to address all the comments gathered in this review.
The updated webrev is
http://cr.openjdk.java.net/~jbachorik/8139727/webrev.00
The issue number had to be changed due to unlucky escape of the fix in
progress into the public repository (which was immediately backed out
but the original issue was 'spent').
Though the issue title still contains "ConstructorProperties" - I will
change it to "ConstructorParameters" as soon as we agree on this name.
Thanks,
-JB-
-Alan.
