Re: RFR 623 build warnings [deprecation] isLoggable(int) in PlatformLogger

2013-06-17 Thread Chris Hegarty
On 06/17/2013 05:15 PM, Mandy Chung wrote: Chris, Thanks for the patch. It looks fine to me. I saw that you have pushed this changeset which is fine. Thanks for looking at this Mandy, your eyes are welcome here. As for the background, we leave these deprecated methods and fields in

RFR 623 build warnings [deprecation] isLoggable(int) in PlatformLogger

2013-06-14 Thread Chris Hegarty
There are 623 occurrences, and hence 623 deprecated build warnings, of PlatformLogger.isLoggable(int) in the jdk source. PlatformLogger is an internal API, and used in may places in the jdk. isLoggable(int) has been deprecated in favor of isLoggable(Level). isLoggable(Level) is slightly more

Re: RFR 623 build warnings [deprecation] isLoggable(int) in PlatformLogger

2013-06-14 Thread Joe Darcy
Looks fine Chris; thanks, -Joe On 6/14/2013 8:00 AM, Chris Hegarty wrote: There are 623 occurrences, and hence 623 deprecated build warnings, of PlatformLogger.isLoggable(int) in the jdk source. PlatformLogger is an internal API, and used in may places in the jdk. isLoggable(int) has been