

<HTML><BODY><div class="cl-50nhx219wf"><div>Sergey! Thanks for the additional updates and for the explanation!</div><div>(The patch LGTM, as it was stated earlier.)</div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Best regards,</div><div>Evgeniy Temirgaleev</div></div></div><br><div class="mail-quote-collapse"><blockquote style="border-left:1px solid #0857A6;margin:10px;padding:0 0 0 10px"><span>From: Sergey Kaplun <<a href="mailto:skaplun@tarantool.org">skaplun@tarantool.org</a>><br>To: Evgeniy Temirgaleev <<a href="mailto:e.temirgaleev@tarantool.org">e.temirgaleev@tarantool.org</a>><br>Cc: tarantool-patches@dev.tarantool.org, Sergey Bronnikov <<a href="mailto:sergeyb@tarantool.org">sergeyb@tarantool.org</a>><br>Date: Tuesday, June 30, 2026 5:04 PM +03:00</span><br> <div><div id=""><div class="cl-q19w6xya9q"><div class="js-helper_mr_css_attr js-readmsg-msg_mr_css_attr"><div id="style_17828282770896167796_mr_css_attr"><div id="style_17828282770896167796_BODY_mr_css_attr">Evgeniy,<br><br>On 30.06.26, Evgeniy Temirgaleev wrote:<br>> Sergey, thanks for the answer!<br>><br>> > I'm not sure that ctype_preplit -> dump_ctype_preplit helps for find the<br>> > original logic for this dumper from the LuaJIT source code. So, I'm open<br>> > to ideas ;).<br>><br>> I think, we can use original names from LuaJIT(ctype_prelit for ctype_prelit) and differ names for routines which are not present in LuaJIT. No more ideas at a time :)<br>><br>> I can’t find cdata_val_int64 or cdata_val_complex in LuaJIT code, ‘dump_’ prefix missed?<br><br>They are partially inspired by __tostring metamethod for FFI.<br>Added the prefix as you suggested:<br><br>===================================================================<br>diff --git a/src/luajit_dbg.py b/src/luajit_dbg.py<br>index 28cbe97d..19cefaed 100644<br>--- a/src/luajit_dbg.py<br>+++ b/src/luajit_dbg.py<br>@@ -2081,9 +2081,9 @@ def dump_lj_gco_cdata(gcobj):<br>size = ctype['size']<br>value = ''<br>if ctype_iscomplex(info):<br>- value = cdata_val_complex(cdata, ctype)<br>+ value = dump_cdata_val_complex(cdata, ctype)<br>elif size == 8 and ctype_isinteger(info):<br>- value = cdata_val_int64(cdata, ctype)<br>+ value = dump_cdata_val_int64(cdata, ctype)<br>else:<br>value = cdataptr(cdata)<br>if ctype_isptr(info):<br>@@ -2425,7 +2425,7 @@ def dump_func(func):<br># FFI dumpers.<br><br><br>-def cdata_val_int64(cdata, ctype):<br>+def dump_cdata_val_int64(cdata, ctype):<br>info = ctype['info']<br>isunsigned = info & CTF_UNSIGNED<br>cdataval = cdataptr(cdata)<br>@@ -2439,7 +2439,7 @@ def cdata_val_int64(cdata, ctype):<br>return str(valueptr[0]) + usuffix + 'LL'<br><br><br>-def cdata_val_complex(cdata, ctype):<br>+def dump_cdata_val_complex(cdata, ctype):<br>size = ctype['size']<br>cdataval = cdataptr(cdata)<br>casttype = None<br>===================================================================<br><br>Branch is force-pushed.<br><br>><br>> --<br>> Best regards,<br>> Evgeniy Temirgaleev<br>><br>> ><br>> > From: Sergey Kaplun <<a href="mailto:skaplun@tarantool.org">skaplun@tarantool.org</a>><br>> > To: Evgeniy Temirgaleev <<a href="mailto:e.temirgaleev@tarantool.org">e.temirgaleev@tarantool.org</a>><br>> > Cc: tarantool-patches@dev.tarantool.org, Sergey Bronnikov <<a href="mailto:sergeyb@tarantool.org">sergeyb@tarantool.org</a><br>> > ><br>> > Date: Tuesday, June 30, 2026 3:27 PM +03:00<br>> > Evgeniy,<br>> > Thanks for the answer, see my thoughts below.<br>> ><br>> > On 30.06.26, Evgeniy Temirgaleev wrote:<br><br><snipped><br><br>> > --<br>> > Best regards,<br>> > Sergey Kaplun<br>> ><br><br>--<br>Best regards,<br>Sergey Kaplun</div></div></div></div></div></div></blockquote></div></div></BODY></HTML>