<HTML><BODY><div>Hi, Sergey!</div><div>I’ve fixed the nit you mentioned, the branch is force pushed.</div><div> </div><div data-signature-widget="container"><div spellcheck="false" 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;">Четверг, 29 сентября 2022, 16:11 +03:00 от Sergey Kaplun <skaplun@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_16644570960766257378_BODY">Hi, Maxim!<br><br>Thanks for the fixes!<br>LGTM, just a single nit below.<br><br>On 29.09.22, Maxim Kokryashkin via Tarantool-patches wrote:<br>><br>> Hi, Sergos!<br>> Thanks for the questions!<br>> Please consider my answers amd changes below.<br><br><snipped><br><br>> Here is the diff:<br>> ===============================================<br>> --- a/test/tarantool-tests/gh-6976-narrowing-of-unary-minus.test.lua<br>> +++ b/test/tarantool-tests/gh-6976-narrowing-of-unary-minus.test.lua<br><br><snipped><br><br>> +test:ok(check(function()<br>> + -- See the comment about `table.new()` above.<br>> + local res = require('table.new')(3, 0)<br>> + for i = 2, 0, -1 do<br>> + table.insert(res, 1 / -i)<br>> + end<br>> + return res<br>> +end),'assertion guard fail')<br><br>Typo: s/,/, /<br><br>> <br><br><snipped><br><br>> Best regards,<br>> Maxim Kokryashkin<br>> <br><br>--<br>Best regards,<br>Sergey Kaplun</div></div></div></div></blockquote><div> </div></BODY></HTML>