<Beans Dev> [9] Review Request: 8132973 @BeanProperty: what is the correct output in case of repeating annotations?

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Tue May 24 12:14:45 UTC 2016


The fix looks good to me.

On 24/05/16 15:38, Sergey Bylokhov wrote:
> On 23.05.16 21:01, Alexandr Scherbatiy wrote:
>> "If the read method and the write method of the property annotated" -
>> may be be it can be slightly rephrased like: "If both the read and the
>> write methods of the property are annotated"
>
> Updated:
> http://cr.openjdk.java.net/~serb/8132973/webrev.01
>
>>
>>  There is the place where "update after 8132973 fix" code is
>> uncommented. But the bug 8132973 is still open. Does it mean that the
>> uncommented code does not relate to the issue 8132973?
>
> This case related to JDK-8155013.
    It does not make something clearly because JDK-8155013 (and 
JDK-8132973 comment from BeanPropertyTest) are still open.

   Thanks,
   Alexandr.
>
>> 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