-----BEGIN PGP SIGNED MESSAGE-----
I'm picking this one up again, since my refactor of formatting.c has
been applied, and I seem to be done playing with psql for the time
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.7 (GNU/Linux)
-----END PGP SIGNATURE-----
On Wed, Jul 18, 2007 at 10:00 AM, Brendan Jurd <direvus(at)gmail(dot)com> wrote:
> On 7/18/07, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> > This is all good but I think that self-inconsistent format strings are
> > not really the main source of to_date problems. Most of the complaints
> > I've seen arise from to_date plowing ahead to deliver a ridiculous
> > answer when the input data string doesn't match the format. I'd like to
> > see the code try a little harder to validate the input data.
> Agreed, but so far it doesn't look like there's much overlap between
> the parsing and validating code and the stuff I'm playing with.
> I'll keep an eye out for any opportunities to improve on this, but it
> might end up being a separate TODO.
Actually, it turns out that there was an opportunity to improve on
validation. At the moment I'm experimenting with using strtol to
harvest integer values from the input string, instead of sscanf. So
far the results are promising. I've got the code throwing errors for
a few different kinds of bogus input:
* String is too short
* Value is out of range
* Value isn't an integer at all
I'm also getting rid of some more code duplication in do_to_timestamp.
I hope to have something worth posting to -patches in the near
In response to
pgsql-hackers by date
|Next:||From: Bryce Nesbitt||Date: 2008-04-28 01:38:05|
|Subject: Re: Proposed patch - psql wraps at window width|
|Previous:||From: Andrew Dunstan||Date: 2008-04-27 18:11:47|
|Subject: Re: Recent buildfarm failures involving statement_timeout|