[tarantool-patches] Re: [PATCH 3/3] box/console: Test for nil value in args directly
Konstantin Osipov
kostja at tarantool.org
Sun Jul 28 21:46:41 MSK 2019
* Cyrill Gorcunov <gorcunov at gmail.com> [19/07/27 01:21]:
> Instead of allocating a variable for optional args
> testing we should use dot notation instead. Otherwise
> it won't work for trivial test case as
Please remove extra "instead"
>
> ```
> require('console').set_default_output('lua,block')
> require('decimal').new('1234.5678')
> ```
>
> | builtin/box/console.lua:47: expected decimal, number or string as 2 argument
>
> and program exits.
>
> Part-of #3834
> ---
> src/box/lua/console.lua | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/src/box/lua/console.lua b/src/box/lua/console.lua
> index 0c521b44a..64086cf5b 100644
> --- a/src/box/lua/console.lua
> +++ b/src/box/lua/console.lua
> @@ -41,10 +41,9 @@ output_handlers["yaml"] = function(status, opts, ...)
> end
>
> output_handlers["lua"] = function(status, opts, ...)
> - local data = ...
> --
> -- Don't print nil if there is no data
> - if data == nil then
> + if not ... then
> return ""
> end
> --
lgtm
--
Konstantin Osipov, Moscow, Russia
More information about the Tarantool-patches
mailing list