teal-language/tl

internals: remove duplicate entries from interface_list

Geschlossen

#1.018 geöffnet am 17.07.2025

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Lua (160 Forks)batch import
good first issue

Repository-Metriken

Stars
 (2.810 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 1T 6h) (1 gemergte PR in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Das Problem betrifft das Entfernen doppelter Einträge aus der interface list von Record Typen. Die Lösung liegt wahrscheinlich im Code, der die interface list während der Typüberprüfung oder der Record Definitionsverarbeitung aufbaut. Suchen Sie nach Stellen, an denen Schnittstellen hinzugefügt werden (z. B. bei der Verarbeitung von 'is' Klauseln in Record Typdeklarationen). Iterieren Sie einfach über die Liste und entfernen Sie Duplikate unter Beibehaltung der Reihenfolge. Der Maintainer hat angegeben, dass Duplikate ignoriert werden sollten, daher sollte eine einfache Deduplizierung funktionieren. Überprüfen Sie die relevante Typauflösungs oder Record Konstruktionslogik im Teal Compiler Quellcode.
Tech Stack
lua
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Lua
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75