And here is the link from the last time you brought it up. http://marc.info/?t=137788807500006&r=1&w=2
My position hasn’t changed, leave them as they are. Ralph On Jan 27, 2014, at 5:47 PM, Gary Gregory <[email protected]> wrote: > Hi All, > > I think I brought this up a long time ago but I thought I'd give it a refresh > since we have a lot more voices participating in conversations. Yes, I am > aware of making source compatibility simple for porting from version 1 but > this is a naming issue that's always displeased me. > > I find the Logger#isLevelEnabled() method names unnecessarily verbose: > isDebugEnabled(), isTraceEnabled() and so on. > > Any likes to rename those simply to isDebug(), isTrace() and so on? > > I find this much more palatable: > > if (logger.isDebug()) { > logger.debug(...); > } > > vs. > > if (logger.isDebugEnabled()) { > logger.debug(...); > } > > ? > > Gary > > -- > E-Mail: [email protected] | [email protected] > Java Persistence with Hibernate, Second Edition > JUnit in Action, Second Edition > Spring Batch in Action > Blog: http://garygregory.wordpress.com > Home: http://garygregory.com/ > Tweet! http://twitter.com/GaryGregory
