On 27/10/2017 12:08, [email protected] 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
