On 2018-Nov-17, Alvaro Herrera wrote:

> Looks good to me, save that I would change the API of getHostaddr() to
> this:
> 
> /* ----------
>  * getHostaddr -
>  * Fills 'host_addr' with the string representation of the currently connected
>  * socket in 'conn'.
>  * ----------
>  */
> static void
> getHostaddr(PGconn *conn, char *host_addr, int host_addr_len)

Fabien, are you planning to fix things per Arthur's review?

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Reply via email to