lua/uloop: fd_add: use absolute indices for arguments
authorKarl Palsson <karlp@etactica.com>
Tue, 30 Jun 2020 10:38:43 +0000 (10:38 +0000)
committerPetr Štetiar <ynezz@true.cz>
Sat, 11 Jul 2020 09:15:12 +0000 (11:15 +0200)
Instead of having to adjust the index repeatedly as the stack is
manipulated, use absolute addressing for the function arguments, so they
stay the same throughout the call.  Zero functional change, just
subjectively easier to follow variables.

Signed-off-by: Karl Palsson <karlp@etactica.com>
lua/uloop.c

index 394970d74426006f81892a072be75d71cf0c3346..ee82104a087af63b51ca3e4567c44c2d5f34c967 100644 (file)
@@ -233,24 +233,23 @@ static int ul_ufd_add(lua_State *L)
        int ref;
        int fd_ref;
 
-       flags = luaL_checkinteger(L, -1);
+       flags = luaL_checkinteger(L, 3);
        if (!flags) {
                lua_pushstring(L, "flags cannot be zero");
                lua_error(L);
        }
-       lua_pop(L, 1);
 
-       luaL_checktype(L, -1, LUA_TFUNCTION);
+       luaL_checktype(L, 2, LUA_TFUNCTION);
 
-       fd = get_sock_fd(L, -2);
+       fd = get_sock_fd(L, 1);
 
        lua_getglobal(L, "__uloop_cb");
-       lua_pushvalue(L, -2);
+       lua_pushvalue(L, 2);
        ref = luaL_ref(L, -2);
        lua_pop(L, 1);
 
        lua_getglobal(L, "__uloop_fds");
-       lua_pushvalue(L, -3);
+       lua_pushvalue(L, 1);
        fd_ref = luaL_ref(L, -2);
        lua_pop(L, 1);