Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork33.3k
bpo-31938: Fix default-value signatures of several functions in the select module#21066
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
Conversation
taleinat commentedJun 23, 2020 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
If this is the proper way to do it, this PR should also fix the default values for |
Oops, I didn't notice that you proposed a PR and so I wrote PR#21067 which is the same fix :-) Copy of my comment there. inspect.signature() doesn't see the event_mask. Is it a bug in signature which doesn't support a signature written on two lines? So pydoc doesn't show the whole signature: |
IMO the fix is correct, so I concur thta the two other functions signature should also be fixed. By the way, I suggest to rephase the NEWS entry to explain that the change fix the "signature" rather than fixing "pydoc". |
I fixed a few others as well -- good catch |
Sorry,@asottile and@vstinner, I could not cleanly backport this to |
Sorry@asottile and@vstinner, I had trouble checking out the |
@asottile: Automated backport failed. Can you try to backport it manually (use cherry_picker or git cherry-pick -x)? |
…elect module (pythonGH-21066)(cherry picked from commitd051801)
bedevere-bot commentedJun 23, 2020
GH-21097 is a backport of this pull request to the3.9 branch. |
bedevere-bot commentedJun 23, 2020
GH-21098 is a backport of this pull request to the3.8 branch. |
…elect module (pythonGH-21066)(cherry picked from commitd051801)
vstinner commentedJun 23, 2020 • edited by bedevere-bot
Loading Uh oh!
There was an error while loading.Please reload this page.
edited by bedevere-bot
Uh oh!
There was an error while loading.Please reload this page.
pydoc doesn't show the eventmask parameter whereas it is in select.epoll.register.text_signature. I createdbpo-41095: "inspect.signature() doesn't parsetext_signature containing a newline character". |
Uh oh!
There was an error while loading.Please reload this page.
this appears to have regressed in6dc57e2
https://bugs.python.org/issue31938