On 2020-May-07, Ranier Vilela wrote: > Can suggest improvements? > > 1. free (398 line) must be pg_free(buf)';
Yeah, there's a lot of frontend code that uses free() instead of pg_free(). There are too many of these that worrying about a single one would not improve things much. I guess we could convert them all, but I don't see much point. > 2. %m, is a format to parameter, right? > But what parameter? Both fatal call, do not pass this parameter, or is > it implied? %m is an implied "strerror(errno)", implemented by our snprintf replacement. -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
