Euler Taveira de Oliveira <euler@timbira.com> writes:
> + /*
> +  * Return the LC_TIME information
> +  */
> + char *
> + pg_get_lc_time(void)
> + {
> +     return locale_time;
> + }
locale_time is a global GUC variable, so there is surely no point in the
above function.  I have not looked at the rest of the patch.
        regards, tom lane