tracedoc.lua 107行是否有问题

[wushi333333]

tracedoc.lua 107行，似乎不正确 for k in pairs(lv) do keys[k] = true end 大神，这个东西是否应该改成这样 for k in pairs(lv._lastversion) do keys[k] = true end

[cloudwu]

bug 的话，给个 testcase ？

[cloudwu]

如果你觉得你的改法有道理，那么一定能制造一个 testcase 在原版本中是错的，改后是对的。

这可以节省所有人的时间。

[wushi333333]

local tracedoc = require "app.tracedoc"

local doc = tracedoc.new { a = 1, b = { 1,2,3 }, c = { d = 4 , e = 5 }, d = {}, }

local function dump1(doc) local changes = tracedoc.commit(doc, {}) print("Dump:") for k,v in pairs(doc) do print(k,v) if type(v) == "table" then for k, v in pairs(v) do print("\t",k,v) end end end print("changes:") for k,v in pairs(changes) do print(k,v) end end

dump1(doc)

dump(doc,"doc",100) print "-----------------------------"

doc.b[3] = nil doc.b = { 1,3 } -- remove [3], change [2] dump(doc,"doc222",100)

-- 结果是 b直接被清空了 -- "doc" = { -- "_changes" = { -- "_doc" = REF -- "_keys" = { -- } -- } -- "_dirty" = false -- "_lastversion" = { -- "a" = 1 -- "b" = { -- "_changes" = { -- "_doc" = REF -- "_keys" = { -- } -- } -- "_dirty" = false -- "_lastversion" = { -- 1 = 1 -- 2 = 2 -- 3 = 3 -- } -- "_parent" = REF -- } -- "c" = { -- "_changes" = { -- "_doc" = REF -- "_keys" = { -- } -- } -- "_dirty" = false -- "_lastversion" = { -- "d" = 4 -- "e" = 5 -- } -- "_parent" = REF -- } -- "d" = { -- "_changes" = { -- "_doc" = REF -- "_keys" = { -- } -- } -- "_dirty" = false -- "_lastversion" = { -- } -- "_parent" = REF -- } -- } -- "_parent" = false -- }

---

-- "doc222" = { -- "_changes" = { -- "_doc" = REF -- "_keys" = { -- } -- } -- "_dirty" = true -- "_lastversion" = { -- "a" = 1 -- "b" = { -- } -- "c" = { -- "_changes" = { -- "_doc" = REF -- "_keys" = { -- } -- } -- "_dirty" = false -- "_lastversion" = { -- "d" = 4 -- "e" = 5 -- } -- "_parent" = REF -- } -- "d" = { -- "_changes" = { -- "_doc" = REF -- "_keys" = { -- } -- } -- "_dirty" = false -- "_lastversion" = { -- } -- "_parent" = REF -- } -- } -- "_parent" = false -- }