[Github-comments] Re: [geany/geany-plugins] GeanyLua: Use luaL_register instead of luaL_openlib (PR #1210)

2023-01-15 Thread Enrico Tröger via Github-comments
Thank you! -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1210#issuecomment-1383212166 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] GeanyLua: Use luaL_register instead of luaL_openlib (PR #1210)

2023-01-15 Thread Enrico Tröger via Github-comments
Merged #1210 into master. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1210#event-8230091512 You are receiving this because you are subscribed to this thread. Message ID:

[Github-comments] Re: [geany/geany-plugins] GeanyLua: Use luaL_register instead of luaL_openlib (PR #1210)

2023-01-15 Thread Chris Mayo via Github-comments
Oops, sorry you're right. There was a luaL_openlib***s***, converted incorrectly. PR updated to put that back: ```diff --- a/geanylua/glspi_run.c +++ b/geanylua/glspi_run.c @@ -246,7 +246,7 @@ static lua_State *glspi_state_new(void) { lua_State *L = luaL_newstate();

[Github-comments] Re: [geany/geany-plugins] GeanyLua: Use luaL_register instead of luaL_openlib (PR #1210)

2023-01-15 Thread Enrico Tröger via Github-comments
Did the changes work for you? With these changes, Geany crashes when using any of the example scripts. -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/1210#issuecomment-1383200490 You are receiving this because you are subscribed to this thread.