From: Tobias Ulmer <tu@emlix.com>
Date: Wed, 20 Dec 2017 13:30:00 +0000 (+0100)
Subject: eio: luaL_argerror never returns, but it helps gcc warnings
X-Git-Tag: e2factory-2.3.18rc1~92
X-Git-Url: https://git.e2factory.org/?a=commitdiff_plain;h=87d49ae719a7c099219901832bbdf7a05bb5d1ce;p=e2factory.git

eio: luaL_argerror never returns, but it helps gcc warnings

Signed-off-by: Tobias Ulmer <tu@emlix.com>
---

diff --git a/generic/leio.c b/generic/leio.c
index a0e22c8..b035478 100644
--- a/generic/leio.c
+++ b/generic/leio.c
@@ -425,7 +425,7 @@ eio_cloexec(lua_State *lua)
 		lua_pushstring(lua, "handle"); // key
 		lua_gettable(lua, 1);
 		if (!lua_islightuserdata(lua, -1))
-		    luaL_argerror(lua, 1, "not a eio table");
+		    return luaL_argerror(lua, 1, "not a eio table");
 		f = (FILE *)lua_topointer(lua, -1);
 	} else if (lua_isuserdata(lua, 1)) {
 		FILE **p;
@@ -443,13 +443,13 @@ eio_cloexec(lua_State *lua)
 	}
 
 	if (fd < 0) {
-		luaL_argerror(lua, 1, "fd/eio/io file required");
+		return luaL_argerror(lua, 1, "fd/eio/io file required");
 	}
 
 	if (lua_isboolean(lua, 2)) {
 		cloexec = lua_toboolean(lua, 2);
 	} else {
-		luaL_argerror(lua, 2, "boolean required");
+		return luaL_argerror(lua, 2, "boolean required");
 	}
 
 	rc = fcntl(fd, F_SETFD, cloexec ? FD_CLOEXEC : 0);