Re: ModuleConfigImpl holds reference to factory

[Robert Leland]

On Tue, 4 Feb 2003, David Graham wrote:

Date: Tue, 04 Feb 2003 20:05:27 -0700
From: David Graham <[EMAIL PROTECTED]>
Reply-To: Struts Developers List <[EMAIL PROTECTED]>
To: [EMAIL PROTECTED]
Subject: ModuleConfigImpl holds reference to factory

ModuleConfigImpl implements Serializable but the ModuleConfigFactory
interface does not.  ModuleConfigImpl has a member variable that stores its
factory but never actually uses that variable.  We have 2 options:

1.  Remove the factory variable from ModuleConfigImpl and the constructor
that sets it.  (my favorite)

+1, I plead guilty. I had noticed that the commons-pool or commons-collections had a pointer back to the factory.
So I used that pattern w/o a use case to support it !



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