lua-cjson icon indicating copy to clipboard operation
lua-cjson copied to clipboard

Published 20 hours ago • verified publisher icon openresty

Lua>=5.2 compatibility (see mpx/#22)

Open msva opened this issue 9 years ago • 8 comments

msva avatar Oct 02 '15 14:10 msva

@msva Thank you for the patch! I'll look into it shortly :)

agentzh avatar Oct 05 '15 02:10 agentzh

@cloudwu Are you cool with merging into this fork?

agentzh avatar Oct 05 '15 02:10 agentzh

@agentzh ok

cloudwu avatar Oct 05 '15 03:10 cloudwu

@cloudwu Thanks.

agentzh avatar Oct 05 '15 03:10 agentzh

should we extend travis-ci matrix to add other version of Lua ?

chipitsine avatar Apr 24 '16 11:04 chipitsine

@chipitsine As long as we already support them :)

agentzh avatar Apr 25 '16 03:04 agentzh

@agentzh Lua5.3 integer compatibility is quite neccesary. When could you merge this pr?

korialuo avatar Jan 06 '18 03:01 korialuo

Note that I merged #49 which lets it build on all modern version of Lua, but integer support was not part of that change as I wanted to keep the changes minimal. The changes here will have to be rebased on top of that.

I also added a new test suite for all versions of Lua with #50 which should make it easier to get these changes in with confidence!

leafo avatar Apr 26 '20 18:04 leafo