LuaLS / LuaLS/lua-language-server
Break in loop makes variable go out of scope early?
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Lua
- Sterne
- 4.4k
- Forks
- 442
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
How are you using the lua-language-server?
Visual Studio Code Extension (sumneko.lua)
Which OS are you using?
Linux
What is the issue affecting?
Type Checking
Expected Behaviour
I was trying to fetch a table that could be within a number of tables. Leaving early if it was found
I wanted that locally scoped variable to be the same type after the loop as before it
Actual Behaviour
Only for my variable scoped outside the loop to be unknown after it.
In an attempt to band-aid what I originally luals choking a little, I tried casting it. Only to find it was in fact unaware it even existed.
Reproduction steps
As minimal I can get the code to be and reproduce it:
---@type {[any]:A}
local T
function F()
---@class A
local a
for _, i in pairs{} do
a = T[1]
if a then break end
end
-- See it complain the `a` variable
---@cast a A
end
Additional Notes
No response
Log File
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit der minimalen Lua-Reproduktion im Issue und untersuche das Verhalten der Typprüfung rund um das lokale a, die for-Schleife und break. Als erledigt gilt die Aufgabe, wenn a nach der Schleife seinen Typ A behält und die Zeile ---@cast a A nicht mehr als unbekannt gemeldet wird; füge einen Regressionstest hinzu, der diese Reproduktion verwendet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- lua
- Bereich
- devtools
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 50/100