The function static char * path_encode(bool closed, int npts, Point *pt)
is actually occasionally passed -1 as its first parameter to mean something like "neither" or "don't print any delimiters". That doesn't seem like good style. Maybe this should be split off into another bool parameter named delimited or something? Or the argument could be made into an integer. -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers