Fixed forgotten int → LuaError

4 files changed, 10 insertions, 10 deletions

diff --git a/src/deep.cpp b/src/deep.cpp
index 5515a62..1685ebb 100644
--- a/src/deep.cpp
+++ b/src/deep.cpp
@@ -256,11 +256,10 @@ char const* DeepFactory::PushDeepProxy(DestState L_, DeepPrelude* prelude_, int
                     lua_rawget(L_, -2);                                                            // L_: DPC proxy metatable require() "module" _R._LOADED module
                     int const alreadyloaded = lua_toboolean(L_, -1);
                     if (!alreadyloaded) { // not loaded
-                        int require_result;
                         lua_pop(L_, 2);                                                            // L_: DPC proxy metatable require() "module"
                         // require "modname"
-                        require_result = lua_pcall(L_, 1, 0, 0);                                   // L_: DPC proxy metatable error?
-                        if (require_result != LUA_OK) {
+                        LuaError const _require_result{ lua_pcall(L_, 1, 0, 0) };                  // L_: DPC proxy metatable error?
+                        if (_require_result != LuaError::OK) {
                             // failed, return the error message
                             lua_pushfstring(L_, "error while requiring '%s' identified by DeepFactory::moduleName: ", _modname.data());
                             lua_insert(L_, -2);                                                    // L_: DPC proxy metatable prefix error
diff --git a/src/lanes.cpp b/src/lanes.cpp
index aa614b7..a68c3aa 100644
--- a/src/lanes.cpp
+++ b/src/lanes.cpp
@@ -406,7 +406,8 @@ LUAG_FUNC(lane_new)
                     raise_luaL_error(L_, "cannot pre-require modules without loading 'package' library first");
                 } else {
                     lua_pushlstring(_L2, _name, _len);                                             // L_: [fixed] args... n "modname"                L2: require() name
-                    if (lua_pcall(_L2, 1, 1, 0) != LUA_OK) {                                       // L_: [fixed] args... n "modname"                L2: ret/errcode
+                    LuaError const _rc{ lua_pcall(_L2, 1, 1, 0) };                                 // L_: [fixed] args... n "modname"                L2: ret/errcode
+                    if (_rc != LuaError::OK) {
                         // propagate error to main state if any
                         InterCopyContext _c{ _U, DestState{ L_ }, SourceState{ _L2 }, {}, {}, {}, {}, {} };
                         std::ignore = _c.inter_move(1);                                            // L_: [fixed] args... n "modname" error          L2:
@@ -870,8 +871,8 @@ LANES_API int luaopen_lanes_core(lua_State* L_)
 
 [[nodiscard]] static int default_luaopen_lanes(lua_State* L_)
 {
-    int const _rc{ luaL_loadfile(L_, "lanes.lua") || lua_pcall(L_, 0, 1, 0) };
-    if (_rc != LUA_OK) {
+    LuaError const _rc{ luaL_loadfile(L_, "lanes.lua") || lua_pcall(L_, 0, 1, 0) };
+    if (_rc != LuaError::OK) {
         raise_luaL_error(L_, "failed to initialize embedded Lanes");
     }
     return 1;
diff --git a/src/linda.cpp b/src/linda.cpp
index 01b089e..76607a4 100644
--- a/src/linda.cpp
+++ b/src/linda.cpp
@@ -165,7 +165,7 @@ int Linda::ProtectedCall(lua_State* L_, lua_CFunction f_)
     lua_pushcfunction(L_, f_);
     lua_insert(L_, 1);
     // do a protected call
-    int const _rc{ lua_pcall(L_, lua_gettop(L_) - 1, LUA_MULTRET, 0) };
+    LuaError const _rc{ lua_pcall(L_, lua_gettop(L_) - 1, LUA_MULTRET, 0) };
     // whatever happens, the keeper state stack must be empty when we are done
     lua_settop(_KL, 0);
 
@@ -173,7 +173,7 @@ int Linda::ProtectedCall(lua_State* L_, lua_CFunction f_)
     _linda->releaseKeeper(_K);
 
     // if there was an error, forward it
-    if (_rc != LUA_OK) {
+    if (_rc != LuaError::OK) {
         raise_lua_error(L_);
     }
     // return whatever the actual operation provided
diff --git a/src/state.cpp b/src/state.cpp
index a6c9859..e258f9e 100644
--- a/src/state.cpp
+++ b/src/state.cpp
@@ -67,12 +67,12 @@ THE SOFTWARE.
 
     _U->requireMutex.lock();
     // starting with Lua 5.4, require may return a second optional value, so we need LUA_MULTRET
-    int _rc{ lua_pcall(L_, _args, LUA_MULTRET, 0 /*errfunc*/) };                                   // L_: err|result(s)
+    LuaError const _rc{ lua_pcall(L_, _args, LUA_MULTRET, 0 /*errfunc*/) };                        // L_: err|result(s)
     _U->requireMutex.unlock();
 
     // the required module (or an error message) is left on the stack as returned value by original require function
 
-    if (_rc != LUA_OK) { // LUA_ERRRUN / LUA_ERRMEM ?
+    if (_rc != LuaError::OK) { // LUA_ERRRUN / LUA_ERRMEM ?
         raise_lua_error(L_);
     }
     // should be 1 for Lua <= 5.3, 1 or 2 starting with Lua 5.4