LOAD_FAST access names outside of co_varnames
Aberta
Ninguém assumiu esta issue ainda.
docs
- Linguagem predominante
- Python
- Estrelas
- 77.2k
- Forks
- 36k
- Métricas de merge de PRs
- Métricas de PR pendentes
Descrição
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',)
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece pela entrada LOAD_FAST na documentação vinculada de dis do Python 3.13 e reproduza a desmontagem fornecida de simple_cellvar. Verifique como o nome exibido se relaciona com co_varnames e com as variáveis de célula e, em seguida, atualize a documentação para que sua descrição corresponda ao comportamento demonstrado.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- python
- Domínio
- documentation
- Tipo de issue
- Documentação
- Dificuldade
- 3/5
- Tempo estimado
- Meio dia
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 48/100