Re: cpython: Remove the old style [...] to denote optional args and show the defaults.

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: cpython: Remove the old style [...] to denote optional args and show the defaults.

Georg Brandl-2
Am 08.11.2011 21:30, schrieb brian.curtin:

> http://hg.python.org/cpython/rev/60ae7979fec8
> changeset:   73463:60ae7979fec8
> user:        Brian Curtin <[hidden email]>
> date:        Tue Nov 08 14:30:02 2011 -0600
> summary:
>   Remove the old style [...] to denote optional args and show the defaults.
>
> files:
>   Doc/library/os.rst |  12 ++++++------
>   1 files changed, 6 insertions(+), 6 deletions(-)
>
>
> diff --git a/Doc/library/os.rst b/Doc/library/os.rst
> --- a/Doc/library/os.rst
> +++ b/Doc/library/os.rst
> @@ -872,7 +872,7 @@
>     .. versionadded:: 3.3
>  
>  
> -.. function:: futimesat(dirfd, path[, (atime, mtime)])
> +.. function:: futimesat(dirfd, path, (atime, mtime)=None)
>  
>     Like :func:`utime` but if *path* is relative, it is taken as relative to *dirfd*.
>     If *path* is relative and *dirfd* is the special value :data:`AT_FDCWD`, then *path*

Hmm, while the [] are old style, they are still correct when the function
doesn't support kwargs.  Please revert.

(Also, the syntax ``(atime, mtime)=None`` would not be valid Python and at
is best confusing.)

Georg

_______________________________________________
Python-Dev mailing list
[hidden email]
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: http://mail.python.org/mailman/options/python-dev/lists%40nabble.com
Reply | Threaded
Open this post in threaded view
|

Re: cpython: Remove the old style [...] to denote optional args and show the defaults.

briancurtin
On Tue, Nov 8, 2011 at 14:47, Georg Brandl <[hidden email]> wrote:

> Am 08.11.2011 21:30, schrieb brian.curtin:
>> http://hg.python.org/cpython/rev/60ae7979fec8
>> changeset:   73463:60ae7979fec8
>> user:        Brian Curtin <[hidden email]>
>> date:        Tue Nov 08 14:30:02 2011 -0600
>> summary:
>>   Remove the old style [...] to denote optional args and show the defaults.
>>
>> files:
>>   Doc/library/os.rst |  12 ++++++------
>>   1 files changed, 6 insertions(+), 6 deletions(-)
>>
>>
>> diff --git a/Doc/library/os.rst b/Doc/library/os.rst
>> --- a/Doc/library/os.rst
>> +++ b/Doc/library/os.rst
>> @@ -872,7 +872,7 @@
>>     .. versionadded:: 3.3
>>
>>
>> -.. function:: futimesat(dirfd, path[, (atime, mtime)])
>> +.. function:: futimesat(dirfd, path, (atime, mtime)=None)
>>
>>     Like :func:`utime` but if *path* is relative, it is taken as relative to *dirfd*.
>>     If *path* is relative and *dirfd* is the special value :data:`AT_FDCWD`, then *path*
>
> Hmm, while the [] are old style, they are still correct when the function
> doesn't support kwargs.  Please revert.
>
> (Also, the syntax ``(atime, mtime)=None`` would not be valid Python and at
> is best confusing.)
>
> Georg

Backed out. http://hg.python.org/cpython/rev/2636df45b630
_______________________________________________
Python-Dev mailing list
[hidden email]
http://mail.python.org/mailman/listinfo/python-dev
Unsubscribe: http://mail.python.org/mailman/options/python-dev/lists%40nabble.com