➜
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 2013-08-21 19:51 by eric.snow, last changed 2022-04-11 14:57 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| inspect_classify_01.patch | yselivanov, 2014-01-30 05:33 | review | ||
| Messages (5) | |||
|---|---|---|---|
| msg195818 - (view) | Author: Eric Snow (eric.snow) *  |
Date: 2013-08-21 19:51 | |
>>> pprint.pprint(inspect.classify_class_attrs(object))
[Attribute(name='__class__', kind='data', defining_class=<class 'object'>, object=<attribute '__class__' of 'object' objects>),
...
Attribute(name='__init__', kind='method', defining_class=<class 'object'>, object=<slot wrapper '__init__' of 'object' objects>),
...
Attribute(name='__new__', kind='data', defining_class=<class 'object'>, object=<built-in method __new__ of type object at 0x8aee20>),
...
]
I haven't had a chance to look into why __new__() falls through the cracks but expect it's due to how __new__() is treated like a staticmethod without being one. I suppose there could be other similar cases, but __new__() is probably the only oddball here.
An extra test using isbuiltin() fixes this.
else:
obj_via_getattr = getattr(cls, name)
if (ismethod(obj_via_getattr) or
- ismethoddescriptor(obj_via_getattr)):
+ ismethoddescriptor(obj_via_getattr) or
+ isbuiltin(obj_via_getattr)):
kind = "method"
else:
kind = "data"
|
|||
| msg209704 - (view) | Author: Yury Selivanov (yselivanov) * |
Date: 2014-01-30 05:33 | |
Tim, Eric, Please review the attached patch. |
|||
| msg209817 - (view) | Author: Roundup Robot (python-dev)  |
Date: 2014-01-31 19:39 | |
New changeset c38f99554be4 by Yury Selivanov in branch 'default': inspect.classify_class_attrs: Classify object.__new__ and __init__ correctly #18801 http://hg.python.org/cpython/rev/c38f99554be4 |
|||
| msg209818 - (view) | Author: Yury Selivanov (yselivanov) * |
Date: 2014-01-31 19:40 | |
Not sure if we need to backport this to older python versions |
|||
| msg209819 - (view) | Author: Roundup Robot (python-dev) |
Date: 2014-01-31 19:42 | |
New changeset d0f95094033d by Yury Selivanov in branch 'default': NEWS: Add news item for #18801 http://hg.python.org/cpython/rev/d0f95094033d |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:57:49 | admin | set | github: 63001 |
| 2014-01-31 19:42:42 | python-dev | set | messages: + msg209819 |
| 2014-01-31 19:40:58 | yselivanov | set | status: open -> closed resolution: fixed |
| 2014-01-31 19:40:38 | yselivanov | set | messages: + msg209818 |
| 2014-01-31 19:39:32 | python-dev | set | nosy:
+ python-dev messages: + msg209817 |
| 2014-01-30 05:33:35 | yselivanov | set | files:
+ inspect_classify_01.patch nosy: + yselivanov, tim.peters messages: + msg209704 keywords: + patch |
| 2013-08-21 19:51:34 | eric.snow | create | |