Skip site navigation (1) Skip section navigation (2)

Re: [PATCHES] to_date() validation

From: "Brendan Jurd" <direvus(at)gmail(dot)com>
To: "Alex Hunsaker" <badalex(at)gmail(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCHES] to_date() validation
Date: 2008-09-08 18:07:05
Message-ID: 37ed240d0809081107o63e91b14h34ae40c7bd1288d2@mail.gmail.com (view raw or flat)
Thread:
Lists: pgsql-hackerspgsql-patches
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

Attachment: to-date-validation-2.diff.gz
Description: application/x-gzip (9.4 KB)

In response to

pgsql-hackers by date

Next:From: Tom LaneDate: 2008-09-08 18:18:16
Subject: Re: [HACKERS] Infrastructure changes for recovery
Previous:From: Simon RiggsDate: 2008-09-08 18:01:50
Subject: Re: [HACKERS] Infrastructure changes for recovery

pgsql-patches by date

Next:From: Tom LaneDate: 2008-09-08 18:18:16
Subject: Re: [HACKERS] Infrastructure changes for recovery
Previous:From: Simon RiggsDate: 2008-09-08 18:01:50
Subject: Re: [HACKERS] Infrastructure changes for recovery

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group