[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