<Beans Dev> [9] Review Request: 8132973 @BeanProperty: what is the correct output in case of repeating annotations?
Alexander Stepanov
alexander.v.stepanov at oracle.com
Mon May 23 13:07:35 UTC 2016
Hello Sergey,
Just concerning: does the fix also cover the situation when, e.g., we
have two annotated getters (regular + indexed) or setters (should their
priority be specified somehow in such a case)? A separate issue exists
for this case (JDK-8132732), so just wondering if it shouldn't be
trailed to the fix (as it seems at a 1st glance that the matter is very
similar).
Thanks,
Alexander
P.S. at least the test fixes look good for me (not a reviewer)
On 5/23/2016 12:31 PM, Sergey Bylokhov wrote:
> Hello.
> Please review the fix for jdk9.
>
> The bug describe the situation when for the same property both
> read/write methods are annotated. Our code assumes that the annotation
> exists only for one of these methods, and we produce the unspecified
> mix of both annotations(some of the data from reader and other from
> writer).
>
> As a fix I changed the order of initialization, first we will try the
> reader and if necessary the writer (before it was expected that the
> data from the read will overwrite the data from write, but it fails
> because of optimization). The specification is clarified and ccc will
> be filed after the technical review. The tests are updated also.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8132973
> Webrev can be found at:
> http://cr.openjdk.java.net/~serb/8132973/webrev.00
>
More information about the beans-dev
mailing list