Couple asserts upgraded to handled errors

zeromq / erlzmq2

Erlang binding for 0MQ (v2)

http://zeromq.org/

MIT License

165 stars 63 forks source link

Couple asserts upgraded to handled errors #16

Closed gar1t closed 13 years ago

gar1t commented 13 years ago

IMO those two asserts want to be handled. There are probably more, but these two jumped out at me.

yrashk commented 13 years ago

Next time, please consider squashing your commits into one, it will make repo history nicer.

gar1t commented 13 years ago

I generally do - I believe that second commit was post pull request. I can -f in the future.

On Sun, Jun 12, 2011 at 7:37 AM, yrashk reply@reply.github.com wrote:

Next time, please consider squashing your commits into one, it will make repo history nicer.

Reply to this email directly or view it on GitHub: https://github.com/zeromq/erlzmq2/pull/16#issuecomment-1354749