From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTP id 4B0F726726 for ; Fri, 2 Aug 2019 08:59:29 -0400 (EDT) Received: from turing.freelists.org ([127.0.0.1]) by localhost (turing.freelists.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id KZlCxgMWQWy1 for ; Fri, 2 Aug 2019 08:59:29 -0400 (EDT) Received: from smtp21.mail.ru (smtp21.mail.ru [94.100.179.250]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTPS id 0BD0026723 for ; Fri, 2 Aug 2019 08:59:28 -0400 (EDT) Date: Fri, 2 Aug 2019 15:59:26 +0300 From: Kirill Yukhin Subject: [tarantool-patches] Re: [PATCH 0/3] box/console: Cleanup and bug fix Message-ID: <20190802125926.ed5lnjopa33elhrb@tarantool.org> References: <20190726221712.11191-1-gorcunov@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20190726221712.11191-1-gorcunov@gmail.com> Sender: tarantool-patches-bounce@freelists.org Errors-to: tarantool-patches-bounce@freelists.org Reply-To: tarantool-patches@freelists.org List-Help: List-Unsubscribe: List-software: Ecartis version 1.0.0 List-Id: tarantool-patches List-Subscribe: List-Owner: List-post: List-Archive: To: tarantool-patches@freelists.org Cc: Alexander Turenko , Konstantin Osipov , Cyrill Gorcunov Hello, On 27 Jul 01:17, Cyrill Gorcunov wrote: > In first two patches simply make code smaller and in > third patch fix a real bug I catched when started to > investigate test migration to lua output. > > --- > > The following changes since commit f21a9932293bbbb5f3b6a3c6fa69a5bb3953e4b1: > > LTO build fails on warning message (2019-07-26 20:52:43 +0300) > > are available in the Git repository at: > > git@github.com:cyrillos/tarantool.git console-repl-serpent-5 > > for you to fetch changes up to dfdfb6d9a67427c39b6f8403c111c61078f68909: > > box/console: Test for nil value in args directly (2019-07-27 01:13:00 +0300) > > ---------------------------------------------------------------- > Cyrill Gorcunov (3): > box/console: Don't allow arguments in get_default_output > box/console: Drop redundant status parameter from return > box/console: Test for nil value in args directly The patchset is pretty much important from stability point of view. So, checked it into master. Cyrill, I still hope you'll prepare tests. -- Regards, Kirill Yukhin