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 (
To make changes to your subscription:

Reply via email to