Skip to content

Update string.dump so it doesn't have strip argument in Lua 5.1 and 5.2#2225

Merged
sumneko merged 1 commit intoLuaLS:masterfrom
C3pa:update-string-annotations
Jul 27, 2023
Merged

Update string.dump so it doesn't have strip argument in Lua 5.1 and 5.2#2225
sumneko merged 1 commit intoLuaLS:masterfrom
C3pa:update-string-annotations

Conversation

@C3pa
Copy link
Contributor

@C3pa C3pa commented Jul 26, 2023

Fixes #2224. I hope I figured out the syntax used in these templates correctly.

Note:

  • LuaJIT also includes the strip argument for string.dump.

@sumneko
Copy link
Collaborator

sumneko commented Jul 27, 2023

Thank you!

@sumneko sumneko merged commit a9adf80 into LuaLS:master Jul 27, 2023
@C3pa C3pa deleted the update-string-annotations branch January 10, 2024 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lua 5.1 and 5.2 shouldn't have second argument in string.dump annotations

2 participants