Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

bpo-31938: Fix select.epoll.register() signature#21067

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
vstinner wants to merge1 commit intopython:masterfromvstinner:pydoc_select
Closed

bpo-31938: Fix select.epoll.register() signature#21067

vstinner wants to merge1 commit intopython:masterfromvstinner:pydoc_select

Conversation

@vstinner
Copy link
Member

@vstinnervstinner commentedJun 23, 2020
edited by bedevere-bot
Loading

@vstinner
Copy link
MemberAuthor

inspect.signature() doesn't see the event_mask. Is it a bug in signature which doesn't support a signature written on two lines?

cc@serhiy-storchaka@1st1@ambv

11:47:01 vstinner@apu$ ./python Python 3.10.0a0 (heads/pydoc_select:486e11db95, Jun 23 2020, 11:45:39) >>> import select>>> select.epoll.register.__text_signature__'($self, /, fd,\n         eventmask=select.EPOLLIN | select.EPOLLPRI | select.EPOLLOUT)'>>> import inspect>>> inspect.signature(select.epoll.register)<Signature (self, /, fd)>

So pydoc doesn't show the whole signature:

./python -m pydoc select(...)     |  register(self, /, fd)     |      Registers a new fd or raises an OSError if the fd is already registered.     |           |        fd     |          the target file descriptor of the operation     |        eventmask     |          a bit set composed of the various EPOLL constants     |           |      The epoll interface supports all file descriptors that support poll.

@vstinner
Copy link
MemberAuthor

Oops,@asottile already proposed PR#21066 which is the same change. I close my PR.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vstinner@the-knights-who-say-ni@bedevere-bot

[8]ページ先頭

©2009-2025 Movatter.jp