[tarantool-patches] Re: [PATCH] add FindICONV and iconv wrapper

[Georgy Kirichenko]

Mostly good, but see on my comments bellow

On Friday, July 20, 2018 7:50:52 PM MSK Konstantin Belyavskiy wrote:
> Fixing build under FreeBSD:
> +# - Find the readline include files and libraries
> +# - Include finding of termcap or curses
Wrong comment

> +iconv_wrap_open
> +iconv_wrap_close
> +iconv_wrap
I would be like if you renamed this using pattern tnt_<name>
>  exception_get_string
>  exception_get_int
> 
>       lua/errno.c
> +     lua/iconv_wrap.c
I would be like if you used name like tnt_iconv.c
>       lua/socket.c
>       lua/pickle.c
>       lua/fio.c

> diff --git a/src/lua/iconv_wrap.h b/src/lua/iconv_wrap.h
> new file mode 100644
> index 000000000..03aee40e4
> --- /dev/null
> +++ b/src/lua/iconv_wrap.h
I am do not really sure we need this file
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20180723/daade095/attachment.sig>