Re: [PATCHES] to_date() validation

On Mon, Sep 8, 2008 at 6:24 PM, Brendan Jurd <direvus(at)gmail(dot)com> wrote:
> On Sun, Sep 7, 2008 at 5:58 AM, Alex Hunsaker <badalex(at)gmail(dot)com> wrote:
>> Code review: one minor nit
>> *** a/src/backend/utils/adt/formatting.c
>> --- b/src/backend/utils/adt/formatting.c
>> ***************
>> *** 781,787 **** static const KeyWord DCH_keywords[] = {
>> {"y", 1, DCH_Y, TRUE, FROM_CHAR_DATE_GREGORIAN},
>>
>> /* last */
>> ! {NULL, 0, 0, 0}
>> };
>>
>> /* ----------
>> --- 781,787 ----
>> {"y", 1, DCH_Y, TRUE, FROM_CHAR_DATE_GREGORIAN},
>>
>> /* last */
>> ! {NULL, 0, 0, 0, 0}
>> };
>
> Ah, thank you for picking that up. I will correct this and send in a
> new patch version in the next 24 hours.
>

New version attached (and link added to wiki).

Cheers,
BJ