Skip to content

Conversation

@yireyun
Copy link

@yireyun yireyun commented Feb 26, 2018

fix hex string '0x7FFFFFFFFFFFFFFF' ToInt64() value is -0x8000000000000000, The correct value should be 9223372036854775807 (0x7FFFFFFFFFFFFFFF) .

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 90.348% when pulling 5332d69 on yireyun:master into 7d7bc87 on yuin:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 90.348% when pulling 5332d69 on yireyun:master into 7d7bc87 on yuin:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 90.348% when pulling 5332d69 on yireyun:master into 7d7bc87 on yuin:master.

@yuin
Copy link
Owner

yuin commented Mar 16, 2018

Lua 5.1(on 64bit environments) behaves almost same as GopherLua.

> print(string.format("%0d", tonumber("0x7FFFFFFFFFFFFFFF")))
-2147483648

@yuin yuin closed this Mar 16, 2018
@yireyun
Copy link
Author

yireyun commented Mar 21, 2018

Note that it is not lua function tonumber () that is fixed here, but Goalng, LState.ToInt64()

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.

4 participants