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.