Hi Daniel, > > Hi Felix, > > On 11/01/2020 07:37, Yangfei (Felix) wrote: > > Hi Daniel, > > > > Thanks for the suggestions. > > I modified the patch making the third port also configurable via the > management.properties file. > > New webrev: http://cr.openjdk.java.net/~fyang/8234484/webrev.01/ > > I think I would prefer the try catch NumberFormatException to be just around > the call to Integer.parseInt(), as it is done elsewhere in this file (e.g. > line 337).
Modified accordingly in new webrev: http://cr.openjdk.java.net/~fyang/8234484/webrev.02/ Also renamed property to: com.sun.management.jmxremote.local.port Thanks, Felix
