From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <vdavydov.dev@gmail.com>
Date: Tue, 16 Apr 2019 20:37:18 +0300
From: Vladimir Davydov <vdavydov.dev@gmail.com>
Subject: Re: [tarantool-patches] [PATCH] fiber: Unify sizeof operator
Message-ID: <20190416173718.b3yscfqg3epmx4g4@esperanza>
References: <20190415211450.20728-1-gorcunov@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20190415211450.20728-1-gorcunov@gmail.com>
To: Cyrill Gorcunov <gorcunov@gmail.com>
Cc: tml <tarantool-patches@freelists.org>
List-ID: <tarantool-patches.dev.tarantool.org>

On Tue, Apr 16, 2019 at 12:14:50AM +0300, Cyrill Gorcunov wrote:
> We use sizeof as a function in most of the overall code,
> fix this nit.
> ---
>  src/lib/core/fiber.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/lib/core/fiber.c b/src/lib/core/fiber.c
> index 283a53d58..ce90f930c 100644
> --- a/src/lib/core/fiber.c
> +++ b/src/lib/core/fiber.c
> @@ -1363,7 +1363,7 @@ cord_costart(struct cord *cord, const char *name, fiber_func f, void *arg)
>  void
>  cord_set_name(const char *name)
>  {
> -	snprintf(cord()->name, sizeof cord()->name, "%s", name);
> +	snprintf(cord()->name, sizeof(cord()->name), "%s", name);
>  	/* Main thread's name will replace process title in ps, skip it */
>  	if (cord_is_main())
>  		return;

Trivial. Pushed to master.