RFR: 5043030 (reflect) unnecessary object creation in reflection

Joel Borggrén-Franck joel.franck at oracle.com
Tue Sep 9 09:21:47 UTC 2014


Hi Andrej,

Can you resend the latest patch attached to a mail to this list?

cheers
/Joel

On 2014-08-29, Andrej Golovnin wrote:
> 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