Vadim Gritsenko wrote:
> Torsten Curdt wrote:
>> On 28.03.2007, at 20:14, Vadim Gritsenko wrote:
>>
>>> [EMAIL PROTECTED] 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.
>
Hmm, why do you think "is not and should not"? Why not (I'm talking
about 2.2+ here)?
In addition, with 2.2+ we provide separate artifacts for "common"
components which might be used outside of Cocoon and might then be used
via a shared class loader. So I think we should not use "static" here.
As most of our components are singletons anyway, there isn't any memory
problem here.
Carsten
--
Carsten Ziegeler
http://www.osoco.org/weblogs/rael/
