Overzealous Optimization For `isinstance`
Personne n'a encore pris cette issue.
- Langage dominant
- Python
- Étoiles
- 77.2k
- Forks
- 35.9k
- Métriques de merge des PR
- Métriques de PR en attente
Description
Bug report
Bug description:
The custom __instancecheck__ dunder method allows full customization (described in the Data model).
However, it produces a spurious result:
class CustomIsInstanceMeta(type):
def __instancecheck__(cls, instance: object) -> bool:
return type(instance) is int
class CustomIsInstance(metaclass=CustomIsInstanceMeta):
...
class CustomIsInstanceSubclass(CustomIsInstance):
...
print(isinstance(5, CustomIsInstance)) # True
print(isinstance(CustomIsInstance(), CustomIsInstance)) # True (should be False)
print(isinstance(CustomIsInstanceSubclass(), CustomIsInstance)) # False
This is caused by:
https://github.com/python/cpython/blob/23c488d6197191d355be6733699a295c20806932/Objects/abstract.c#L2641-L2644
which is invalid in this case.
This is a reframing of #79264 (request to fix documentation)
CPython versions tested on:
3.13
Operating systems tested on:
Linux
Linked PRs
- gh-144874
- gh-144997
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Examinez Objects/abstract.c aux lignes 2641-2644 ainsi que les PR liés gh-144874 et gh-144997. Exécutez la reproduction isinstance fournie sur Python 3.13/Linux pour confirmer les résultats erronés. C’est terminé lorsque le comportement signalé est corrigé et que le résultat est conforme au problème de documentation soulevé dans #79264.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- c, python
- Domaine
- backend
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 25/100