This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author giampaolo.rodola
Recipients christian.heimes, felipecruz, giampaolo.rodola, gvanrossum, meador.inge, neologix, pitrou, rosslagerwall, sbt, vstinner
Date 2013-08-30.12:30:49
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1377865849.93.0.0483429571021.issue16853@psf.upfronthosting.co.za>
In-reply-to
Content
Follows a couple of minor concerns.

- What about Solaris' /dev/poll?

- I'm not sure why in case of EINTR you retry with a different timeout; can't you just return []? 

- this is probably because I'm paranoid about performances but given that select() method will be called repeatedly I would not use a decorator. Also, similarly to what has been done elsewhere in the stdlib, for "critical" parts I would recommend localizing variable access in order to minimize overhead as in:

    def select(self, timeout=None):
        ... 
        key_from_fd = self._key_from_fd
        ready_append = ready.append
        for fd in r | w:
            ...
            key = key_from_fd(fd)
            if key:
                ready_append((key, events & key.events))
History
Date User Action Args
2013-08-30 12:30:50giampaolo.rodolasetrecipients: + giampaolo.rodola, gvanrossum, pitrou, vstinner, christian.heimes, meador.inge, neologix, rosslagerwall, sbt, felipecruz
2013-08-30 12:30:49giampaolo.rodolasetmessageid: <1377865849.93.0.0483429571021.issue16853@psf.upfronthosting.co.za>
2013-08-30 12:30:49giampaolo.rodolalinkissue16853 messages
2013-08-30 12:30:49giampaolo.rodolacreate