cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Vadim Gritsenko <va...@reverycodes.com>
Subject Re: svn commit: r522901 - in /cocoon/trunk/core/cocoon-pipeline/cocoon-pipeline-impl/src/main: java/org/apache/cocoon/caching/impl/CacheImpl.java resources/META-INF/cocoon/spring/cocoon-pipeline-impl-cache.xml
Date Wed, 28 Mar 2007 23:37:42 GMT
Torsten Curdt wrote:
> 
> On 28.03.2007, at 20:14, Vadim Gritsenko wrote:
> 
>> reinhard@apache.org wrote:
>>> -public class CacheImpl
>>> -extends AbstractLogEnabled
>>> -implements Cache, ThreadSafe, Serviceable, Disposable, 
>>> Parameterizable {
>>> +public class CacheImpl implements Cache {
>>>  +    private Log logger = LogFactory.getLog(getClass());
>>
>> Is there a reason why logger can not be static?
> 
> http://wiki.apache.org/jakarta-commons/Logging/StaticLog

So since Cocoon is not and should not be deployed in container's shared class 
loader, usage of 'static' is preferred.

Vadim

Mime
View raw message