[Tarantool-patches] [PATCH 2/4] sql: refactor sqlVdbeMemNumerify()
Vladislav Shpilevoy
v.shpilevoy at tarantool.org
Tue Feb 11 02:25:01 MSK 2020
Hi! Thanks for the patchset!
On 05/02/2020 17:19, Nikita Pettik wrote:
> Fix codestyle and comment; allow conversion from boolean to number
> (since it is legal to convert boolean to integer, and in turn number
> type completely includes integer type).
>
> Part of #4233
> ---
> src/box/sql/vdbeInt.h | 12 +++++++++++-
> src/box/sql/vdbemem.c | 46 +++++++++++++++++++---------------------------
> 2 files changed, 30 insertions(+), 28 deletions(-)
>
> diff --git a/src/box/sql/vdbeInt.h b/src/box/sql/vdbeInt.h
> index 1393f3fd2..a80a691e1 100644
> --- a/src/box/sql/vdbeInt.h
> +++ b/src/box/sql/vdbeInt.h
> @@ -533,7 +533,17 @@ mem_value_bool(const struct Mem *mem, bool *b);
>
> int mem_apply_integer_type(Mem *);
> int sqlVdbeMemRealify(Mem *);
> -int sqlVdbeMemNumerify(Mem *);
> +
> +/**
> + * Convert @mem to NUMBER type, so that after conversion it has one
'@' is a command prefix in Doxygen. The word after '@' is considered
a command. I suppose you didn't mean command 'mem', but rather wanted
to mention a parameter called 'mem'. In that case I suggest '@a mem'.
We use '@a' usually, except some old files, which still can use '\a'
or something like this.
More information about the Tarantool-patches
mailing list