Bruce Momjian <pgman@candle.pha.pa.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