dont throw errors

[kguirguis]

No description provided.

[reedog117]

I haven't had time to maintain this module -- why no longer throw errors? Just trying to understand rationale before merging everything in.

[kguirguis]

The reason I was getting rid of throwing errors, was since it's asynchronous, once the error is thrown, we would not be able to catch it, since the block of code would've already 'executed'; so by removing the throwing of errors, we could just listen to the event 'error' and should be able to handle that.

For example:

try {
var c = new Client(CREDS_TO_THROW_ERROR);
} catch (e) { // would not catch the error since the block would've completed before the error is thrown; and the error would exit the node process.
console.log(e);
}

Thanks a lot for the response!