RFR: 5043030 (reflect) unnecessary object creation in reflection
Andrej Golovnin
andrej.golovnin at gmail.com
Fri Aug 29 15:30:07 UTC 2014
Hi Joel,
>> I have changed the test TestMethodReflectValueOf as you suggested and I have changed the summary of both tests too. You will find the changes in the attached patch. Here is the new webrev: https://db.tt/wQBLVceA
>>
>> And here is the coverage report in the HTML format:
>>
>> https://db.tt/JTZjpnMM
>>
>
> Out of curiosity, did you generate the coverage report running the jdk_lang test suite?
Yes and I used the following config for JCov:
include_list=jcov_jdk_lang.txt,field=on,abstract=on,native=on,type=all,file=$JCOV_OUT,merge=merge
where jcov_jdk_lang.txt contains the single line:
sun.reflect.*
> I think this patch is good to go. I need to file some Oracle internal requests, should take about a week, then I can sponsor this.
I am very pleased to hear that and I hope to contribute more.
Best regards,
Andrej Golovnin
More information about the core-libs-dev
mailing list