<HTML><BODY><div>Hi, Sergey!</div><div>LGTM, except for a commit message.</div><div>I think it would be better to rephrase it like the following:</div><div>| This patch fixes warnings found by the checkpatch.pl script.</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;">Среда, 2 августа 2023, 11:58 +03:00 от Sergey Bronnikov via Tarantool-patches <tarantool-patches@dev.tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_16909667270601427517_BODY">From: Sergey Bronnikov <<a href="/compose?To=sergeyb@tarantool.org">sergeyb@tarantool.org</a>><br><br>Warnings found by checkpatch.<br>---<br> test/tarantool-c-tests/CMakeLists.txt | 1 -<br> test/tarantool-c-tests/test.c | 2 +-<br> test/tarantool-tests/lj-356-ir-khash-non-string-obj.test.lua | 2 +-<br> 3 files changed, 2 insertions(+), 3 deletions(-)<br><br>diff --git a/test/tarantool-c-tests/CMakeLists.txt b/test/tarantool-c-tests/CMakeLists.txt<br>index 17255345..4b1d8832 100644<br>--- a/test/tarantool-c-tests/CMakeLists.txt<br>+++ b/test/tarantool-c-tests/CMakeLists.txt<br>@@ -66,4 +66,3 @@ add_custom_command(TARGET tarantool-c-tests<br> ${C_TEST_FLAGS}<br> WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}<br> )<br>-<br>diff --git a/test/tarantool-c-tests/test.c b/test/tarantool-c-tests/test.c<br>index 74cba3a3..7907c12a 100644<br>--- a/test/tarantool-c-tests/test.c<br>+++ b/test/tarantool-c-tests/test.c<br>@@ -164,7 +164,7 @@ static void test_diagnostic(void)<br> while ((ent_end = strchr(ent, '\n')) != NULL) {<br> char *next_ent = ent_end + 1;<br> /*<br>- * Limit string with with the zero byte for<br>+ * Limit string with the zero byte for<br> * formatted output. Anyway, don't need this \n<br> * anymore.<br> */<br>diff --git a/test/tarantool-tests/lj-356-ir-khash-non-string-obj.test.lua b/test/tarantool-tests/lj-356-ir-khash-non-string-obj.test.lua<br>index 5159ac32..ee0f5583 100644<br>--- a/test/tarantool-tests/lj-356-ir-khash-non-string-obj.test.lua<br>+++ b/test/tarantool-tests/lj-356-ir-khash-non-string-obj.test.lua<br>@@ -85,7 +85,7 @@ test:ok(not traceinfo(2), 'the second trace should not be compiled')<br> jit.on()<br> <br> for i = 1, N_ITERATIONS do<br>- -- Check that that all lookups are correct and there is no<br>+ -- Check that all lookups are correct and there is no<br> -- value from other cdata stored in the table.<br> test:ok(result_tab[i] == MAGIC, 'correct hash lookup from the table')<br> end<br>--<br>2.34.1</div></div></div></div></blockquote><div> </div></BODY></HTML>