| msg207518 - (view) |
Author: Serhiy Storchaka (serhiy.storchaka) *  |
Date: 2014-01-07 09:29 |
| Here is a patch which converts xml.etree.ElementTree accelerator module to use Argument Clinic. 34 methods are converted. Not converted __init__ methods (is Argument Clinic support it?) and the SubElement function which seems can't be converted. |
|
|
| msg207597 - (view) |
Author: Larry Hastings (larry) * |
Date: 2014-01-07 20:56 |
| Can you refresh the patch? The comments Argument Clinic uses were all changed. I'll review when you have a fresh patch. And I'll look at __init__ and subelement then. |
|
|
| msg207624 - (view) |
Author: Larry Hastings (larry) * |
Date: 2014-01-07 23:52 |
| (Converting this issue into Derby entry #7. Serhiy has stated that he is abandoning his ElementTree patch; it may make a good starting point for you if you take over this issue.) 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: Modules/_elementtree.c: 31 sites PC/_msi.c: 15 sites PC/bdist_wininst/install.c: 5 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 |
|
|
| msg207630 - (view) |
Author: Serhiy Storchaka (serhiy.storchaka) * |
Date: 2014-01-08 00:00 |
| Here is refreshed patch. |
|
|
| msg207631 - (view) |
Author: Serhiy Storchaka (serhiy.storchaka) * |
Date: 2014-01-08 00:06 |
| Oh, what's the strange idea to group unrelated modules together? |
|
|
| msg207632 - (view) |
Author: Larry Hastings (larry) * |
Date: 2014-01-08 00:10 |
| As I discussed with you on IRC, the problem is, we have 129 files with sites that need examining. I was advised that creating 129 issues on the tracker was a bad idea. So I had to combine them together somehow. I wrote a Python script to group them into bundles of approximately 50 sites, and it work largely at random. I did ask you several times for feedback on the bundles, as you had expressed some interest in how the bundles were aggregated, but you did not respond. |
|
|
| msg207633 - (view) |
Author: Larry Hastings (larry) * |
Date: 2014-01-08 00:13 |
| Since you have abandoned your patch, please stop changing the issue title on me. |
|
|
| msg207638 - (view) |
Author: Serhiy Storchaka (serhiy.storchaka) * |
Date: 2014-01-08 00:17 |
| This is ridiculous. |
|
|
| msg240106 - (view) |
Author: Serhiy Storchaka (serhiy.storchaka) * |
Date: 2015-04-05 09:53 |
| Updated patch. Converted __init__ methods and removed explicit declarations of self parameters. SubElement and Element.__init__ still are not converted, as they need the support of **kwargs. |
|
|
| msg242545 - (view) |
Author: Serhiy Storchaka (serhiy.storchaka) * |
Date: 2015-05-04 07:05 |
| Made compatible with Windows compiler (I hope). |
|
|
| msg242546 - (view) |
Author: Larry Hastings (larry) * |
Date: 2015-05-04 07:17 |
| LGTM |
|
|
| msg242552 - (view) |
Author: Roundup Robot (python-dev)  |
Date: 2015-05-04 12:33 |
| New changeset fea94f9cb5a0 by Serhiy Storchaka in branch 'default': Issue #20159. Converted the _elementtree module to Argument Clinic. https://hg.python.org/cpython/rev/fea94f9cb5a0 |
|
|
| msg242554 - (view) |
Author: Serhiy Storchaka (serhiy.storchaka) * |
Date: 2015-05-04 12:51 |
| Original issue is resolved and I have no any relations to modules added by Larry. |
|
|