

<HTML><BODY><div>Hi, Igor!</div><div>Thanks for the fixes!</div><div>LGTM</div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Best regards,</div><div>Maxim Kokryashkin</div></div></div><div> </div><div> </div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">Понедельник, 27 февраля 2023, 12:18 +03:00 от Igor Munkin <imun@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_16774894942140151709_BODY">Sergey,<br><br>Thanks for your review!<br><br>On 13.02.23, Sergey Kaplun wrote:<br>> Hi, Igor!<br>> Thanks for the patch!<br>> LGTM, except a single nit regarding the commit message.<br>><br>> On 13.02.23, Igor Munkin wrote:<br>> > struct GCtrace is defined only if LJ_HASJIT is set. Hence all spots<br>><br>> Typo: s/Hence,/Hence/<br><br>Fixed, force-pushed.<br><br>><br>> > where GCtrace is used should be also moved under LJ_HASJIT define.<br>> ><br>> > Relates to tarantool/tarantool#8252<br>><br>> Side note: I see "Relates" and "Related" in our commit logs, so I<br>> suggest to use "Relates" for future commits.<br><br>I always use "Relates" (except maybe some early commits).<br><br>><br>> ><br>> > Signed-off-by: Igor Munkin <<a href="/compose?To=imun@tarantool.org">imun@tarantool.org</a>><br>><br>> <snipped><br>><br>> > --<br>> > 2.30.2<br>> ><br>><br>> --<br>> Best regards,<br>> Sergey Kaplun<br><br>--<br>Best regards,<br>IM</div></div></div></div></blockquote><div> </div></BODY></HTML>