Thanks Mandy for the review. I incorporated your comments and updated changes are available in this webrev. http://cr.openjdk.java.net/~asapre/webrev/8151099/webrev.02/
PS: Also updated copyright year for all modified files. Amit > -----Original Message----- > From: Mandy Chung > Sent: Tuesday, October 25, 2016 3:24 AM > To: Amit Sapre > Cc: serviceability-dev; Daniel Fuchs; David Holmes > Subject: Re: RFR : JDK-8151099 : > java.lang.management.ManagementFactory.getPlatformMXBeans() should work > even if jdk.management is not present. > > > > On Oct 24, 2016, at 5:49 AM, Amit Sapre <amit.sa...@oracle.com> > wrote: > > > > Hello, > > > > I have incorporate review comments and updated changes are available > in this webrev: > > Updated Webrev : > http://cr.openjdk.java.net/~asapre/webrev/8151099/webrev.01/ > > > > Looks okay to me. > > sun/management/VMManagementImpl.java > 106 try { > 107 > Class.forName("com.sun.management.GarbageCollectorMXBean"); > 108 } catch (Exception x) { > > ==> catch specific exception would be better e.g. > ClassNotFoundException > > 109 isSupported = false; > > Nit: 4-space indent > > jdk/test/com/sun/management/GarbageCollectorMXBean/GarbageCollectionNot > ificationContentTest.java > 75 final Boolean isNotificationSupported = > test/com/sun/management/GarbageCollectorMXBean/GarbageCollectionNotific > ationTest.java > 74 final Boolean isNotificationSupported = > > It can use “boolean” primitive type. > > DefaultManagementProviderTest.java > Can you break the long @summary line > > Mandy