jmx-dev RFR 7199353: Allow ConstructorProperties annotation from any package
Jaroslav Bachorik
jaroslav.bachorik at oracle.com
Fri Oct 16 12:04:11 UTC 2015
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.
>
More information about the jigsaw-dev
mailing list