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 | christian.heimes |
|---|---|
| Recipients | christian.heimes, therve |
| Date | 2007年12月22日.11:09:03 |
| SpamBayes Score | 0.03969196 |
| Marked as misclassified | No |
| Message-id | <1198321745.01.0.202070533098.issue1657@psf.upfronthosting.co.za> |
| In-reply-to |
| Content | |
|---|---|
> I attached a patch with a more complete test of kqueue. It's not that > great, but it's a thing. I've only tested on OS X, but it works. A small unit test is better than no unit test :) > Regarding the ability of building an epoll object from a fd, it might be > usefull in some corner cases, but that's not a priority. > It should be trivial to add an epoll.fromfd() classmethod. > exarkun looked at the patch and told me that there may be some > threadsafety issues: for example, when calling epoll_wait, you use > self->evs unprotected. It's not very important, but you may want to tell > it in the documentation. I found an interesting article about epoll. It states that epoll_wait() is thread safe. Ready lists of two parallel threds never contain the same fd. http://lwn.net/Articles/224240/ It's easier to remove the buffer and allocate memory inside the wait() method than to add semaphores. It makes the code. > As you started the rich comparison for kevent objects, it may be > interesting to have full comparison (to sort list of events). It's not a > high priority though. What do you suggest as sort criteria? |
|
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2007年12月22日 11:09:05 | christian.heimes | set | spambayes_score: 0.039692 -> 0.03969196 recipients: + christian.heimes, therve |
| 2007年12月22日 11:09:05 | christian.heimes | set | spambayes_score: 0.039692 -> 0.039692 messageid: <1198321745.01.0.202070533098.issue1657@psf.upfronthosting.co.za> |
| 2007年12月22日 11:09:05 | christian.heimes | link | issue1657 messages |
| 2007年12月22日 11:09:03 | christian.heimes | create | |