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