Request for Review JDK-8006948: Update javac for MethodParameters format change
Remi Forax
forax at univ-mlv.fr
Sun Feb 17 15:29:58 PST 2013
I'm not an official reviewer, but I'm currently implementing
MethodParameters in ASM5,
and this patch woks :)
http://cr.openjdk.java.net/~emc/8006948/webrev.01/
Rémi
On 02/14/2013 07:55 PM, Eric McCorkle wrote:
> The changeset for 8006949 has been integrated into TL, so I want to move
> towards pushing this one as well. These changes allow hotspot to
> recognize class files with both the old and the new format of flags.
>
> I have updated the webrev to reflect some recent changes (pushed for
> 8006345 and others).
>
> Note: there is a comprehensive test suite for the parameter reflection
> feature that is about to be pushed. I do not plan on submitting this
> changeset until the tests have been pushed.
>
> Please review and comment on the latest webrev.
>
> Thanks,
> Eric
>
> On 01/25/13 12:49, Eric McCorkle wrote:
>> Hello,
>>
>> Please review this change, which updates javac and the classfile library
>> to reflect changes in the class file format of MethodParameters attributes.
>>
>> The webrev is here:
>> http://cr.openjdk.java.net/~emc/8006948/webrev.00/
>>
>> The latest version of the spec can be found here:
>> http://cr.openjdk.java.net/~abuckley/8.misc
>>
>> The enhancement request is here:
>> http://bugs.sun.com/view_bug.do?bug_id=8006948
>>
>>
>> Note that this will break compatibility with existing versions.
>> Furthermore, the companion changeset for hotspot will go into hsx, and
>> will take some time to propagate to TL. However, there are no tests
>> enabled at this point that rely on the old binary format, so there
>> should be no JPRT failures arising from this fix.
>>
>> Thanks,
>> Eric
More information about the compiler-dev
mailing list