forked from yuin/gopher-lua
-
Notifications
You must be signed in to change notification settings - Fork 0
goto statement #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
metafates
wants to merge
44
commits into
awirix:master
Choose a base branch
from
yuin:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Write trunc
Fix lua coroutine context leak
Convert string math params to number instead of failing with error
This reverts commit e61dab4.
In the C implementation of the Lua interpreter,
in the io.popen function the second argument can be nil:
Lua 5.1.5 Copyright (C) 1994-2012 Lua.org, PUC-Rio:
> x,y = io.popen("ls", nil)
> assert(x)
> assert(y == nil)
>
Gopher lua throws an exception:
bad argument #2 to popen (string expected, got nil)
Closes #459
popen: support nil in second argument
Fix xpcall error in error handler
regstry.Set* were very simple functions at the beginning of the project, but recently not. It seems inlining these functions make improvements better.
GopherLua is a relatively old project. Old Go runtime had slow interface type assertions, but now Go runtime better its performance.
…uaperiphery library
Re-add link to relocated gluasocket library
$ lua
Lua 5.1.5 Copyright (C) 1994-2012 Lua.org, PUC-Rio
> file = "/tmp"
> fd, _, code = io.open(file, "r")
> _, _, ecode = fd:read(1)
> print(ecode)
21
>
gopher-lua throws an exception:
read /tmp: is a directory
stack traceback:
[G]: in function 'read'
extra/wrapper.lua:17: in function 'exec'
<string>:1: in main chunk
[G]: ?
This patch results in behavior similar to the vanilla lua
implementation.
Closes #455
iolib: fix different behavior in read function
https://github.com/BixData/gluabit32 is a 404 now, so implement a fresh bit32 for gopher-lua
fix README format error
fix compiler producing incorrect LOADNIL byte code
Remove io/ioutil usages (deprecated since Go 1.16)
fix: return value error for CompileLua example
docs: update README.rst
Add github.com/PeerDB-io/gluabit32 to README.rst
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.