ModuleConfigImpl holds reference to factory

[David Graham]

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.

David





_________________________________________________________________
The new MSN 8: smart spam protection and 2 months FREE* http://join.msn.com/?page=features/junkmail


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