[tarantool-patches] Re: [PATCH] sql: remove unused mask
Roman Khabibov
roman.habibov at tarantool.org
Wed Aug 14 17:35:22 MSK 2019
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