A bit of code cleanup, more comments here and there.

Major change, collabclient.c is now always built, but the body
of the functions are #ifdef guarded. This way code through the rest
of fontforge can call the collabclient_foo() functions without needing
any #ifdef guards. They simply do nothing when the zeromq + collab code
is not built.