cloudwu / luareload Goto Github PK

看代码 luareload 大致明白在reload的时候做了这些事情：

1. 构建了一个沙盒， 将运行reload时已经加载的模块保护起来
2. 在沙盒中调用lua加载器来加载update模块
3. 将新加载的模块与老模块进行合并和替换

但还是有些不明白（毕竟代码里面注释也写的少）其所以然，
吴老师能讲一下这个reload工具去做lua reload 的一些思路吗？

万分感谢

应用到skynet中的时候发现，因为codecache的存在， sandbox的require第二次require同一文件的话，无法更新，所以要和skynet结合使用的话是否findloader改成返回load和load需要的参数？

1.不能正确热更的代码：

local a = 10
local data = {
    a = a,
    c = c,
}

function data.globalTest()
   return c
end

return data

如果原有模块没有定义globalTest函数，因为globalTest在沙盒中被require，其env指向的是沙盒内的global，会有错误。

原始代码如下：
489行
https://github.com/cloudwu/luareload/blob/master/reload.lua#:~:text=end-,for%20_%2C%20item%20in%20ipairs(data.objects)%20do,end,-else

2.修改方案：
假如有原有函数，可以设置原有函数的env；但是因为是新增函数，按照一般理解来看，这里的env应该可以理解成_G，根据项目约束，我们可以增加代码如下：

if type(v) == "function" then
    debug.setfenv(v, _G)
end

reload.lua:125: bad argument #1 to 'ipairs' (table expected, got nil)

但是使用skynet修改版的Lua运行就没有问题=.=

1. 实现中有数个 error 调用参数写错（145 行、162 行、176 行）
2. 对于其他 thread 从 level 为 2 的栈帧开始更新会导致 level 为 0 和 1 的栈帧没有更新到（668 行）
3. solve_global 的时候没有考虑到模块名可能包含"."的情况（524 行）
4. solve_global 函数中第 527 行和第 532 行的 break 会导致直接中断对 global_dummy 的遍历，从逻辑上来说应该是改成 goto
5. reload 一个模块，其依赖的模块也会被 reload，导致所有依赖的模块都必须遵循 reload 的限制（123 行设置的 require 函数）

例如新增
function mod.foo4()
print(a)
end
a无法和旧的a做关联。

经过排查觉得 function match_objects(objects, old_module, map, globals)内的 map[obj] = old_one有误。
我现在是改为 map[obj] = old_one or map[obj] or false，经过测试能解决这个问题。

在 reload 重新加载 mymod 后添加下面打印报错了

print("===mymod.getinfo", mymod.getinfo)

下面是报错

lua: ./reload.lua:41: attempt to index a nil value (upvalue 'dummy_module_cache')
stack traceback:
        ./reload.lua:41: in function <./reload.lua:41>
        [C]: in function 'tostring'
        [C]: in function 'print'
        test.lua:34: in main chunk
        [C]: in ?