[tarantool-patches] Re: [PATCH] sql: remove unused mask
Vladislav Shpilevoy
v.shpilevoy at tarantool.org
Wed Aug 14 23:48:57 MSK 2019
LGTM.
On 14/08/2019 16:35, Roman Khabibov wrote:
> Vlad, could you do a second review?
>
>> On Aug 9, 2019, at 18:46, n.pettik <korablev at tarantool.org> wrote:
>>
>>
>>
>>> On 2 Aug 2019, at 15:46, Roman Khabibov <roman.habibov at tarantool.org> wrote:
>>>
>>> Field mask of "struct Keyword" is always set to one not null
>>> macroses and used only once for checking that it isn't null. So,
>>> the field should be removed.
>>>
>>> Closes #4155
>>
>> I’ve fixed commit message and force pushed.
>>
>> sql: remove mask from struct Keyword
>>
>> Originally, mask in struct Keyword served to reduce set of reserved
>> keywords for build-dependent features. For instance, it was allowed to
>> disable triggers as a compilation option, and in this case TRIGGER
>> wouldn't be reserved word. Nowadays, our build always comprises all
>> features, so there's no need in this option anymore. Hence, we can
>> remove mask alongside with related to it macros.
>>
>> Closes #4155
>>
>> The rest is OK as obvious.
More information about the Tarantool-patches
mailing list