LuaLS / LuaLS/lua-language-server
Multiline definitions bug in @meta files
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?
Windows
What is the issue affecting?
Diagnostics/Syntax Checking
Expected Behaviour
Not throwing a warning
Actual Behaviour
Throws a warning for this piece of code: for missing return
---@meta
-- bunch of other definitions
---Create a new RotMatrix.
---Available in worker threads.
---@param x1 number
---@param y1 number
---@param z1 number
---@param x2 number
---@param y2 number
---@param z2 number
---@param x3 number
---@param y3 number
---@param z3 number
---@return RotMatrix rotMatrix The created rotation matrix.
function RotMatrix(
x1, y1, z1,
x2, y2, z2,
x3, y3, z3
) end
But not for the same code, with the RotMatrix shortened to 1 line;
function RotMatrix(x1, y1, z1, x2, y2, z2, x3, y3, z3) end
Reproduction steps
- Check code sample in editor
Additional Notes
No response
Log File
No response
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 damit, das Lua-Beispiel in der Visual Studio Code-Erweiterung zu reproduzieren und den Diagnose-/Syntaxprüfungsbericht für die mehrzeilige RotMatrix-Definition nachzuverfolgen. Vergleiche sie mit der einzeiligen Definition; fertig, wenn die mehrzeilige @meta-Definition keine Warnung über eine fehlende Rückgabe mehr meldet, während das gemeldete Verhalten korrekt bleibt.
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
- Größtenteils klar
- Anfängerfreundlichkeit
- 42/100