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 2018年03月08日 18:39 by serhiy.storchaka, last changed 2022年04月11日 14:58 by admin. This issue is now closed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 6120 | merged | serhiy.storchaka, 2018年03月15日 07:36 | |
| PR 6433 | merged | serhiy.storchaka, 2018年04月09日 17:19 | |
| Messages (3) | |||
|---|---|---|---|
| msg313452 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | Date: 2018年03月08日 18:39 | |
The array of PyMethodDef for OrderedDict contains explicit definitions of methods like __delitem__, __eq__ and __init__. The purpose is aligning docstrings with Python implementation. But this doesn't work. Slot wrappers replace these descriptors. And docstings are standard docstrings for corresponding slot wrappers. Thus this code doesn't work. And it looks dangerous, since functions are casted to incompatible function types. Even if they are never used, the compiler (gcc 8) produces warnings (see issue33012). May be this is even undefined behavior. In that case the compiler can generate arbitrary code. I suggest to remove these definitions. |
|||
| msg315134 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | Date: 2018年04月09日 16:14 | |
New changeset 827d49f3cf0296f1e267eae6834a977cf312cc1e by Serhiy Storchaka in branch 'master': bpo-33031: Remove dead code in C implementation of OrderedDict. (GH-6120) https://github.com/python/cpython/commit/827d49f3cf0296f1e267eae6834a977cf312cc1e |
|||
| msg315146 - (view) | Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) | Date: 2018年04月09日 18:46 | |
New changeset b0f387d7ca126d486fc82744b8ec90c131230311 by Serhiy Storchaka in branch '3.7': [3.7] bpo-33031: Remove dead code in C implementation of OrderedDict. (GH-6120) (GH-6433) https://github.com/python/cpython/commit/b0f387d7ca126d486fc82744b8ec90c131230311 |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022年04月11日 14:58:58 | admin | set | github: 77212 |
| 2018年04月09日 18:47:41 | serhiy.storchaka | set | status: open -> closed resolution: fixed stage: patch review -> resolved |
| 2018年04月09日 18:46:47 | serhiy.storchaka | set | messages: + msg315146 |
| 2018年04月09日 17:19:27 | serhiy.storchaka | set | pull_requests: + pull_request6129 |
| 2018年04月09日 16:14:31 | serhiy.storchaka | set | messages: + msg315134 |
| 2018年03月15日 12:10:15 | serhiy.storchaka | link | issue33012 dependencies |
| 2018年03月15日 07:37:18 | serhiy.storchaka | set | stage: patch review |
| 2018年03月15日 07:36:47 | serhiy.storchaka | set | stage: patch review -> (no value) versions: + Python 3.7 |
| 2018年03月15日 07:36:21 | serhiy.storchaka | set | keywords:
+ patch stage: patch review pull_requests: + pull_request5884 |
| 2018年03月08日 18:39:09 | serhiy.storchaka | create | |