SC should not have virtual functions

It is not designed for inheritance
SC: remove useless empty destructor
Broker now stops modules before quitting

A QuitFunction can now optionally be registered for each module in the
call to RegisterModule
Add missing traces to SC

Also, fix some that were printing at Debug level
Fix a major indentation snafu in StateCollection