[PATCH] Unnecessary Amount Of Internal Class Conversion

Alan Bateman Alan.Bateman at oracle.com
Mon Oct 30 07:52:57 UTC 2017



On 27/10/2017 12:08, serguei.spitsyn at oracle.com wrote:
> Let's check if Alan has a time to look at it.
> The patch is not that big. Hi Ben,
There isn't any changes to the hotspot code so one reviewer should be fine.

In any case, I looked through it and it seems okay. My only comment is 
that setHasTransformers would be clearer if it were named something like 
enableClassFileLoadHook as it just enables or disables the CFLH. I 
realize this would mean find a better name for 
setHasRetransformableTransformers too.

-Alan


More information about the serviceability-dev mailing list