python / python/cpython

Proposal to add gc_list_for_each macro

Aberta
#133,058 2 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

interpreter-core pending type-feature
Linguagem predominante
Python
Estrelas
77.2k
Forks
36k
Métricas de merge de PRs
Métricas de PR pendentes

Descrição

Feature or enhancement

Proposal:

In the implementation of the reference cycle garbage collector, gc.c file, I have noticed that we use the following for-loop pattern over and over to traverse a PyGC_Head list:

for (gc = GC_NEXT(list); gc != list; gc = GC_NEXT(gc)) {
  /* ... */
}

However, in pycore_llist.h we avoided that by using llist_for_each macro as:

// Iterate over a list.
#define llist_for_each(node, head) \
    for (node = (head)->next; node != (head); node = node->next)

I propose adding a new macro as gc_list_for_each to the gc_list_xxx family to serve the same purpose:

#define gc_list_for_each(gc, list) \
    for (gc = GC_NEXT((list)); gc != (list); gc = GC_NEXT(gc))
Has this already been discussed elsewhere?

No response given

Links to previous discussion of this feature:

No response

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Direção de pesquisa

Comece revisando os padrões repetidos de travessia de PyGC_Head em gc.c e a macro llist_for_each existente em pycore_llist.h. Adicione a macro gc_list_for_each proposta à família gc_list_xxx e, em seguida, verifique se o coletor de lixo de ciclos de referência continua compilando e se seu comportamento existente permanece inalterado.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
c
Domínio
backend
Tipo de issue
Funcionalidade
Dificuldade
2/5
Tempo estimado
1-3 horas
Status de atividade
Estagnada
Clareza
Claramente especificada
Facilidade para iniciantes
48/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.