Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> I have two questions. First, setlocale() seemed to be inconsistently
> set inside and outside of ENABLE_NLS. I assume the proper location is
> inside.
Please do *not* go adding setlocale calls that were not there before.
You *will* break things.
> ! setlocale(LC_ALL, "");
^^^^^^^^^^^^^^^^^^^^^
Putting this call in the backend is a very serious mistake. It might be
okay in clients, but not in the backend.
regards, tom lane