classification
Title: Derby #18: Convert 31 sites to Argument Clinic across 23 files
Type: enhancement Stage: patch review
Components: Argument Clinic, Extension Modules Versions: Python 3.7
process
Status: open Resolution:
Dependencies: 34797 Superseder:
Assigned To: Nosy List: georg.brandl, larry, martin.panter, python-dev, rhettinger, serhiy.storchaka, taleinat
Priority: normal Keywords: patch

Created on 2014-01-08 00:23 by larry, last changed 2018-09-25 13:33 by serhiy.storchaka.

Files
File name Uploaded Description Edit
heapq_clinic.patch georg.brandl, 2014-01-12 09:04 review
modules_issue20186.patch georg.brandl, 2014-01-13 07:14 review
issue20186.mathmodule.patch taleinat, 2015-06-03 08:11 updated patch for mathmodule review
issue20186.enumobject.patch taleinat, 2015-06-03 10:52 review
issue20186._operator.patch taleinat, 2015-06-07 20:38 AC conversion of nearly all functions in the _operator module review
issue20186._operator.v2.patch taleinat, 2015-06-07 21:20 slightly revised patch after Serhiy's review review
issue20186._operator.v3.patch taleinat, 2015-06-07 21:48 slightly revised (again) patch after Serhiy's additional review review
issue20186._operator.v4.patch taleinat, 2015-06-08 06:18 slightly revised (yet again) patch after Serhiy's latest review review
issue20186.mathmodule.v2.patch taleinat, 2015-06-12 21:23 revised patch for mathmodule.c review
csv_clinic.patch serhiy.storchaka, 2017-01-20 16:38 review
lsprof_clinic.patch serhiy.storchaka, 2017-01-20 16:38 review
tracemalloc_clinic.patch serhiy.storchaka, 2017-01-20 16:38 review
symtable_clinic.patch serhiy.storchaka, 2017-01-20 16:38 review
enumobject-docstrings.patch serhiy.storchaka, 2017-01-20 18:18 review
Pull Requests
URL Status Linked Edit
PR 614 merged serhiy.storchaka, 2017-03-11 08:34
Messages (54)
msg207644 - (view) Author: Larry Hastings (larry) * (Python committer) Date: 2014-01-08 00:23
This issue is part of the Great Argument Clinic Conversion Derby,
where we're trying to convert as much of Python 3.4 to use
Argument Clinic as we can before Release Candidate 1 on January 19.

This issue asks you to change the following bundle of files:
    Objects/tupleobject.c: 2 sites
    Objects/memoryobject.c: 2 sites
    Objects/descrobject.c: 2 sites
    Objects/complexobject.c: 2 sites
    Modules/_operator.c: 2 sites
    Modules/_opcode.c: 2 sites
    Modules/_lsprof.c: 2 sites
    Modules/_heapqmodule.c: 2 sites
    Objects/weakrefobject.c: 1 sites
    Objects/structseq.c: 1 sites
    Objects/rangeobject.c: 1 sites
    Objects/object.c: 1 sites
    Objects/moduleobject.c: 1 sites
    Objects/funcobject.c: 1 sites
    Objects/fileobject.c: 1 sites
    Objects/enumobject.c: 1 sites
    Objects/codeobject.c: 1 sites
    Objects/boolobject.c: 1 sites
    Modules/symtablemodule.c: 1 sites
    Modules/mathmodule.c: 1 sites
    Modules/_tracemalloc.c: 1 sites
    Modules/_io/_iomodule.c: 1 sites
    Modules/_csv.c: 1 sites

Talk to me (larry) if you only want to attack part of a bundle.

For instructions on how to convert a function to work with Argument
Clinic, read the "howto":
    http://docs.python.org/dev/howto/clinic.html
msg207914 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2014-01-11 21:02
Looking at _csv.c, I see a few functions using PyArg_UnpackTuple.  They should be converted too, no?
msg207917 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2014-01-11 21:56
Attached part 1 of mathmodule (17 functions).

