CLEANUP: lua: remove test
authorThierry FOURNIER <thierry.fournier@ozon.io>
Wed, 26 Apr 2017 11:27:05 +0000 (13:27 +0200)
committerWilly Tarreau <w@1wt.eu>
Wed, 26 Apr 2017 13:13:18 +0000 (15:13 +0200)
The man of "luaL_unref" says "If ref is LUA_NOREF or LUA_REFNIL,
luaL_unref does nothing.", so I remove the check.

src/hlua.c

index 6ce18e1..643d3fc 100644 (file)
@@ -3309,8 +3309,7 @@ __LJMP static int hlua_applet_tcp_set_priv(lua_State *L)
        MAY_LJMP(check_args(L, 2, "set_priv"));
 
        /* Remove previous value. */
-       if (hlua->Mref != -1)
-               luaL_unref(L, LUA_REGISTRYINDEX, hlua->Mref);
+       luaL_unref(L, LUA_REGISTRYINDEX, hlua->Mref);
 
        /* Get and store new value. */
        lua_pushvalue(L, 2); /* Copy the element 2 at the top of the stack. */
@@ -3772,8 +3771,7 @@ __LJMP static int hlua_applet_http_set_priv(lua_State *L)
        MAY_LJMP(check_args(L, 2, "set_priv"));
 
        /* Remove previous value. */
-       if (hlua->Mref != -1)
-               luaL_unref(L, LUA_REGISTRYINDEX, hlua->Mref);
+       luaL_unref(L, LUA_REGISTRYINDEX, hlua->Mref);
 
        /* Get and store new value. */
        lua_pushvalue(L, 2); /* Copy the element 2 at the top of the stack. */
@@ -4859,8 +4857,7 @@ __LJMP static int hlua_set_priv(lua_State *L)
        hlua = hlua_gethlua(L);
 
        /* Remove previous value. */
-       if (hlua->Mref != -1)
-               luaL_unref(L, LUA_REGISTRYINDEX, hlua->Mref);
+       luaL_unref(L, LUA_REGISTRYINDEX, hlua->Mref);
 
        /* Get and store new value. */
        lua_pushvalue(L, 2); /* Copy the element 2 at the top of the stack. */