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) > > 2. Make the factory variable transient and add a setFactory() method > because there is no way to reset the variable after deserialization. > > Comments? I see no problem with number 1 so I'll make the change in a > couple days if no one objects. > The grand prize is hiding behind Door #1. :-) > David > Craig --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]