Re: [PATCHES] Changes in /contrib/fulltextindex - Mailing list pgsql-hackers

From Florian Helmberger
Subject Re: [PATCHES] Changes in /contrib/fulltextindex
Date
Msg-id OPEOLKEAAIMGBAMJNACKCEDAEKAA.f.helmberger@uptime.at
Whole thread Raw
In response to Re: [PATCHES] Changes in /contrib/fulltextindex  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: [PATCHES] Changes in /contrib/fulltextindex  (Tom Lane <tgl@sss.pgh.pa.us>)
Re: [PATCHES] Changes in /contrib/fulltextindex  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-hackers
Hi.

> Huh?  isalpha() *is* locale-aware according to the ANSI C spec.
> For instance, the attached test program finds 52 alpha characters
> in C locale and 114 in fr_FR locale under HPUX.
>
> I am not at all sure that this aspect of Florian's change is a good
> idea, as it appears to eliminate locale-awareness in favor of a hard
> coded delimiter list.

Just tried your example - you're right of course! I will remove the hard
coded delimited list and replace it with the proper calls as shown in the
code you've sent.

Florian





pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: DROP COLUMN Proposal
Next
From: Tom Lane
Date:
Subject: Re: [PATCHES] Changes in /contrib/fulltextindex