[tarantool-patches] Re: [PATCH 4/4] sql: move SQL statistics to server

[n.pettik]

> No, I checked that. The code below is compiled ok. So you can remove the cast and
> z variable.
> 
> diff --git a/src/box/sql/analyze.c b/src/box/sql/analyze.c
> index 4f21cd25f..061fcc7ab 100644
> --- a/src/box/sql/analyze.c
> +++ b/src/box/sql/analyze.c
> @@ -1230,7 +1230,7 @@ struct analysis_index_info {
> static void
> decode_stat_string(const char *stat_string, int stat_size, tRowcnt *stat_exact,
>                   LogEst *stat_log) {
> -       char *z = (char *) stat_string;
> +       const char *z = stat_string;
>        if (z == NULL)
>                z = "";
>        for (int i = 0; *z && i < stat_size; i++) {

Ok, I misunderstood you. I thought you mean kind of:
char *z = stat_string;

Fixed this point as well.