Issue #48 fix #50

Merged
merged 1 commit into from Feb 13, 2013

Conversation

Projects
None yet
2 participants
Contributor

okeuday commented Feb 9, 2013

Adds option {active_pid, pid()}

@okeuday okeuday Add {active_pid, pid()} option to socket/2, which implies {active, tr…
…ue}. The change provides a controlling_process concept for active mode sockets (so that the code doesn't assume self() is always the controlling process).
a45c2ad
Member

yrashk commented Feb 12, 2013

So, is this change good only for 3.x?

Contributor

okeuday commented Feb 12, 2013

Yeah, i did the separate pull request for the master branch. This one is only for 3.x.

Member

yrashk commented Feb 12, 2013

was it #49 ?

Contributor

okeuday commented Feb 12, 2013

nope both pull requests are for #48, just separate branches/versions

Member

yrashk commented Feb 12, 2013

I understand that they both are the same, I was referring to a particular
pullreq that I think I closed and reopened thereafter

On Tue, Feb 12, 2013 at 11:46 AM, Michael Truog notifications@github.comwrote:

nope both pull requests are for #48#48,
just separate branches/versions


Reply to this email directly or view it on GitHubhttps://github.com/zeromq/erlzmq2/pull/50#issuecomment-13452289.

@yrashk yrashk added a commit that referenced this pull request Feb 13, 2013

@yrashk yrashk Merge pull request #50 from okeuday/3.x
Issue #48 fix
63ce4e8

@yrashk yrashk merged commit 63ce4e8 into zeromq:3.x Feb 13, 2013

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