[Tarantool-patches] [PATCH v4 40/53] sql: move MEM flags to mem.c

[Vladislav Shpilevoy]

Thanks for the patch!

> diff --git a/src/box/sql/mem.h b/src/box/sql/mem.h
> index 70224b55a..8b6f6749d 100644
> --- a/src/box/sql/mem.h
> +++ b/src/box/sql/mem.h
> @@ -563,13 +511,6 @@ columnNullValue(void);
>  
>  int sqlVdbeMemTooBig(Mem *);
>  
> -/* Return TRUE if Mem X contains dynamically allocated content - anything
> - * that needs to be deallocated to avoid a leak.
> - */
> -#define VdbeMemDynamic(X)  \
> -  (((X)->flags&(MEM_Agg|MEM_Dyn|MEM_Frame))!=0)

Why did you remove that? And why don't you have MEM_Agg|MEM_Frame in
mem_is_dynamic()?