I'm looking forward to a suggestion for handling the rest (see FUNC1/1A/2 macros :)
msg207923 - (view) Author: Larry Hastings (larry) * (Python committer) Date: 2014-01-11 23:32
1)
Wow.  I never knew about PyArg_UnpackTuple.  You're right, those should be converted too.  Hooray, more entry points to convert.

I'll write something up for the howto about UnpackTuple.

I just did a quick check, and there are 96 entry points (by my count) that use PyArg_UnpackTuple().  Shall I create Derby issues #19 and #20, or do you have a better idea?

2) For FUNC1 / 1A / 2 macros: right now you'd have to just copy and paste over and over.  There might be something you could do with a [python] block where you automatedly reuse the existing sigantures.  I was thinking about having Clinic support it directly, maybe with the syntax:

/*[clinic input]
func_name = existing_func_name

docstring goes here
[...]*/

You'd skip the parameters and the return annotation.  You could only reuse functions from the current file.  Would that be a big boon to you?
msg207931 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2014-01-12 07:56
> Wow.  I never knew about PyArg_UnpackTuple.  You're right, those 
> should be converted too.  Hooray, more entry points to convert.
> I'll write something up for the howto about UnpackTuple.

One thing to note is that (at least in math) many instances of UnpackTuple could have been replaced by ParseTuple.  See for example math_hypot: it uses UnpackTuple to get two objects, and then immediately calls PyFloat_AsDouble on them.  I've converted these using 'd' and not 'O' specifiers.

> I just did a quick check, and there are 96 entry points (by my count) 
> that use PyArg_UnpackTuple().  Shall I create Derby issues #19 and 
> #20, or do you have a better idea?

Probably better to add them to the issues that cover their modules, otherwise people might get confused.

> 2) For FUNC1 / 1A / 2 macros: right now you'd have to just copy and 
> paste over and over.  There might be something you could do with a 
> [python] block where you automatedly reuse the existing sigantures.  
> I was thinking about having Clinic support it directly, maybe with 
> the syntax:

> /*[clinic input]
> func_name = existing_func_name
> 
> docstring goes here
> [...]*/
> 
> You'd skip the parameters and the return annotation.  You could only 
> reuse functions from the current file.  Would that be a big boon to you?

That sounds good.

On the other hand, if clinic expanded cpp macros we could... *:-)
msg207933 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2014-01-12 08:32
OK, here's a patch for _csv.  Two problems here:

First problem is the __new__ method of the Dialect class:

* it has no docstring and no methoddef entry
* it is a class method, but the first arg is conventionally called "type"

I tried to hack something into clinic with a new decorator, but it may not be how you want it to look, take care.

Second problem is the functions reader(), writer(), register_dialect(): they parse their *args but pass their **kwargs through to another class.
Is there anything like a "**kwds" argument specifier?

BTW, for a module like _csv that is exported through a Python module named csv, should we use the "real" or the "nice" module name?
msg207934 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2014-01-12 08:35
Tried to tackle symtable -- it uses an O& converter.  The clinic howto says

``'O&'``    ``object(converter='name_of_c_function')``

but

Traceback (most recent call last):
  File "Tools/clinic/clinic.py", line 2817, in <module>
    sys.exit(main(sys.argv[1:]))
  File "Tools/clinic/clinic.py", line 2813, in main
    parse_file(filename, output=ns.output, verify=not ns.force)
  File "Tools/clinic/clinic.py", line 1116, in parse_file
    cooked = clinic.parse(raw)
  File "Tools/clinic/clinic.py", line 1066, in parse
    parser.parse(block)
  File "Tools/clinic/clinic.py", line 2109, in parse
    self.state(line)
  File "Tools/clinic/clinic.py", line 2378, in state_parameter
    converter = dict[name](parameter_name, self.function, value, **kwargs)
  File "Tools/clinic/clinic.py", line 1403, in __init__
    self.converter_init(**kwargs)
TypeError: converter_init() got an unexpected keyword argument 'converter'
msg207935 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2014-01-12 08:46
_tracemalloc converted.

Its existing docstrings did use the

func(arg: argtype) -> rettype

