[Tarantool-patches] [PATCH luajit] test: fix checking macro support in dbg tests
Sergey Kaplun
skaplun at tarantool.org
Fri Sep 18 17:48:53 MSK 2026
Hi, Mikhail!
Thanks for the clarification!
LGTM, with minor nits regarding the commit message.
On 16.09.26, Mikhail Elhimov wrote:
> Hi, Sergey!
>
> Thanks for the review!
>
> Please, see comments below.
>
> On 14.09.2026 11:38, Sergey Kaplun wrote:
> > Hi, Mikhail!
> > Thanks for the patch!
> > Honestly, I don't understand what problem it solves. Could you please
> > add a more verbose commit message?
>
> Done
>
> > On 09.09.26, Mikhail Elhimov wrote:
> follow-up 7b919c0c0cc5089dc4a61511a502a9ab46c72a2b
Minor: Please, use the following form:
| Follows-up 7b919c0c0cc5089dc4a61511a502a9ab46c72a2b ("dbg: introduce
| lj-gco command").
>
> This patch fixes the mistake that was made initially. PROCESS_RUN is a
> string and unpack operator should not be applied to it because beeing
Typo: s/string/string,/
Typo: s/unpack/the unpack/
Typo: s/beeing/being/
> applied to string it produces the sequence of the characters which is
Typo: s/string/a string,/
Typo: s/characters/characters,/
> finally transformed into the unexpected sequence of one-character
> debugger's commands that don't make any sense.
> >> ---
> >> Branch: https://github.com/tarantool/luajit/tree/elhimov/gh-noticket-test-dbg-fix-checking-macro-support
> >>
<snipped>
> --
> Best regards,
> Mikhail Elhimov
>
--
Best regards,
Sergey Kaplun
More information about the Tarantool-patches
mailing list