LOAD_FAST access names outside of co_varnames
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
Documentation
LOAD_FAST is documented (https://docs.python.org/3.13/library/dis.html#opcode-LOAD_FAST) as:
Pushes a reference to the local co_varnames[var_num] onto the stack.
However running dis on the following snippet shows that it may load other names:
def simple_cellvar(): # a cellvar in f
a = 1
def g(): # a freevar in g
return a
return g
import dis
dis.dis(simple_cellvar)
print()
print(f"Var names {simple_cellvar.__code__.co_varnames}")
Output
-- MAKE_CELL 1 (a)
5 RESUME 0
6 LOAD_CONST 1 (1)
STORE_DEREF 1 (a)
8 LOAD_FAST 1 (a) <---- a does not appear in the co_varnames !!!
BUILD_TUPLE 1
LOAD_CONST 2 (<code object g at 0x0000027404FB0ED0, ...)
MAKE_FUNCTION
SET_FUNCTION_ATTRIBUTE 8 (closure)
STORE_FAST 0 (g)
11 LOAD_FAST 0 (g)
RETURN_VALUE
Disassembly of <code object g at 0x0000027404FB0ED0, ...>:
-- COPY_FREE_VARS 1
8 RESUME 0
9 LOAD_DEREF 0 (a)
RETURN_VALUE
Var names ('g',)
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
Commencez par l’entrée LOAD_FAST dans la documentation liée de dis pour Python 3.13 et reproduisez le désassemblage fourni de simple_cellvar. Vérifiez comment le nom affiché est lié à co_varnames et aux variables de cellule, puis mettez à jour la documentation afin que sa description corresponde au comportement démontré.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- documentation
- Type d'issue
- Documentation
- Difficulté
- 3/5
- Temps estimé
- Une demi-journée
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 48/100