convention.  Is there a way in clinic to retain that?
msg207936 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2014-01-12 08:55
Here's _iomodule.  _io.open has a whopping 100-line docstring, which is ... unfortunate ... to have duplicated in the file :)
msg207937 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2014-01-12 09:04
And _heapq.  No problems there, except that it also used "->" return annotations in the docstring.
msg207938 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2014-01-12 09:16
And lsprof.
msg208009 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2014-01-13 07:07
OK, new patches coming in.
msg208011 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2014-01-13 07:14
Actually I put all I have in one.  Rietveld doesn't care.

The mathmodule still awaits some kind of solution for the macro atrocities.

Objects will be attacked next.
msg218717 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2014-05-17 22:36
New changeset 060cfd049d14 by Stefan Krah in branch 'default':
Issue #20186: memoryobject.c: add function signatures.
http://hg.python.org/cpython/rev/060cfd049d14
msg218718 - (view) Author: Stefan Krah (skrah) * (Python committer) Date: 2014-05-17 22:39
memoryobject.c is converted with a minimal patch (I would like to keep
100% code coverage for the file).
msg224767 - (view) Author: Larry Hastings (larry) * (Python committer) Date: 2014-08-04 20:14
All the Derby patches should only go into trunk at this point.
msg244731 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2015-06-03 08:11
Attached is an updated patch for Modules/mathmodule.c.

This is based on Georg's patch, updated to apply to current 3.5, with several improvements:

* replaced legacy converters
* converted math.ceil() and math.floor() functions
* converted the new math.gcd() and math.isclose() functions
* AC generated code in separate file: Modules/clinic/mathmodule.c.h
* this patch doesn't change any internal variable names in the C code
msg244736 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2015-06-03 10:20
Should Argument Clinic conversion patches still be against the 'default' branch, and not 3.5, even though they don't include any functionality changes?
msg244737 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2015-06-03 10:52
Attached is an AC conversion patch for Objects/enumobject.c.

Note that this file contains the implementations of the 'enumerate' and 'reversed' classes, but *not* the 'Enum' class.

This is based on the 3.5 branch.
msg244971 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2015-06-07 20:38
Attached is a patch for all of _operator except for itemgetter, attrgetter and methodcaller. The entire test suite passes after applying this patch.

Using AC has allowed the removal of all of the cryptic "spam*()" macros in the code, making it much more straightforward. In terms of readability, IMO this is a great improvement.

I skipped itemgetter, attrgetter and methodcaller since they all support passing a variable number of arguments and treating those as a sequence of values.
msg244972 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2015-06-07 20:54
Is there any performance difference?
msg244973 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2015-06-07 20:59
I tried running the pystone benchmark, but the results were inconclusive. I saw very large differences (up to 20%) between runs at different times, with no clear differences with or without the patch.

However, a quick search shows that the operator module is almost completely unused by the rest of the code and the stdlib. So I really don't think performance is an issue here.
msg244974 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2015-06-07 21:20
Attached a slightly revised patch thanks to Serhiy's review.

In addition to Serhiy's remarks, I used "_operator._compare_digest = _operator.eq" to reduce a bit more boilerplate.
msg244975 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2015-06-07 21:36
The operator module is rarely used in the stdlib, but if it is used in user code (mainly with map(), reduce() or like) the performance often is important. You can use microbenchmarks like following (operator.add is twice faster than lambda x, y: x + y).

    ./python -m timeit -s "import operator; a = list(range(10000)); b = a[:]" -- "list(map(operator.add, a, b))"
msg244977 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2015-06-07 21:41
I just ran such microbenchmarks for operator.add and operator.not_, and saw no significant difference with or without the patch.
msg244978 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2015-06-07 21:48
Here's another complete conversion patch for _operator.

As suggested by Serhiy, I changed the comparison operators to copy the signature from _operator.eq() instead of _operator.lt(), which is easier to understand.
msg244987 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2015-06-08 06:18
Here's another version of the _operator patch, with another small change after Serhiy's latest review.
msg244989 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2015-06-08 06:42
issue20186._operator.v4.patch LGTM.
msg245276 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2015-06-12 21:23
Attached is a revised patch for Modules/mathmodule.c incorporating changes suggested by Serhiy in his review.

