Hi Leonid,

this looks good to me! Please remove the extra whitespace at the end of 
compileBroker.hpp:420 before
the ")". No new webrev required.

Thanks,
Tobias

On 07.06.19 02:36, Leonid Mesnik wrote:
> Hi
> 
> Could you please review following fix which verify parameter for 
> Compiler.CodeHeap_Analytics
> command. So jcmd just exits instead of crashing target VM. Regression test 
> was added, hs-tier1/2
> passed.
> 
> webrev: http://cr.openjdk.java.net/~lmesnik/8225388/webrev.00/
> 
> bug: https://bugs.openjdk.java.net/browse/JDK-8225388
> 
> Leonid
> 
> 

Reply via email to