[8u-dev] Request for Approval to Backport: 8065172: More core reflection final and volatile annotations

[Ivan Gerasimov]

On 19.12.2014 22:38, Martin Buchholz wrote:
> Again, this change was originally written without planning for a
> backport (I might have done things a little differently), but looks
> good!

I though it is in line with
     8062771: Core reflection should use final fields whenever possible
     8064391: More thread safety problems in core reflection
     8064846: Lazy-init thread safety problems in core reflection

which had been backported earlier :-)

It wouldn't hurt to integrate this also, right?

Sincerely yours,
Ivan


> On Fri, Dec 19, 2014 at 11:04 AM, Ivan Gerasimov
> <ivan.gerasimov at oracle.com> wrote:
>> Hello!
>>
>> This is the last one left (at the time of posting) change, adding
>> final/volatile annotations in reflection.
>>
>> After the recent integration of JDK-8064846, the unshuffled patch applied
>> cleanly.
>>
>> Jprt + test subset jdk_lang with no failures.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8065172
>> Jdk9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/63c558ffb833
>> Jdk9 review:
>> http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-November/029739.html
>>
>> Sincerely yours,
>> Ivan
>