Re: Australian timezone configure option

Bruce Momjian <pgman@candle.pha.pa.us> writes:
> Here is a new version of the patch.

It's still horridly ugly.  Why not put the three Aussie-specific entries
in a separate aussie_datetktbl array, and have the lookup look like

    if (Australian_timezones)
    {
        tp = datebsearch(lowtoken, aussie_datetktbl, sz_aussie_datetktbl);
        if (tp == NULL)
            tp = datebsearch(lowtoken, datetktbl, szdatetktbl);
    }
    else
        tp = datebsearch(lowtoken, datetktbl, szdatetktbl);

instead of modifying the lookup table on the fly.

> I found I didn't need to clear the date cache.

Hmm, are you sure about that?  I'm not.

            regards, tom lane