priestjim / gen_rpc

A scalable RPC library for Erlang-VM based languages
Apache License 2.0
221 stars 81 forks source link

(Question) about commit 822f8. #21

Closed linearregression closed 9 years ago

linearregression commented 9 years ago

For commit https://github.com/priestjim/gen_rpc/commit/822f844896934a4b898b98d1dd0b1fed136ed50d did you mean to remove just the catch or the entire socket close?

priestjim commented 9 years ago

gen_tcp closes the socket when the owning process dies. There's no reason to close the socket explicitly!

linearregression commented 9 years ago

cool. How about putting your presentation slides here or iin github or a link in Readme?