It is still worth it - I was being lazy here (fixed now) :-)
I would recommend caching the log level somewhere rather than calling
isDebugEnabled() for every request - I've seen some profiles where the time
in isTraceEnabled() was significant.
Should this be a big problem if the isDebugEnabled() is marked as final? If it is, then it should be inlineable by a decent JIT.
Alex.
