teal-language/tl

internals: remove duplicate entries from interface_list

Fermée

#1 018 ouverte le 17 juil. 2025

Voir sur GitHub
 (2 commentaires) (0 réaction) (0 personne assignée)Lua (156 forks)batch import
good first issue

Métriques du dépôt

Stars
 (2 808 étoiles)
Métriques de merge PR
 (Merge moyen 1j 6h) (1 PR mergée en 30 j)

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.

Guide contributeur

Direction de recherche
Le problème concerne la suppression des entrées en double de l'interface list des types d'enregistrement. La correction se trouve probablement dans le code qui construit l'interface list pendant la vérification des types ou le traitement de la définition d'enregistrement. Cherchez où les interfaces sont ajoutées (par exemple, lors du traitement des clauses 'is' dans les déclarations de type d'enregistrement). Il suffit de parcourir la liste et de supprimer les doublons tout en préservant l'ordre. Le mainteneur a indiqué que les doublons doivent être ignorés, donc une simple déduplication devrait fonctionner. Vérifiez la logique de résolution de types ou de construction d'enregistrement pertinente dans le code source du compilateur Teal.
Stack technique
lua
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Lua
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75