Skip to content

Commit 362ef5f

Browse files
sfan5sfan5
committedMar 15, 2014
Make sure we get a stacktrace for as many lua errors as possible
·
5.11.00.4.10
1 parent d753d35 commit 362ef5f

File tree

6 files changed

+29
-29
lines changed

6 files changed

+29
-29
lines changed
 

‎src/script/common/c_content.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -654,7 +654,7 @@ ItemStack read_item(lua_State* L, int index,Server* srv)
654654
}
655655
else
656656
{
657-
throw LuaError(NULL, "Expecting itemstack, itemstring, table or nil");
657+
throw LuaError(L, "Expecting itemstack, itemstring, table or nil");
658658
}
659659
}
660660

@@ -941,7 +941,7 @@ std::vector<ItemStack> read_items(lua_State *L, int index, Server *srv)
941941
while (lua_next(L, index)) {
942942
s32 key = luaL_checkinteger(L, -2);
943943
if (key < 1) {
944-
throw LuaError(NULL, "Invalid inventory list index");
944+
throw LuaError(L, "Invalid inventory list index");
945945
}
946946
if (items.size() < (u32) key) {
947947
items.resize(key);

‎src/script/common/c_internal.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ void script_error(lua_State *L)
7171
{
7272
const char *s = lua_tostring(L, -1);
7373
std::string str(s ? s : "");
74-
throw LuaError(NULL, str);
74+
throw LuaError(L, str);
7575
}
7676

7777
// Push the list of callbacks (a lua table).

‎src/script/cpp_api/s_inventory.cpp

+3-3
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ int ScriptApiDetached::detached_inventory_AllowMove(
5454
if(lua_pcall(L, 7, 1, errorhandler))
5555
scriptError();
5656
if(!lua_isnumber(L, -1))
57-
throw LuaError(NULL, "allow_move should return a number");
57+
throw LuaError(L, "allow_move should return a number");
5858
int ret = luaL_checkinteger(L, -1);
5959
lua_pop(L, 2); // Pop integer and error handler
6060
return ret;
@@ -86,7 +86,7 @@ int ScriptApiDetached::detached_inventory_AllowPut(
8686
if(lua_pcall(L, 5, 1, errorhandler))
8787
scriptError();
8888
if(!lua_isnumber(L, -1))
89-
throw LuaError(NULL, "allow_put should return a number");
89+
throw LuaError(L, "allow_put should return a number");
9090
int ret = luaL_checkinteger(L, -1);
9191
lua_pop(L, 2); // Pop integer and error handler
9292
return ret;
@@ -118,7 +118,7 @@ int ScriptApiDetached::detached_inventory_AllowTake(
118118
if(lua_pcall(L, 5, 1, errorhandler))
119119
scriptError();
120120
if(!lua_isnumber(L, -1))
121-
throw LuaError(NULL, "allow_take should return a number");
121+
throw LuaError(L, "allow_take should return a number");
122122
int ret = luaL_checkinteger(L, -1);
123123
lua_pop(L, 2); // Pop integer and error handler
124124
return ret;

‎src/script/cpp_api/s_nodemeta.cpp

+3-3
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ int ScriptApiNodemeta::nodemeta_inventory_AllowMove(v3s16 p,
6161
scriptError();
6262
lua_remove(L, errorhandler); // Remove error handler
6363
if(!lua_isnumber(L, -1))
64-
throw LuaError(NULL, "allow_metadata_inventory_move should"
64+
throw LuaError(L, "allow_metadata_inventory_move should"
6565
" return a number, guilty node: " + nodename);
6666
int num = luaL_checkinteger(L, -1);
6767
lua_pop(L, 1); // Pop integer
@@ -100,7 +100,7 @@ int ScriptApiNodemeta::nodemeta_inventory_AllowPut(v3s16 p,
100100
scriptError();
101101
lua_remove(L, errorhandler); // Remove error handler
102102
if(!lua_isnumber(L, -1))
103-
throw LuaError(NULL, "allow_metadata_inventory_put should"
103+
throw LuaError(L, "allow_metadata_inventory_put should"
104104
" return a number, guilty node: " + nodename);
105105
int num = luaL_checkinteger(L, -1);
106106
lua_pop(L, 1); // Pop integer
@@ -139,7 +139,7 @@ int ScriptApiNodemeta::nodemeta_inventory_AllowTake(v3s16 p,
139139
scriptError();
140140
lua_remove(L, errorhandler); // Remove error handler
141141
if(!lua_isnumber(L, -1))
142-
throw LuaError(NULL, "allow_metadata_inventory_take should"
142+
throw LuaError(L, "allow_metadata_inventory_take should"
143143
" return a number, guilty node: " + nodename);
144144
int num = luaL_checkinteger(L, -1);
145145
lua_pop(L, 1); // Pop integer

‎src/script/cpp_api/s_server.cpp

+6-6
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ bool ScriptApiServer::getAuth(const std::string &playername,
3333
getAuthHandler();
3434
lua_getfield(L, -1, "get_auth");
3535
if(lua_type(L, -1) != LUA_TFUNCTION)
36-
throw LuaError(NULL, "Authentication handler missing get_auth");
36+
throw LuaError(L, "Authentication handler missing get_auth");
3737
lua_pushstring(L, playername.c_str());
3838
if(lua_pcall(L, 1, 1, errorhandler))
3939
scriptError();
@@ -48,13 +48,13 @@ bool ScriptApiServer::getAuth(const std::string &playername,
4848
std::string password;
4949
bool found = getstringfield(L, -1, "password", password);
5050
if(!found)
51-
throw LuaError(NULL, "Authentication handler didn't return password");
51+
throw LuaError(L, "Authentication handler didn't return password");
5252
if(dst_password)
5353
*dst_password = password;
5454

5555
lua_getfield(L, -1, "privileges");
5656
if(!lua_istable(L, -1))
57-
throw LuaError(NULL, "Authentication handler didn't return privilege table");
57+
throw LuaError(L, "Authentication handler didn't return privilege table");
5858
if(dst_privs)
5959
readPrivileges(-1, *dst_privs);
6060
lua_pop(L, 1);
@@ -74,7 +74,7 @@ void ScriptApiServer::getAuthHandler()
7474
}
7575
lua_remove(L, -2); // Remove minetest
7676
if(lua_type(L, -1) != LUA_TTABLE)
77-
throw LuaError(NULL, "Authentication handler table not valid");
77+
throw LuaError(L, "Authentication handler table not valid");
7878
}
7979

8080
void ScriptApiServer::readPrivileges(int index, std::set<std::string> &result)
@@ -108,7 +108,7 @@ void ScriptApiServer::createAuth(const std::string &playername,
108108
lua_getfield(L, -1, "create_auth");
109109
lua_remove(L, -2); // Remove auth handler
110110
if(lua_type(L, -1) != LUA_TFUNCTION)
111-
throw LuaError(NULL, "Authentication handler missing create_auth");
111+
throw LuaError(L, "Authentication handler missing create_auth");
112112
lua_pushstring(L, playername.c_str());
113113
lua_pushstring(L, password.c_str());
114114
if(lua_pcall(L, 2, 0, errorhandler))
@@ -128,7 +128,7 @@ bool ScriptApiServer::setPassword(const std::string &playername,
128128
lua_getfield(L, -1, "set_password");
129129
lua_remove(L, -2); // Remove auth handler
130130
if(lua_type(L, -1) != LUA_TFUNCTION)
131-
throw LuaError(NULL, "Authentication handler missing set_password");
131+
throw LuaError(L, "Authentication handler missing set_password");
132132
lua_pushstring(L, playername.c_str());
133133
lua_pushstring(L, password.c_str());
134134
if(lua_pcall(L, 2, 1, errorhandler))

‎src/script/lua_api/l_craft.cpp

+14-14
Original file line numberDiff line numberDiff line change
@@ -150,24 +150,24 @@ int ModApiCraft::l_register_craft(lua_State *L)
150150
if(type == "shaped"){
151151
std::string output = getstringfield_default(L, table, "output", "");
152152
if(output == "")
153-
throw LuaError(NULL, "Crafting definition is missing an output");
153+
throw LuaError(L, "Crafting definition is missing an output");
154154

155155
int width = 0;
156156
std::vector<std::string> recipe;
157157
lua_getfield(L, table, "recipe");
158158
if(lua_isnil(L, -1))
159-
throw LuaError(NULL, "Crafting definition is missing a recipe"
159+
throw LuaError(L, "Crafting definition is missing a recipe"
160160
" (output=\"" + output + "\")");
161161
if(!readCraftRecipeShaped(L, -1, width, recipe))
162-
throw LuaError(NULL, "Invalid crafting recipe"
162+
throw LuaError(L, "Invalid crafting recipe"
163163
" (output=\"" + output + "\")");
164164

165165
CraftReplacements replacements;
166166
lua_getfield(L, table, "replacements");
167167
if(!lua_isnil(L, -1))
168168
{
169169
if(!readCraftReplacements(L, -1, replacements))
170-
throw LuaError(NULL, "Invalid replacements"
170+
throw LuaError(L, "Invalid replacements"
171171
" (output=\"" + output + "\")");
172172
}
173173

@@ -181,25 +181,25 @@ int ModApiCraft::l_register_craft(lua_State *L)
181181
else if(type == "shapeless"){
182182
std::string output = getstringfield_default(L, table, "output", "");
183183
if(output == "")
184-
throw LuaError(NULL, "Crafting definition (shapeless)"
184+
throw LuaError(L, "Crafting definition (shapeless)"
185185
" is missing an output");
186186

187187
std::vector<std::string> recipe;
188188
lua_getfield(L, table, "recipe");
189189
if(lua_isnil(L, -1))
190-
throw LuaError(NULL, "Crafting definition (shapeless)"
190+
throw LuaError(L, "Crafting definition (shapeless)"
191191
" is missing a recipe"
192192
" (output=\"" + output + "\")");
193193
if(!readCraftRecipeShapeless(L, -1, recipe))
194-
throw LuaError(NULL, "Invalid crafting recipe"
194+
throw LuaError(L, "Invalid crafting recipe"
195195
" (output=\"" + output + "\")");
196196

197197
CraftReplacements replacements;
198198
lua_getfield(L, table, "replacements");
199199
if(!lua_isnil(L, -1))
200200
{
201201
if(!readCraftReplacements(L, -1, replacements))
202-
throw LuaError(NULL, "Invalid replacements"
202+
throw LuaError(L, "Invalid replacements"
203203
" (output=\"" + output + "\")");
204204
}
205205

@@ -224,12 +224,12 @@ int ModApiCraft::l_register_craft(lua_State *L)
224224
else if(type == "cooking"){
225225
std::string output = getstringfield_default(L, table, "output", "");
226226
if(output == "")
227-
throw LuaError(NULL, "Crafting definition (cooking)"
227+
throw LuaError(L, "Crafting definition (cooking)"
228228
" is missing an output");
229229

230230
std::string recipe = getstringfield_default(L, table, "recipe", "");
231231
if(recipe == "")
232-
throw LuaError(NULL, "Crafting definition (cooking)"
232+
throw LuaError(L, "Crafting definition (cooking)"
233233
" is missing a recipe"
234234
" (output=\"" + output + "\")");
235235

@@ -240,7 +240,7 @@ int ModApiCraft::l_register_craft(lua_State *L)
240240
if(!lua_isnil(L, -1))
241241
{
242242
if(!readCraftReplacements(L, -1, replacements))
243-
throw LuaError(NULL, "Invalid replacements"
243+
throw LuaError(L, "Invalid replacements"
244244
" (cooking output=\"" + output + "\")");
245245
}
246246

@@ -254,7 +254,7 @@ int ModApiCraft::l_register_craft(lua_State *L)
254254
else if(type == "fuel"){
255255
std::string recipe = getstringfield_default(L, table, "recipe", "");
256256
if(recipe == "")
257-
throw LuaError(NULL, "Crafting definition (fuel)"
257+
throw LuaError(L, "Crafting definition (fuel)"
258258
" is missing a recipe");
259259

260260
float burntime = getfloatfield_default(L, table, "burntime", 1.0);
@@ -264,7 +264,7 @@ int ModApiCraft::l_register_craft(lua_State *L)
264264
if(!lua_isnil(L, -1))
265265
{
266266
if(!readCraftReplacements(L, -1, replacements))
267-
throw LuaError(NULL, "Invalid replacements"
267+
throw LuaError(L, "Invalid replacements"
268268
" (fuel recipe=\"" + recipe + "\")");
269269
}
270270

@@ -274,7 +274,7 @@ int ModApiCraft::l_register_craft(lua_State *L)
274274
}
275275
else
276276
{
277-
throw LuaError(NULL, "Unknown crafting definition type: \"" + type + "\"");
277+
throw LuaError(L, "Unknown crafting definition type: \"" + type + "\"");
278278
}
279279

280280
lua_pop(L, 1);

0 commit comments

Comments
 (0)
Please sign in to comment.