I found a bug in AceEvent-2.0. In the following block of code around line 348 (r20260 but introduced at r19845):
if type(method) == "string" then
local obj_method = obj[method]
if obj_method then
obj_method(obj, ...) -- old unsafe call
local success, err = pcall(obj_method, obj, ...) -- new safe call
if not success then geterrorhandler()((debugstack():match("(.-: )in.-\n") or "") .. err) end
end
the method obj_method is called twice, once 'unsafely' and once safely.
Fix remove the "obj_method(obj,...)" call and just leave the pcall.
Rollback Post to RevisionRollBack
To post a comment, please login or register a new account.
if type(method) == "string" then local obj_method = obj[method] if obj_method then obj_method(obj, ...) -- old unsafe call local success, err = pcall(obj_method, obj, ...) -- new safe call if not success then geterrorhandler()((debugstack():match("(.-: )in.-\n") or "") .. err) end endthe method obj_method is called twice, once 'unsafely' and once safely.
Fix remove the "obj_method(obj,...)" call and just leave the pcall.