The major change is the reformatting of the doc-strings of most of the functions in the module to use the same format as doc-strings generated by AC.
msg262010 - (view) Author: Martin Panter (martin.panter) * (Python committer) Date: 2016-03-19 00:33
Summary of the work here as I pass through :)

Files already done:
    Objects/memoryobject.c: converted to signatures without Arg Clinic (Py 3.5)
    Modules/_opcode.c: Issue 19674 (3.4). Only one function I can see there (Larry’s original post says two sites).
    Modules/_io/_iomodule.c: part of Georg’s modules_issue20186.patch, but already handled in Issue 20175 (3.5)

Files being worked on:
    Modules/_operator.c: patch by Tal, ready for commit?
    Modules/_lsprof.c: Georg’s modules_issue20186.patch
    Modules/_heapqmodule.c: restored Georg’s patch; looks like he meant to add it to the main patch but it got lost
    Objects/enumobject.c: patch by Tal
    Modules/symtablemodule.c: Georg’s modules_issue20186.patch
    Modules/mathmodule.c: patch by Tal
    Modules/_tracemalloc.c: modules_issue20186.patch; needs update from review comments
    Modules/_csv.c: modules_issue20186.patch; couple of review comments

That leaves the following files from OP:
    Objects/tupleobject.c: 2 sites
    Objects/descrobject.c: 2 sites
    Objects/complexobject.c: 2 sites
    Objects/weakrefobject.c: 1 sites
    Objects/structseq.c: 1 sites
    Objects/rangeobject.c: 1 sites
    Objects/object.c: 1 sites
    Objects/moduleobject.c: 1 sites
    Objects/funcobject.c: 1 sites
    Objects/fileobject.c: 1 sites
    Objects/codeobject.c: 1 sites
    Objects/boolobject.c: 1 sites
msg262022 - (view) Author: Larry Hastings (larry) * (Python committer) Date: 2016-03-19 06:59
>    Modules/_opcode.c: Issue 19674 (3.4). Only one function I can see there (Larry’s original post says two sites).

I produced the post with a big grep through the codebase.  Which was quite a while ago now.  Code changes and moves around; if you can only find one site in the current file I believe you.
msg262036 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2016-03-19 10:22
My patches haven't been looked at for a while, anyone considering them should make sure they still apply cleanly and don't break anything. Here their status as far as I can tell:

Quite a bit of work went into Modules/_operator.c, and it got an "LGTM" from Serhiy, so it should probably go in after a quick check.

Modules/mathmodule.c was also in a good state when I left it, and it underwent some review from Serhiy whose comments were implemented. It should probably be good to go after a thorough review.

Objects/enumobject.c is simpler but requires review.
msg285798 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2017-01-19 15:44
New changeset 3713f7de576d by Serhiy Storchaka in branch 'default':
Issue #20186: Converted the _operator module to Argument Clinic.
https://hg.python.org/cpython/rev/3713f7de576d
msg285799 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-01-19 15:46
The only change I made is used the return converter in length_hint().
msg285802 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2017-01-19 16:13
New changeset 112f27b8c8ea by Serhiy Storchaka in branch 'default':
Issue #20186: Converted the math module to Argument Clinic.
https://hg.python.org/cpython/rev/112f27b8c8ea
msg285803 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-01-19 16:15
issue20186.mathmodule.v2.patch needed just synchronizing docstrings.
msg285806 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-01-19 16:46
issue20186.enumobject.patch LGTM too.
msg285807 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2017-01-19 16:48
New changeset e3db9bccff3f by Serhiy Storchaka in branch 'default':
Issue #20186: Converted builtins enumerate() and reversed() to Argument Clinic.
https://hg.python.org/cpython/rev/e3db9bccff3f
msg285808 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-01-19 16:49
Sorry for committing patches for you Tal, but they were hanging so long time.
msg285833 - (view) Author: Tal Einat (taleinat) * (Python committer) Date: 2017-01-19 19:16
Serhiy, no apology is required. On the contrary, thank you for the taking the time to review this and commit, I don't have time available for this these days.
msg285915 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-01-20 16:38
There are patches based on modules_issue20186.patch that convert to Argument 
Clinic 4 modules: _csv, _lsprof, _tracemalloc and symtable. They are 
synchronized with current sources and updated to current Argument Clinic. 
Other changes:

