Community
Participate
Working Groups
Created attachment 224919 [details] Patch for debugger.lua 0.9 RC2 Integrating debugger.lua into the project I'm currently working on required some fixes & changes. I've attached a patch to apply those. Here is a short description with the line in the original file. 1188 Don't invoke metamethods of debuggee metatables 1648 Local variable fix 1831 Avoid _G metamethods 2505 Allow start without active debugger session 2603 Avoid _G metamethods 2647 Allow (quick) start without active debugger session The debugged scripting environment is "locked down" thus __index or __newindex on any table triggers an error(). That's why I added the metamethod avoidance. The change in line 1188 might be a bit controversial though, as this keeps the "length" feature from working correctly if the metatable of a table is a pure proxy table. IMHO a very rare case. The connect-loop in line 2647 has been removed, which might break some existing use case. However in my case it's preferable to not wait for X seconds on startup. Maybe there is a better way of getting both to work, e.g. a delayed debuggee start? "Start without active debugger" is already covered in Bug 379965, but not fully, as the change to the Init function isn't mentioned there.
Hi thx for the patch. Somes remarks, after discution with Julien Desgats (the main debugger developper): 1188: We are dicussing about that. 1648: wi will integrate it. 1831,2603 : we will integrate it. 2505 : we will integrate it. 2647 : we will think about that too.
1648,1831,2603,2505,2647 : integrated in commit ba70f77ce90c5b934b1e24f02a3e589ec63b9ac4 and c9e2fafc4538541ec5c7d3dc5acfe1412e8c725c
Comment on attachment 224919 [details] Patch for debugger.lua 0.9 RC2 flagging iplog+ since latest fixes integrated are inspired by/taken from this patch
I close this bug. Christian if you think the metamethods problems is important, open another bug to start a discution on it. Thx.
