select.epoll.register raises an IOError for already registered fds, however the docstring says the fd gets modified:


Python 2.7.1+ (r271:86832, Feb 24 2011, 15:00:15) 
[GCC 4.5.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import select
>>> print select.epoll.register.__doc__
register(fd[, eventmask]) -> None

Registers a new fd or modifies an already registered fd.
fd is the target file descriptor of the operation.
events is a bit set composed of the various EPOLL constants; the default
is EPOLL_IN | EPOLL_OUT | EPOLL_PRI.

The epoll interface supports all file descriptors that support poll.
>>>

The patch fixes the docstring

The patch is fine: but would you be interested in trying to write a unittest for select.epoll.register ? it would be really nice to start having 'select' tested (oh no, don't ask me how to test select since I have no idea :)).

New changeset 0610f70e6694 by Senthil Kumaran in branch '3.2':
Fix closes issue 11568 - update select.epoll.register docstring with mention of correct behavior.
http://hg.python.org/cpython/rev/0610f70e6694

New changeset a6586cb660dc by Senthil Kumaran in branch '2.7':
Fix closes issue 11568 - update select.epoll.register docstring with mention of correct behavior.
http://hg.python.org/cpython/rev/a6586cb660dc

On Sun, Jun 26, 2011 at 03:12:30PM +0000, Sandro Tosi wrote:
> The patch is fine: but would you be interested in trying to write a
> unittest for select.epoll.register ? it would be really nice to

This is covered in test_epoll.py

Ah, I find the test file name a bit unhappy (why not test select.epoll in test_select? or add select in the filename?) but since it's covered - I'm fine! :) Next time I'll grep instead of simple file glob - thanks for your help, Senthil.