* Addressed Larry's comments.

* Used converter names instead of format units (object instead of 'O' etc).

* Removed unneeded self declarations in _lsprof.

* Removed changes for _csv.Dialect.__new__. Generated signature has None as 
default values, but this is not true. Actually parameters don't have default 
values. This function can't be converted.

* Parameters of _lsprof.Profiler.enable should have the int converter, not 
bool. Default values are -1 (meaning "don't change current value"), not True.

* Made the parameter of _tracemalloc._get_object_traceback() positional-only.

* Simplified code for creating a result in _traceback functions.
msg285916 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2017-01-20 17:07
The application of AC to enumerate() lost information about the start argument and the signature of the call.   We're going backwards.

---- New help -----------------------------------------------------
class enumerate(object)
 |  Return an enumerate object.
 |
 |    iterable
 |      an object supporting iteration
 |
 |  The enumerate object yields pairs containing a count (from start, which
 |  defaults to zero) and a value yielded by the iterable argument.
 |
 |  enumerate is useful for obtaining an indexed list:
 |      (0, seq[0]), (1, seq[1]), (2, seq[2]), ...

---- Old help -----------------------------------------------------

class enumerate(object)
 |  enumerate(iterable[, start]) -> iterator for index, value of iterable
 |  
 |  Return an enumerate object.  iterable must be another object that supports
 |  iteration.  The enumerate object yields pairs containing a count (from
 |  start, which defaults to zero) and a value yielded by the iterable argument.
 |  enumerate is useful for obtaining an indexed list:
 |      (0, seq[0]), (1, seq[1]), (2, seq[2]), ...


Also, reversed() lost the indication of its signature:
      reversed(sequence) -> reverse iterator over values of the sequence

And the help doesn't have the usual:

    iterable
    |      an object supporting iteration
msg285917 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2017-01-20 17:17
When reviewing AC patches, we should always compare the help() before and after.  

Also, if the code already had fast parsing like:
   if (!PyArg_UnpackTuple(args, "reversed", 1, 1, &seq) )
there needs to be a before and after timing to make sure there wasn't a performance regression.

When PyArg_NoKeywords was present, we need to verify that the AC version also precludes keyword arguments (to prevent the creation of unhelpful keyword args and to keep compatibility with other versions of Python).
msg285919 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-01-20 18:04
Microbenchmarks:

$ ./python -m perf timeit --duplicate 100 "enumerate('abc')"
Unpatched:  Median +- std dev: 1.76 us +- 0.10 us
Patched:    Median +- std dev: 1.61 us +- 0.07 us

$ ./python -m perf timeit --duplicate 100 "enumerate('abc', 1)"
Unpatched:  Median +- std dev: 2.14 us +- 0.09 us
Patched:    Median +- std dev: 1.76 us +- 0.07 us

$ ./python -m perf timeit --duplicate 100 "reversed('abc')"
Unpatched:  Median +- std dev: 1.20 us +- 0.06 us
Patched:    Median +- std dev: 1.20 us +- 0.07 us

enumerate() is 9-21% faster (due to avoiding of tuple creating), reversed() is not changed (Argument Clinic generates the same parsing code for it).
msg285920 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-01-20 18:18
Here is a patch that restores ol docstrings for enumerate() and reversed(). But it may be better to change pydoc so that it would output a text signature of class constructor if available.
msg286939 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-02-04 09:48
Argument Clinic generates incorrect parsing code for _csv.field_size_limit().
msg286947 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-02-04 10:08
The problem with lsprof_clinic.patch is that it exposes default value of _lsprof.Profiler.enable() parameters as -1. Actually _lsprof.Profiler.enable() should accept boolean arguments without default value.
msg286950 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2017-02-04 10:13
New changeset 7f8a3eb3459e by Serhiy Storchaka in branch 'default':
Issue #20186: Converted the symtable module to Argument Clinic.
https://hg.python.org/cpython/rev/7f8a3eb3459e
msg286952 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2017-02-04 10:18
New changeset e1df73b46094 by Serhiy Storchaka in branch 'default':
Issue #20186: Converted the tracemalloc module to Argument Clinic.
https://hg.python.org/cpython/rev/e1df73b46094
msg286955 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2017-02-04 11:00
New changeset b0ef37ec83f337b4b77275b367288a5656a0682c by Serhiy Storchaka in branch 'master':
Issue #20186: Converted the symtable module to Argument Clinic.
https://github.com/python/cpython/commit/b0ef37ec83f337b4b77275b367288a5656a0682c

