Skip to content
This repository has been archived by the owner on Jan 4, 2021. It is now read-only.

Support lua 5.3 #20

Open
daurnimator opened this issue Mar 14, 2016 · 1 comment
Open

Support lua 5.3 #20

daurnimator opened this issue Mar 14, 2016 · 1 comment

Comments

@daurnimator
Copy link

Work with lua 5.3

Major new feature is integer support, this means the lua_int64 code can be dropped.

Currently these issues are present:

plluaspi.c: In function ‘luaP_cursorfetch’:
plluaspi.c:738:34: warning: implicit declaration of function ‘luaL_optlong’ [-Wimplicit-function-declaration]
   SPI_cursor_fetch(c->cursor, 1, luaL_optlong(L, 2, FETCH_ALL));
                                  ^
pllua_errors.c: In function ‘luaB_error’:
pllua_errors.c:16:17: warning: implicit declaration of function ‘luaL_optint’ [-Wimplicit-function-declaration]
     int level = luaL_optint(L, 2, 1);
                 ^

These should be easily solved by changing to luaL_optinteger

@mbalmer
Copy link
Member

mbalmer commented Mar 21, 2016

I changed these calls to luaL_optineger, PL/Lua should now work just fine on 5.3. The int64 stuff need inspection, though.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants