Issue 16666: docs wrongly imply socket.getaddrinfo takes keyword arguments in 2.x (original) (raw)

Created on 2012-12-11 23:00 by Mikel.Ward, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Messages (9)
msg177356 - (view) Author: Mikel Ward (Mikel.Ward) Date: 2012-12-11 23:00
The docs for 2.6 and 2.7 say socket.getaddrinfo takes keyword arguments. http://docs.python.org/release/2.6.6/library/socket.html?highlight=socket#socket.getaddrinfo > socket.getaddrinfo(host, port, family=0, socktype=0, proto=0, flags=0) But that doesn't actually work in Python 2.7. $ ./python Python 2.7.3+ (2.7:ec4ea40be2f6, Dec 11 2012, 14:43:35) [GCC 4.6.3] on linux2 Type "help", "copyright", "credits" or "license" for more information. >>> import socket >>> s = socket.getaddrinfo('localhost', 999, family=socket.AF_INET) Traceback (most recent call last): File "", line 1, in TypeError: getaddrinfo() takes no keyword arguments Looks like it was introduced in 61999:c963478b9092.
msg177362 - (view) Author: Chris Jerdonek (chris.jerdonek) * (Python committer) Date: 2012-12-12 03:25
This might be a good place to use the newly-documented "positional-only" nomenclature: http://docs.python.org/2.7/glossary.html#term-parameter
msg177423 - (view) Author: Éric Araujo (eric.araujo) * (Python committer) Date: 2012-12-13 17:13
There are two or three issues on this tracker where we discuss the parameters notation conventions. I think I remember a proposal to add text explaining that the keyword notation does not imply that the function accepts keyword argument, only that there is a default value. Chris, does that ring a bell?
msg177427 - (view) Author: Chris Jerdonek (chris.jerdonek) * (Python committer) Date: 2012-12-13 17:56
Is issue 13386 one of the issues you had in mind, Éric? I don't know the current best practices for all of the signature edge cases, but Ezio might. Personally, I think explicitly stating in the text that the parameters are positional-only can't hurt, especially since that's the rarer case, no?
msg177430 - (view) Author: Ezio Melotti (ezio.melotti) * (Python committer) Date: 2012-12-13 18:24
> Personally, I think explicitly stating in the text that the parameters > are positional-only can't hurt, especially since that's the rarer case, > no? OTOH it's an implementation detail that might be changed at some point. The other option is to use [] in the signature and document the default values in the text, and possibly fix it for 3.4 (or does it work there already?).
msg177433 - (view) Author: Chris Jerdonek (chris.jerdonek) * (Python committer) Date: 2012-12-13 19:26
This issue only applies to 2.7. Keyword arguments are accepted in 3.x. See the interactive example here, for example: http://docs.python.org/3.2/library/socket.html#socket.getaddrinfo
msg177435 - (view) Author: Ezio Melotti (ezio.melotti) * (Python committer) Date: 2012-12-13 20:13
In that case I think it's OK to use the [] and mention the default values in the text.
msg178121 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2012-12-25 12:46
New changeset 1e5e7064e872 by Ezio Melotti in branch '2.7': #16666: document default values for socket.getaddrinfo in the text to clarify that it doesn't accept keyword args. http://hg.python.org/cpython/rev/1e5e7064e872
msg178122 - (view) Author: Ezio Melotti (ezio.melotti) * (Python committer) Date: 2012-12-25 12:47
Fixed, thanks for the report!
History
Date User Action Args
2022-04-11 14:57:39 admin set github: 60870
2012-12-25 12:47:59 ezio.melotti set status: open -> closedtype: enhancementmessages: + assignee: docs@python -> ezio.melottiresolution: fixedstage: resolved
2012-12-25 12:46:29 python-dev set nosy: + python-devmessages: +
2012-12-13 20:13:38 ezio.melotti set messages: +
2012-12-13 19:26:42 chris.jerdonek set messages: + versions: - Python 2.6
2012-12-13 18:24:53 ezio.melotti set messages: +
2012-12-13 17:56:06 chris.jerdonek set nosy: + ezio.melottimessages: +
2012-12-13 17:13:40 eric.araujo set nosy: + eric.araujomessages: +
2012-12-13 16:40:05 asvetlov set nosy: + asvetlov
2012-12-12 03:25:11 chris.jerdonek set nosy: + chris.jerdonekmessages: +
2012-12-11 23:00:09 Mikel.Ward create