Review all these *Client receiver methods. They should probably either

all be receivers, or none.

Maybe put auth-related stuff into its own structure inside Client,

instead of at Client's top level.

Add a way to broadcast status information as negotiation happens or

disconnects occur. Possibly a new type of object that can be sent on

Recv along with stanzas. Or use sync.Cond to protect a state

variable.

Asynchronously updating Client.Features is not thread safe.

NewClient shouldn't be asynchronous.

Add a Reconnect() function.

Put roster and bind into separate packages, if possible.

Eliminate as many uses of Generic as possible.