From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp43.i.mail.ru (smtp43.i.mail.ru [94.100.177.103]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 85396469710 for ; Mon, 8 Jun 2020 18:31:09 +0300 (MSK) From: Oleg Babin References: <20200608104450.157253-1-gorcunov@gmail.com> <20200608104450.157253-3-gorcunov@gmail.com> Message-ID: Date: Mon, 8 Jun 2020 18:31:06 +0300 MIME-Version: 1.0 In-Reply-To: <20200608104450.157253-3-gorcunov@gmail.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-GB Content-Transfer-Encoding: 7bit Subject: Re: [Tarantool-patches] [PATCH 2/3] lua/cfg: drop redundant variable List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Cyrill Gorcunov , tml Thanks for your patch. LGTM. On 08/06/2020 13:44, Cyrill Gorcunov wrote: > No need for variable v, we use module_cfg itself. > > Part-of #689 > > Signed-off-by: Cyrill Gorcunov > --- > src/box/lua/load_cfg.lua | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/box/lua/load_cfg.lua b/src/box/lua/load_cfg.lua > index 7a77b93f5..7dc40a47f 100644 > --- a/src/box/lua/load_cfg.lua > +++ b/src/box/lua/load_cfg.lua > @@ -494,7 +494,7 @@ local function apply_default_cfg(cfg, default_cfg, module_cfg) > apply_default_cfg(cfg[k], v) > end > end > - for k, v in pairs(module_cfg) do > + for k in pairs(module_cfg) do > if cfg[k] == nil then > cfg[k] = module_cfg[k].cfg_get(k) > end >