8181087: Module system implementation refresh (6/2017 update)

Peter Levart peter.levart at gmail.com
Thu Jun 15 07:40:18 UTC 2017

Hi Alan,

On 06/15/2017 09:34 AM, Alan Bateman wrote:
>> 2453 reflectionFactory.getExecutableSharedParameterTypes(method),
>> reflectionFactory should be accessed via getReflectionFactory().  I 
>> see Peter
>> already comments on this.
> Thanks, it should be getReflectionFactory(). 

...and I would also introduce a local variable in getReflectionFactory() 
as was done recently in String.hashCode(). It might not be a problem 
now, but could become one in the future...

Regards, Peter

More information about the hotspot-runtime-dev mailing list