From 19a1ac1f34c8dba8ada039c916858e16e2637f36 Mon Sep 17 00:00:00 2001 From: Perttu Ahola Date: Mon, 28 Nov 2011 01:13:55 +0200 Subject: [PATCH] For consistency, implement calling of on_chat_message callbacks in C --- data/builtin.lua | 10 ---------- src/scriptapi.cpp | 32 +++++++++++++++++++++----------- 2 files changed, 21 insertions(+), 21 deletions(-) diff --git a/data/builtin.lua b/data/builtin.lua index a3276192..b14ffa3b 100644 --- a/data/builtin.lua +++ b/data/builtin.lua @@ -166,14 +166,4 @@ minetest.registered_on_generateds, minetest.register_on_generated = make_registr minetest.registered_on_newplayers, minetest.register_on_newplayer = make_registration() minetest.registered_on_respawnplayers, minetest.register_on_respawnplayer = make_registration() -minetest.on_chat_message = function(name, message) - for i,func in ipairs(minetest.registered_on_chat_messages) do - local ate = func(name, message) - if ate then - return true - end - end - return false -end - -- END diff --git a/src/scriptapi.cpp b/src/scriptapi.cpp index fb1b58f5..a9112254 100644 --- a/src/scriptapi.cpp +++ b/src/scriptapi.cpp @@ -1548,18 +1548,28 @@ bool scriptapi_on_chat_message(lua_State *L, const std::string &name, assert(lua_checkstack(L, 20)); StackUnroller stack_unroller(L); - // Get minetest.on_chat_message builtin function + // Get minetest.registered_on_chat_messages lua_getglobal(L, "minetest"); - lua_getfield(L, -1, "on_chat_message"); - luaL_checktype(L, -1, LUA_TFUNCTION); - - // Call function - lua_pushstring(L, name.c_str()); - lua_pushstring(L, message.c_str()); - if(lua_pcall(L, 2, 1, 0)) - script_error(L, "error: %s\n", lua_tostring(L, -1)); - bool ate = lua_toboolean(L, -1); - return ate; + lua_getfield(L, -1, "registered_on_chat_messages"); + luaL_checktype(L, -1, LUA_TTABLE); + int table = lua_gettop(L); + // Foreach + lua_pushnil(L); + while(lua_next(L, table) != 0){ + // key at index -2 and value at index -1 + luaL_checktype(L, -1, LUA_TFUNCTION); + // Call function + lua_pushstring(L, name.c_str()); + lua_pushstring(L, message.c_str()); + if(lua_pcall(L, 2, 1, 0)) + script_error(L, "error: %s\n", lua_tostring(L, -1)); + bool ate = lua_toboolean(L, -1); + lua_pop(L, 1); + if(ate) + return true; + // value removed, keep key for next iteration + } + return false; } /*