This issue tracker has been migrated to GitHub ,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2014年01月07日 09:29 by serhiy.storchaka, last changed 2022年04月11日 14:57 by admin.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| etree_clinic.patch | serhiy.storchaka, 2014年01月18日 11:51 | review | ||
| etree_clinic_2.patch | serhiy.storchaka, 2015年04月05日 09:53 | review | ||
| etree_clinic_3.patch | serhiy.storchaka, 2015年05月04日 07:05 | review | ||
| Messages (13) | |||
|---|---|---|---|
| msg207518 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | 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) * (Python committer) | 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) * (Python committer) | 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) * (Python committer) | Date: 2014年01月08日 00:00 | |
Here is refreshed patch. |
|||
| msg207631 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | Date: 2014年01月08日 00:06 | |
Oh, what's the strange idea to group unrelated modules together? |
|||
| msg207632 - (view) | Author: Larry Hastings (larry) * (Python committer) | 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) * (Python committer) | 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) * (Python committer) | Date: 2014年01月08日 00:17 | |
This is ridiculous. |
|||
| msg240106 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | 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) * (Python committer) | Date: 2015年05月04日 07:05 | |
Made compatible with Windows compiler (I hope). |
|||
| msg242546 - (view) | Author: Larry Hastings (larry) * (Python committer) | Date: 2015年05月04日 07:17 | |
LGTM |
|||
| msg242552 - (view) | Author: Roundup Robot (python-dev) (Python triager) | 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) * (Python committer) | Date: 2015年05月04日 12:51 | |
Original issue is resolved and I have no any relations to modules added by Larry. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:57:56 | admin | set | github: 64358 |
| 2015年05月04日 12:52:21 | serhiy.storchaka | set | nosy:
- serhiy.storchaka |
| 2015年05月04日 12:51:40 | serhiy.storchaka | set | nosy:
scoder, larry, eli.bendersky, python-dev, serhiy.storchaka dependencies: - Add docstrings for ElementTree module messages: + msg242554 stage: patch review -> needs patch |
| 2015年05月04日 12:33:29 | python-dev | set | nosy:
+ python-dev messages: + msg242552 |
| 2015年05月04日 07:17:35 | larry | set | messages: + msg242546 |
| 2015年05月04日 07:05:27 | serhiy.storchaka | set | files:
+ etree_clinic_3.patch messages: + msg242545 |
| 2015年04月05日 09:53:32 | serhiy.storchaka | set | files:
+ etree_clinic_2.patch messages: + msg240106 |
| 2015年02月25日 15:24:55 | serhiy.storchaka | set | components: + Argument Clinic |
| 2014年01月18日 11:51:18 | serhiy.storchaka | set | files: + etree_clinic.patch |
| 2014年01月18日 11:49:42 | serhiy.storchaka | set | files: - etree_clinic.patch |
| 2014年01月18日 11:49:34 | serhiy.storchaka | set | files: - etree_clinic.patch |
| 2014年01月09日 09:34:36 | serhiy.storchaka | set | assignee: eli.bendersky dependencies: + Add docstrings for ElementTree module stage: needs patch -> patch review |
| 2014年01月08日 01:36:37 | r.david.murray | link | issue20187 dependencies |
| 2014年01月08日 00:17:25 | serhiy.storchaka | set | messages: + msg207638 |
| 2014年01月08日 00:13:16 | larry | set | messages:
+ msg207633 title: Derby: Convert the ElementTree module to use Argument Clinic -> Derby #7: Convert 51 sites to Argument Clinic across 3 files -> Derby: Convert the ElementTree module to use Argument Clinic |
| 2014年01月08日 00:10:39 | larry | set | messages: + msg207632 |
| 2014年01月08日 00:06:00 | serhiy.storchaka | set | messages: + msg207631 |
| 2014年01月08日 00:04:27 | serhiy.storchaka | set | versions: + Python 3.5, - Python 3.4 |
| 2014年01月08日 00:00:46 | serhiy.storchaka | set | files:
+ etree_clinic.patch messages: + msg207630 title: Derby #7: Convert 51 sites to Argument Clinic across 3 files -> Derby: Convert the ElementTree module to use Argument Clinic |
| 2014年01月07日 23:52:09 | larry | set | title: Derby: Convert the ElementTree module to use Argument Clinic -> Derby #7: Convert 51 sites to Argument Clinic across 3 files messages: + msg207624 stage: patch review -> needs patch |
| 2014年01月07日 20:56:04 | larry | set | messages:
+ msg207597 title: Convert the ElementTree module to use Argument Clinic -> Derby: Convert the ElementTree module to use Argument Clinic |
| 2014年01月07日 09:29:52 | serhiy.storchaka | set | dependencies: + Argument Clinic: broken support for 'O!', Argument Clinic generates wrong keyword parameter name for "default" |
| 2014年01月07日 09:29:26 | serhiy.storchaka | create | |