Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Make sure there is no process leak #7

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Member

aggelgian commented Jan 21, 2013

gen_server didn't terminate and exit(Client, normal) should have no effect on the client (however, all processes die before the exit/2 call so there is no real problem)

@yiannist yiannist was assigned Jan 21, 2013

Member

yiannist commented Jan 21, 2013

Thanks @aggelgian!

I tested it and it seems reasonable! :-)

Just FYI, we tested this by running:

  • Before the patch:
1> c(genstress).
{ok,genstress}
2> genstress:run([gen_call, 10, 10, 10], ok, ok).
ok
3> whereis(genstress).
<0.41.0>
  • After the patch:
1> c(genstress).
{ok,genstress}
2> genstress:run([gen_call, 10, 10, 10], ok, ok).
ok
3> whereis(genstress).
undefined

@yiannist yiannist closed this Jan 21, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment