Conversation

According to RedisClient.prototype.on_data, "Callbacks that throw exceptions will land in return_reply(), below.". Although this handling is now in try_callback, exceptions thrown by "reply" callbacks are being silenced by the try/catch in lib/parser/javascript.js. Allow those errors to be thrown to the right place.

I think that many try-catches have a bigger perf impact than just one. Hopefully #361 will take care of this. Closing for now, as I'd like 361's approach to make it in (and of course I will run perf tests on it to make sure it's not slower).