Check more errors

Caught by errcheck. All remaining errcheck warnings have been audited and are
harmless (mostly because Broker.Open returns such a limited set of errors).
prod: minor simplification

Exactly logically equivalent, no functional change.
Merge pull request #181 from Shopify/more-errcheck

Check more errors
Merge pull request #183 from Shopify/simplify

prod: minor simplification
prod: drain successes on close just like errors

We kindly drain the errors channel on close so that users who stop reading from
it when calling close (e.g. in a defer) don't hang. We should do the same thing
for the successes channel when AckSuccesses is defined.

May or may not fix the bug reported tangentially at the bottom of
https://github.com/Shopify/sarama/issues/180#issuecomment-63127314
where shutting down with AckSuccesses could be very slow.
Launch those two goroutines withRecover