diff options
| author | Warner Losh <imp@FreeBSD.org> | 2024-02-16 03:52:41 +0000 |
|---|---|---|
| committer | Warner Losh <imp@FreeBSD.org> | 2024-02-16 03:59:22 +0000 |
| commit | 7fc95c31f007ef01c53aa5f9d8802e9579f408ee (patch) | |
| tree | ca058465fec7b1c3c1fb8ea27217792a964f62f4 | |
| parent | 33a2406eed009dbffd7dfa44285c23f0db5a3750 (diff) | |
| download | src-7fc95c31f007ef01c53aa5f9d8802e9579f408ee.tar.gz src-7fc95c31f007ef01c53aa5f9d8802e9579f408ee.zip | |
loader: Simplify the loader.has_command
luaL_checkstring already checks for the right number of
arguments. There's no need to do that by hand here. Now an exception
will be thrown like any other function with the wrong args. Also,
push a boolean instead of an int.
Sponsored by: Netflix
Differential Revision: https://reviews.freebsd.org/D43820
| -rw-r--r-- | stand/liblua/lutils.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/stand/liblua/lutils.c b/stand/liblua/lutils.c index 8066d3f685f7..182bd699dbc3 100644 --- a/stand/liblua/lutils.c +++ b/stand/liblua/lutils.c @@ -65,14 +65,15 @@ lua_has_command(lua_State *L) { const char *cmd; - if (lua_gettop(L) != 1) { - lua_pushnil(L); + cmd = luaL_checkstring(L, 1); + if (interp_has_builtin_cmd(cmd)) { + lua_pushboolean(L, 1); return 1; } - cmd = luaL_checkstring(L, 1); - lua_pushinteger(L, interp_has_builtin_cmd(cmd)); - return 1; + lua_pushnil(L); + lua_pushstring(L, "Builtin command not found"); + return 2; } static int |