On 1/30/2015 9:38 AM, shanliang wrote:
Thanks for all your comments, here is the new version: http://cr.openjdk.java.net/~sjiang/JDK-8065213/01/
Thank you for the update. Looks fine to me. Minor comment: PlatformMBeanProviderImpl.java line 43: better to rename it as "list" or other name. same as in DefaultPlatformMBeanProvider.java line 54 No need to generate new webrev. Mandy