Bruce Momjian writes:
In mbprint.c:
+ #ifndef PGSCRIPTS
+ #include "settings.h"
+ #endif
Why is this needed, considering that the rest of the code was not changed
to make use of any setting? The print modules in psql should not make use
of psql-specific settings. That's why there are extensive separate
structures to pass this information.
Also, I'm beginning to feel that there are too many places where the lack
of getpwuid() is being worked around. That needs to be centralized.
Lastly, since there is not going to be a Windows port in 7.4, why was this
committed to the head branch?
--
Peter Eisentraut peter_e@gmx.net