LuaLS / LuaLS/lua-language-server

Custom __pairs expects only 1 return value

Offen
#2,512 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

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 expected no warnings

Actual Behaviour

There is a redundant return value warning

Reproduction steps

Here's an example where string paths are transformed to their package.loaded value

local imports = {
    mod1 = "src.mod1",
    mod2 = "src.mod2"
}

return setmetatable({}, {
    __pairs = function(_)
        local iter = pairs(imports)

        local newiter = function (table, index) 
            local k, path = iter(table, index)

            return k, package.loaded[path]
        end

        --- LuaLS is wrong here
        ---@diagnostic disable-next-line:redundant-return-value
        return newiter, imports, nil
    end
})
Additional Notes

https://www.lua.org/pil/7.3.html

Log File

No response

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Reproduziere die Warnung in der LuaLS Visual Studio Code extension unter Linux mit dem bereitgestellten Beispiel für setmetatable und __pairs. Beginne mit der Behandlung der Typprüfung für __pairs und vergleiche sie mit der im Issue verlinkten Lua 7.3-Referenz. Als erledigt gilt die Aufgabe, wenn das Beispiel keine Warnung über einen redundanten Rückgabewert mehr meldet.

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
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.