[issue9751] _PyInstance_Lookup() defeats its purpose

[Armin Rigo]

Armin Rigo <arigo at users.sourceforge.net> added the comment:

I propose that we first attempt to fix the crasher; depending on the solution we might then either fix the doc or the code for _PyInstance_Lookup().

If no-one is willing to fix this bug I am fine to let it go.  But somehow I am sure that there is code *somewhere* that sticks non-string keys in old-style classes or instances, and whose __eq__ has side-effects, like caching.  This would create hard-to-reproduce and hard-to-diagnose segfaults.  That's mainly the reason why I think that it should be fixed.

----------

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue9751>
_______________________________________