Bruce Momjian <bruce(at)momjian(dot)us> writes:
> Now that upper/lower/initcase do not modify the passed string, I have
> simplified formatting.c with the attached patch.
I was thinking the same thing while reading the patch. But please,
make str_toupper() and friends declare their argument "const" if you're
going to do this.
Another issue in this area is that these functions could do with some
refactoring to eliminate useless text/cstring conversions; I'm pretty
sure some code paths are doing cstring->text->cstring in direct
succession. Also, it seems a bit inconsistent to be relying on
oracle_compat.c for upper/lower but not initcap.
regards, tom lane
In response to
pgsql-patches by date
|Next:||From: Tom Lane||Date: 2008-05-20 02:37:58|
|Subject: Re: libpq object hooks (libpq events) |
|Previous:||From: Andrew Chernow||Date: 2008-05-20 02:21:07|
|Subject: Re: libpq object hooks (libpq events)|