[Tarantool-patches] [PATCH v2 3/3] box: replace port_tuple with port_c everywhere
Igor Munkin
imun at tarantool.org
Mon Apr 27 12:12:13 MSK 2020
Vlad,
Thanks for the fixes.
On 26.04.20, Vladislav Shpilevoy wrote:
> Hi! Thanks for the review!
>
> On 25/04/2020 02:21, Igor Munkin wrote:
> > Vlad,
> >
> > Thanks for the patch! Please consider the comments I left below.
> >
> > On 23.04.20, Vladislav Shpilevoy wrote:
<snipped>
> >> diff --git a/src/box/port.c b/src/box/port.c
> >> index 2c1fadb5c..9d9fc1dbc 100644
> >> --- a/src/box/port.c
> >> +++ b/src/box/port.c
> >> @@ -38,106 +38,15 @@
> >> + * result data when it fits into an object from the pool.
> >> */
> >> static struct mempool port_entry_pool;
> >>
> >> enum {
> >> - PORT_ENTRY_SIZE = MAX(sizeof(struct port_c_entry),
> >> - sizeof(struct port_tuple_entry)),
> >> + PORT_ENTRY_SIZE = sizeof(struct port_c_entry),
> >
> > Minor: PORT_ENTRY_SIZE is introduced in the first patch of the series.
> > After applying these changes it looks excess. Feel free to ignore if you
> > see any rationale to leave this constant.
>
> If I change it, I need to change its usage place too, and increase
> the diff. If we want to make the diff even smaller and remove this
> constant, it would be better to merge this commit and the first one.
> I can do that, if you want.
I see just little occurences and all:
| $ grep -rnF 'PORT_ENTRY_SIZE' src | cut -f 1 -d ':' | sort | uniq -c
| 5 src/box/port.c
But, this comment definitely doesn't deserve merging indepentent
commits. Let's leave this as is or until Nikita asks to fix it in
another way.
>
<snipped>
--
Best regards,
IM
More information about the Tarantool-patches
mailing list