Affirmative on the "svn mv" for the C module.

The duplicated code, thanks for pointing that out.  Someone else mentioned it, but I didn't understand what they were saying and they didn't reply to my request for clarification.  Fixed.

On the modules() list, how about if I just make it a list and build it at import time?  The class was the way I thought most straightforward to do it as a function, so maybe this is more reasonable?

Per the documentation, I pulled down the description from above, which I think captured the uses of *salt* and removed the duplication.