Re: Simplify formatting.c - Mailing list pgsql-patches

From Alvaro Herrera
Subject Re: Simplify formatting.c
Date
Msg-id 20080614212942.GD8519@alvh.no-ip.org
Whole thread Raw
In response to Re: Simplify formatting.c  (Bruce Momjian <bruce@momjian.us>)
Responses Re: Simplify formatting.c  (Bruce Momjian <bruce@momjian.us>)
List pgsql-patches
Bruce Momjian wrote:

> I moved str_initcap() over into oracle_compat.c and then had initcap()
> convert to/from TEXT to call it.  The code is a little weird because
> str_initcap() needs to convert to text to use texttowcs(), so in
> multibyte encodings initcap converts the string to text, then to char,
> then to text to call texttowcs().  I didn't see a cleaner way to do
> this.

Why not use wchar2char?  It seems there's room for extra cleanup here.

Also, the prototype of str_initcap in builtins.h looks out of place.

--
Alvaro Herrera                                http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

pgsql-patches by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: Tentative patch for making DROP put dependency info in DETAIL
Next
From: Alvaro Herrera
Date:
Subject: Re: relscan.h split