New changeset 18a02e9d1f8e981b7b2f4287a4ed871021b13ade by Serhiy Storchaka in branch 'master':
Issue #20186: Converted the tracemalloc module to Argument Clinic.
https://github.com/python/cpython/commit/18a02e9d1f8e981b7b2f4287a4ed871021b13ade
msg287062 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2017-02-05 20:59
New changeset 8ccb3ad39ee4 by Serhiy Storchaka in branch 'default':
Issue #20186: Regenerated Argument Clinic.
https://hg.python.org/cpython/rev/8ccb3ad39ee4
msg287063 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2017-02-05 21:00
New changeset 5762cf299f863e06244e6b44ba3a91efee7b35c1 by Serhiy Storchaka in branch 'master':
Issue #20186: Regenerated Argument Clinic.
https://github.com/python/cpython/commit/5762cf299f863e06244e6b44ba3a91efee7b35c1
msg289440 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-03-11 08:35
PR 614: Objects/tupleobject.c
msg290163 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-03-24 22:08
New changeset 0b5615926a573c19c887a701a2f7047f4fd06de6 by Serhiy Storchaka in branch 'master':
bpo-20186: Convert tuple object implementation to Argument Clinic. (#614)
https://github.com/python/cpython/commit/0b5615926a573c19c887a701a2f7047f4fd06de6
History
Date User Action Args
2018-09-25 13:33:22serhiy.storchakasetdependencies: + Convert heapq to the argument clinic
2017-03-24 22:08:52serhiy.storchakasetmessages: + msg290163
2017-03-11 08:35:50serhiy.storchakasetmessages: + msg289440
2017-03-11 08:34:55serhiy.storchakasetpull_requests: + pull_request507
2017-02-05 21:00:23python-devsetmessages: + msg287063
2017-02-05 20:59:05python-devsetmessages: + msg287062
2017-02-04 11:00:20python-devsetmessages: + msg286955
2017-02-04 10:18:53python-devsetmessages: + msg286952
2017-02-04 10:13:46python-devsetmessages: + msg286950
2017-02-04 10:08:16serhiy.storchakasetmessages: + msg286947
2017-02-04 09:48:32serhiy.storchakasetmessages: + msg286939
2017-01-20 18:18:19serhiy.storchakasetfiles: + enumobject-docstrings.patch
2017-01-20 18:18:02serhiy.storchakasetmessages: + msg285920
2017-01-20 18:04:07serhiy.storchakasetmessages: + msg285919
2017-01-20 17:17:41rhettingersetmessages: + msg285917
2017-01-20 17:07:37rhettingersetnosy: + rhettinger
messages: + msg285916
2017-01-20 16:38:18serhiy.storchakasetfiles: + csv_clinic.patch, lsprof_clinic.patch, tracemalloc_clinic.patch, symtable_clinic.patch

messages: + msg285915
2017-01-19 19:16:17taleinatsetmessages: + msg285833
2017-01-19 16:49:52serhiy.storchakasetmessages: + msg285808
2017-01-19 16:48:40python-devsetmessages: + msg285807
2017-01-19 16:46:49serhiy.storchakasetmessages: + msg285806
2017-01-19 16:15:06serhiy.storchakasetmessages: + msg285803
2017-01-19 16:13:39python-devsetmessages: + msg285802
2017-01-19 15:46:05serhiy.storchakasetmessages: + msg285799
versions: + Python 3.7, - Python 3.6
2017-01-19 15:44:41python-devsetmessages: + msg285798
2016-03-19 10:22:58taleinatsetmessages: + msg262036
2016-03-19 06:59:22larrysetmessages: + msg262022
2016-03-19 00:33:05martin.pantersetnosy: + martin.panter
messages: + msg262010
2016-03-19 00:00:31martin.pantersetfiles: + heapq_clinic.patch
2015-06-12 21:23:19taleinatsetfiles: + issue20186.mathmodule.v2.patch

messages: + msg245276
2015-06-08 06:42:02serhiy.storchakasetmessages: + msg244989
2015-06-08 06:18:49taleinatsetfiles: + issue20186._operator.v4.patch

messages: + msg244987
2015-06-07 21:48:31taleinatsetfiles: + issue20186._operator.v3.patch

messages: + msg244978
2015-06-07 21:41:47taleinatsetmessages: + msg244977
2015-06-07 21:36:24serhiy.storchakasetmessages: + msg244975
2015-06-07 21:20:00taleinatsetfiles: + issue20186._operator.v2.patch

messages: + msg244974
2015-06-07 20:59:32taleinatsetmessages: + msg244973
2015-06-07 20:54:35serhiy.storchakasetnosy: + serhiy.storchaka
messages: + msg244972
2015-06-07 20:38:36taleinatsetfiles: + issue20186._operator.patch

messages: + msg244971
2015-06-03 10:52:38taleinatsetfiles: + issue20186.enumobject.patch

messages: + msg244737
2015-06-03 10:20:51taleinatsetmessages: + msg244736
2015-06-03 09:54:09serhiy.storchakasetstage: needs patch -> patch review
versions: + Python 3.6, - Python 3.5
2015-06-03 08:11:30taleinatsetfiles: + issue20186.mathmodule.patch
nosy: + taleinat
messages: + msg244731

2015-02-25 15:29:12serhiy.storchakasetcomponents: + Argument Clinic
2014-10-14 15:42:23skrahsetnosy: - skrah
2014-08-04 20:14:59larrysetmessages: + msg224767
versions: + Python 3.5, - Python 3.4
2014-05-17 22:39:59skrahsetnosy: + skrah
messages: + msg218718
2014-05-17 22:36:20python-devsetnosy: + python-dev
messages: + msg218717
2014-01-13 07:14:04georg.brandlsetfiles: + modules_issue20186.patch

messages: + msg208011
2014-01-13 07:07:43georg.brandlsetmessages: + msg208009
2014-01-13 07:07:33georg.brandlsetfiles: - lsprof_clinic.patch
2014-01-13 07:07:29georg.brandlsetfiles: - heapq_clinic.patch
2014-01-13 07:07:25georg.brandlsetfiles: - io_clinic.patch
2014-01-13 07:07:19georg.brandlsetfiles: - tracemalloc_clinic.patch
2014-01-13 07:07:16georg.brandlsetfiles: - csv_module_clinic.patch
2014-01-13 07:07:09georg.brandlsetfiles: - mathmodule_part1.patch
2014-01-12 09:16:05georg.brandlsetfiles: + lsprof_clinic.patch

messages: + msg207938
2014-01-12 09:04:37georg.brandlsetfiles: + heapq_clinic.patch

messages: + msg207937
2014-01-12 08:55:40georg.brandlsetfiles: + io_clinic.patch

messages: + msg207936
2014-01-12 08:46:13georg.brandlsetfiles: + tracemalloc_clinic.patch

messages: + msg207935
2014-01-12 08:35:21georg.brandlsetmessages: + msg207934
2014-01-12 08:32:19georg.brandlsetfiles: + csv_module_clinic.patch

messages: + msg207933
2014-01-12 07:56:58georg.brandlsetmessages: + msg207931
2014-01-11 23:32:59larrysetmessages: + msg207923
2014-01-11 21:56:24georg.brandlsetfiles: + mathmodule_part1.patch
keywords: + patch
messages: + msg207917
2014-01-11 21:02:17georg.brandlsetnosy: + georg.brandl
messages: + msg207914
2014-01-08 01:36:37r.david.murraylinkissue20187 dependencies
2014-01-08 00:23:06larrycreate