From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 11 Feb 2019 18:49:15 +0300 From: Vladimir Davydov Subject: Re: [PATCH v1 2/4] box: rework memtx_tree class to be reusable Message-ID: <20190211154915.fhqiztrxlvs4xuto@esperanza> References: <3dde48d41f6cc51483837242af8d6d4873526445.1549367041.git.kshcherbatov@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <3dde48d41f6cc51483837242af8d6d4873526445.1549367041.git.kshcherbatov@tarantool.org> To: Kirill Shcherbatov Cc: tarantool-patches@freelists.org List-ID: On Tue, Feb 05, 2019 at 02:58:37PM +0300, Kirill Shcherbatov wrote: > The memtx tree class has been redesigned so that it can be used > to store arbitrary structures in the future. This makes possible > to implement type hints in memtx in subsequent patches. > > Needed for #3961 > --- > src/box/memtx_tree.c | 715 ++-------------------------- > src/box/memtx_tree.h | 68 +-- > src/box/memtx_tree_impl.h | 960 ++++++++++++++++++++++++++++++++++++++ > 3 files changed, 1008 insertions(+), 735 deletions(-) > create mode 100644 src/box/memtx_tree_impl.h You move a huge chunk of code to another file and modify it at the same time, which makes the patch impossible to review. Please split it in two. First, do the necessary modifications in memtx_tree.c (it's OK to include *.c files in C), then rename memtx_tree.c to memtx_tree_impl.h.