Re: Set arbitrary GUC options during initdb

Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com> writes:
> This patch looks good to me. It's a very nice simplification of the
> initdb.c code, even without the new feature.

Thanks for looking!

> I found that the addition of
> #include <ctype.h>
> didn't appear to be necessary. Maybe it was required before
> guc_value_requires_quotes() was changed?

There's still an isspace() added by the patch ... oh, the #include
is not needed because port.h includes ctype.h. That's spectacularly
awful from an include-footprint standpoint, but I can't say that
I want to go fix it right this minute.

> I would remove the
> #if DEF_PGPORT != 5432
> This was in the previous code too, but now if we remove it, then we
> don't have any more hardcoded 5432 left, which seems like a nice
> improvement in cleanliness.

Hm. That'll waste a few cycles during initdb; not sure if the extra
cleanliness is worth it. It's not like that number is going to change.

regards, tom lane