From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <kostja@tarantool.org>
Date: Fri, 5 Oct 2018 00:50:04 +0300
From: Konstantin Osipov <kostja@tarantool.org>
Subject: Re: [PATCH 05/13] gc: fold gc_consumer_new and gc_consumer_delete
Message-ID: <20181004215004.GE22855@chai>
References: <cover.1538671546.git.vdavydov.dev@gmail.com>
 <174ff75d6da63c095fc742a4bf857f66056e4952.1538671546.git.vdavydov.dev@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <174ff75d6da63c095fc742a4bf857f66056e4952.1538671546.git.vdavydov.dev@gmail.com>
To: Vladimir Davydov <vdavydov.dev@gmail.com>
Cc: tarantool-patches@freelists.org
List-ID: <tarantool-patches.dev.tarantool.org>

* Vladimir Davydov <vdavydov.dev@gmail.com> [18/10/05 00:11]:
> gc_consumer_new is used in one place while gc_consumer_delete is used in
> two places, but it's a one-liner. Let's fold them to make the code flow
> more straightforward.

Can't agree with you on unrestricted use of free(), it's a time
bomb, please keep gc_consumer_delete().

Otherwise OK to push.

 

-- 
Konstantin Osipov, Moscow, Russia, +7 903 626 22 32
http://tarantool.io - www.twitter.com/kostja_osipov