FYI...From the Struts List...I know that NetUI uses this (anti-)pattern all
over it.  I'm pretty sure we are not deploying in shared classloaders, but
I'm not sure of all of the scenario's we expect to be deployed in.  I don't
believe this pattern is used in Controls at all.

---------- Forwarded message ----------
From: Simon Kitching <[EMAIL PROTECTED]>
Date: Mar 7, 2006 7:50 PM
Subject: commons-logging and static log members
To: [email protected]

Hi All,

Recently there was a discussion on the MyFaces list regarding the use of
the following pattern with commons logging classes:
  private static Log log = LogFactory.getLog(Foo.class)

It was pointed out that the use of static here is a really bad idea for
any code that may be deployed via a shared classloader in a container.
Craig McClanahan indicated that this (anti-)pattern was in use in struts
code and that the issue should be pointed out here.

I haven't personally checked whether this usage *is* present in the
current struts codebase. Neither do I know whether struts jarfiles are
expected to be deployed in shared classpaths (it's definitely true for
MyFaces).

More information on this issue can be found here:
http://wiki.apache.org/jakarta-commons/Logging/StaticLog

Regards,

Simon (commons-logging committer)


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to