[issue1657] [patch] epoll and kqueue wrappers for the select module
Christian Heimes
report at bugs.python.org
Thu Dec 20 20:58:38 CET 2007
Christian Heimes added the comment:
> 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.
__________________________________
Tracker <report at bugs.python.org>
<http://bugs.python.org/issue1657>
__________________________________
More information about the Python-bugs-list
mailing list