[8u40] Request for approval for 8065132 and 8029012
Seán Coffey
sean.coffey at oracle.com
Mon Nov 24 22:21:05 UTC 2014
Eric,
Since it's not a clean patch application, can you obtain a code review
first before pushing ?
Approved but subject to above request.
regards,
Sean.
On 24/11/2014 21:25, Eric McCorkle wrote:
> Hello,
>
> Please approve the backport for these two fixes, which cause javac to
> generate Runtime[In]Visible[Type,Parameter]Annotations attributes that
> correctly account for synthetic parameters.
>
> Note that the patches did not apply cleanly. In the case of 8029012,
> langtools SQE staff have added cases to the
> referenceinfos/Constructor.java test that are not present in 8u. For
> 8065132, diamondification made a behavior-invariant change to the source
> in ClassWriter.java
>
> After these changes, all tests pass.
>
> The JBS issues are here:
> https://bugs.openjdk.java.net/browse/JDK-8065132
> https://bugs.openjdk.java.net/browse/JDK-8029012
>
> A webrev containing both changesets is here:
> http://cr.openjdk.java.net/~emc/8065132/
More information about the jdk8u-dev
mailing list