Re: [PATCHES] Changes in /contrib/fulltextindex

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Florian Helmberger <f(dot)helmberger(at)uptime(dot)at>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Christopher Kings-Lynne <chriskl(at)familyhealth(dot)com(dot)au>, Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCHES] Changes in /contrib/fulltextindex
Date: 2002-07-11 22:01:51
Message-ID: 200207112201.g6BM1p200563@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


Florian, I haven't seen this patch yet. Did you send it in?

---------------------------------------------------------------------------

Florian Helmberger wrote:
> 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
>
>
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo(at)postgresql(dot)org
>
>
>

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Ben 2002-07-11 22:10:29 Re: Jan's Name (Was: Re: I am being interviewed by OReilly)
Previous Message Bruce Momjian 2002-07-11 21:51:20 Re: Jan's Name (Was: Re: I am being interviewed by OReilly)