Review makefile changes for JDK-8004729: Method parameter reflection API

Tim Bell tim.bell at oracle.com
Wed Jan 23 01:14:33 UTC 2013


On 01/22/13 16:02, David Holmes wrote:
> Hi Eric,
>
> On 23/01/2013 4:47 AM, Eric McCorkle wrote:
>> Hello,
>>
>> Please review and approve the makefile changes for JDK-8004729.
>>
>> This patch add a native method (getParameters0) to the Executable class,
>> and uses the JVM_GetMethodParameters function exported from the JVM.
>>
>> http://cr.openjdk.java.net/~emc/8004729/webrev.00/
>>
>> For more discussion of the feature, see the review on core-libs-dev.
>
> The build changes look okay to me and there is no impact on the new 
> build AFAICS.
>
> Tim is also taking a look.

The build changes look fine to me.

This is not Makefile related, but I wonder whether you need lines 
33...40 of src/share/native/java/lang/reflect/Executable.c.  Is that 
#define and comment serving a purpose?  I did not find anything similar 
in nearby source files.

Tim




More information about the build-dev mailing list