teal-language/tl

internals: remove duplicate entries from interface_list

Fechada

#1.018 aberto em 17 de jul. de 2025

Ver no GitHub
 (2 comentários) (0 reação) (0 responsável)Lua (156 forks)batch import
good first issue

Métricas do repositório

Stars
 (2.808 estrelas)
Métricas de merge de PR
 (Mesclagem média 1d 6h) (1 fundiu PR em 30d)

Description

Reported by @upedd :

One quick question about Teal, when dealing with complex interface hierarchies, sometimes there are duplicate entries in the interface_list of a record type. Is this the expected behavior? See example:

local interface A 
end

local interface B is A
end

local interface C is A
end

-- interface_list of D is now {B, A, C, A} 
local record D is B, C
end

My reply:

it should not hurt in practice, but we should clean it up. The order does matter since that is how conflicts are handled, but basically we should be able to ignore duplicate entries, so {B, A, C} would be the correct list in that case. I'll file an issue to remind myself to clean it up.

Guia do colaborador

Direção de pesquisa
O problema é sobre remover entradas duplicadas da interface list de tipos de registro. A correção provavelmente está no código que constrói a interface list durante a verificação de tipos ou o processamento da definição do registro. Procure onde as interfaces são adicionadas (por exemplo, ao processar cláusulas 'is' em declarações de tipo de registro). Basta iterar sobre a lista e remover duplicatas preservando a ordem. O mantenedor indicou que duplicatas devem ser ignoradas, então uma simples deduplicação deve funcionar. Verifique a lógica de resolução de tipos ou construção de registros relevante no código fonte do compilador Teal.
Pilha de tecnologia
lua
Domain
backend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Lua
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75