cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Peter Royal <pro...@managingpartners.com>
Subject Re: CocoonServlet.java patch
Date Tue, 08 Jan 2002 19:53:38 GMT
On Tuesday 08 January 2002 02:48 pm, Ihor Strutynskyj wrote:
> The other thing I would like to mention is:
> Why org.apache.log.Logger.log.warn() (or .info(), .error(), .debug()) call
> is always wrapped with if (log.isWarnEnabled()) {}? The
> org.apache.log.Logger has already isXxxxEnabled() wrappers in corresponding
> xxxx() functions. The code would look much simpler without those wrappers.

Speed, mainly for debug() statements. If the string you want to log is 
expensive to create, you can skip the creation by testing the loglevel first.
-pete

-- 
peter royal -> proyal@managingpartners.com

---------------------------------------------------------------------
To unsubscribe, e-mail: cocoon-dev-unsubscribe@xml.apache.org
For additional commands, email: cocoon-dev-help@xml.apache.org


Mime
View raw message