Talk About Network

Google


Register and Login
Nick
Password
Register create new account Sign up is FREE and you can post replies, new topics, bookmark posts and more!
Recover lost password


Data Bases > Pgsql Hackers > Re: PATCH: CITE...
Latest [ Topics | Posts ] Archive Post A New Topic Post a Reply
<< Topic < Post Post 1 of 2 Topic 9632 of 11013
Post > Topic >>

Re: PATCH: CITEXT 2.0

by stark@[EMAIL PROTECTED] (Gregory Stark) Jul 5, 2008 at 10:58 AM

"David E. Wheeler" <david@[EMAIL PROTECTED]
> writes:

> On Jul 3, 2008, at 09:53, Alvaro Herrera wrote:
>
>>> Thanks. What would citext_hash() look like? I don't see a  text_hash()
=
to
>>> borrow from anywhere in src/.
>>
>> See hash_any().  I assume the difficulty is making sure that
>> hash("FOO") =3D hash("foo") ...
>
> Great, big help, thank you. So does this look sensible?
>
>     txt =3D cilower( PG_GETARG_TEXT_PP(0) );
>     str =3D VARDATA_ANY(txt);
>
>     result =3D hash_any((unsigned char *) str, VARSIZE_ANY_EXHDR(txt));

I thought your data type implemented a locale dependent collation, not
just
a case insensitive collation. That is, does this hash agree with your
citext_eq on strings like "foo bar" <=3D> "foobar" and "foo=DF" <=3D>
"foos=
s" ?

You may have to use strxfrm

--=20
  Gregory Stark
  EnterpriseDB          http://www.enterprisedb.com
  Ask me about EnterpriseDB's PostGIS sup****t!

--=20
Sent via pgsql-hackers mailing list (pgsql-hackers@[EMAIL PROTECTED]
)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
 




 2 Posts in Topic:
Re: PATCH: CITEXT 2.0
stark@[EMAIL PROTECTED]   2008-07-05 10:58:04 
Re: PATCH: CITEXT 2.0
david@[EMAIL PROTECTED]   2008-07-05 17:46:39 

Post A Reply:
  Go here to Signup

AddThis Feed Button


About - Advertising - Contact - Frequently Asked Questions - Privacy Policy - Terms of Use - Signup

Contact
tan12V112 Fri Dec 5 9:12:48 CST 2008.