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

Support for the Poller class #2

Merged
merged 2 commits into from
Oct 7, 2011
Merged

Support for the Poller class #2

merged 2 commits into from
Oct 7, 2011

Conversation

flowblok
Copy link
Contributor

@flowblok flowblok commented Oct 6, 2011

What I’ve done:

  • added the zmq_poll function
  • a port of pyzmq’s _poll function
  • a port of pyzmq’s Poller class
  • changed the nonzero error check to check specifically for -1
    (since the nonzero assumption doesn’t hold for zmq_poll)

What I haven’t done:

  • tested that the nonzero change doesn’t break anything else
  • rigorously tested the Poller class to see it works properly
    (I only need it for a small script, hence I only know .register() and .poll() work fine in my scenario…)

svpcom added a commit that referenced this pull request Oct 7, 2011
Support for the Poller class
@svpcom svpcom merged commit 820c938 into svpcom:master Oct 7, 2011
@svpcom
Copy link
Owner

svpcom commented Oct 7, 2011

thanks

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.

2 participants