Message58920
> Some remarks:
> * the name of the function used for PyArg_ParseTupleAndKeywords in
> register, modify, unregister is set to control instead of the good name.
Fixed
> * there is a leak in pyepoll_new if the parsing of arguments fails.
Fixed
> * the indentation is sometimes tabs, sometimes spaces. That should be
> good to unify this (to tabs I guess, since the select module used tabs
> before).
Fixed except for switch() and goto. I find the 4 space indention of the
case and the goto lables easier to read.
> * it seems there is an unrelated change in sunau.py
Fixed
> * I don't think the stdlib unittest module has skip support. You have
> to find another way to skip the tests if the modules aren't present.
Fixed ;)
> I've been able to port the epollreactor to your implementation and run
> the whole twisted tests with it, so I don't think there are outstanding
> problems. The code is fairly simple anyway.
>
> That's it for epoll and general remarks. I'll look at kqueue asap. Thanks!
Thansk for the code review and your remarks. I've fixed the problems
locally. I've also fixed a problem in unregister when fd is already
closed and I'm using PyFile_AsFileDescriptor(). It supports ints and
objects with a fileno() method.
I'm waiting for your test of kqueue before I upload the patch. |
|
Date |
User |
Action |
Args |
2007-12-20 19:58:38 | christian.heimes | set | spambayes_score: 6.67498e-05 -> 6.67498e-05 recipients:
+ christian.heimes, therve |
2007-12-20 19:58:38 | christian.heimes | set | spambayes_score: 6.67498e-05 -> 6.67498e-05 messageid: <1198180718.04.0.263727886879.issue1657@psf.upfronthosting.co.za> |
2007-12-20 19:58:38 | christian.heimes | link | issue1657 messages |
2007-12-20 19:58:37 | christian.heimes | create | |
|