Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allowing multiple handlers through a hack #72

Merged
merged 2 commits into from Mar 10, 2012
Merged

Conversation

ferd
Copy link
Collaborator

@ferd ferd commented Mar 9, 2012

This is done by using dynamic atoms (a bad thing) based on
the number of ports you have a connection on.

A good design would have required a substantial rewrite of
the application. However, to simply get things working in
a decent way, although not perfect in terms of patterns,
I decided to go this way. It is worth the time for an
application in maintenance -- no technical debt on this
one, unless someone else wants to take control.

ferd added 2 commits March 8, 2012 17:39
This is done by using dynamic atoms (a bad thing) based on
the number of ports you have a connection on.

A good design would have required a substantial rewrite of
the application. However, to simply get things working in
a decent way, although not perfect in terms of patterns,
I decided to go this way. It is worth the time for an
application in maintenance -- no technical debt on this
one, unless someone else wants to take control.
This now works using gproc instead of list_to_atom.
This is done using gproc and adds a dependency,
warranting a version bump from 1 to 1.1
yrashk added a commit that referenced this pull request Mar 10, 2012
Allowing multiple handlers through a hack
@yrashk yrashk merged commit 993b2ec into yrashk:master Mar 10, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants