

<HTML><BODY><div>Igor, I’ve corrected these messages, thank you.<br><br> <blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">Пятница, 20 марта 2020, 16:52 +03:00 от Igor Munkin <imun@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><style type="text/css"></style><div><div id="style_15847123410833055824_BODY">Sasha,<br><br>Thanks for the patch! LGTM except the one general nit: please adjust<br>test:ok messages considering those I proposed in the previous review.<br><br>New series is not needed, just update the remote branch.<br><br>On 20.03.20, Alexander V. Tikhonov wrote:<br>> Cleaned up the tests code according to the Lua style guide:<br>> - made scoped variables local<br>> - added os.exit call at the end of the test chunk<br>> - adjusted the messages in test:ok calls<br>><br>> Part of #4655<br>> ---<br>> test/fix_string_find_recording.test.lua | 6 +++---<br>> test/fold_bug_LuaJIT_505.test.lua | 7 ++++---<br>> test/fold_bug_LuaJIT_524.test.lua | 4 ++--<br>> test/gh.test.lua | 8 ++++----<br>> test/table_chain_bug_LuaJIT_494.test.lua | 23 ++++++++++++-----------<br>> test/unsink_64_kptr.test.lua | 7 ++++---<br>> 6 files changed, 29 insertions(+), 26 deletions(-)<br>><br><br><snipped><br><br>> --<br>> 2.17.1<br>><br><br>--<br>Best regards,<br>IM</div></div></div></div></blockquote> <div> </div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Alexander Tikhonov</div></div></div><div> </div></div></BODY></HTML>