[Tarantool-patches] [PATCH v5 35/52] sql: introduce mem_set_frame()

[Vladislav Shpilevoy]

Good job on the patch!

> diff --git a/src/box/sql/vdbe.c b/src/box/sql/vdbe.c
> index c21a6576f..7f1e0bcbe 100644
> --- a/src/box/sql/vdbe.c
> +++ b/src/box/sql/vdbe.c
> @@ -4163,8 +4163,7 @@ case OP_Program: {        /* jump */
>  			goto no_mem;
>  		}
>  		mem_destroy(pRt);

You can remove this destroy call.

> -		pRt->flags = MEM_Frame;
> -		pRt->u.pFrame = pFrame;
> +		mem_set_frame(pRt, pFrame);
>  
>  		pFrame->v = p;
>  		pFrame->nChildMem = nMem;
>