PATCH: CITEXT 2.0 v4

Howdy,

I've attached a new patch with the latest revisions of for the citext  
contrib module patch. The changes include:

* Using strlen() to pass string lengths to the comparison function,
   since lowercasing the value can change the length. Per Tom Lane.
* Made citextcmp consistently return int32, per Tom Lane.
* Made the hash index function return the proper value, per Tom Lane.
* Removed the COMMENTs and GRANTs from citext.sql.in.
* Added a cast function from bpchar to citext, as suggested by Tom Lane.
* Set the storage type for CITEXT to "extended", to ensure that it will
   be toastable. Per Tom Lane.
* Fixed the COMMUTATOR of <=.
* Changed the cast from citext to bpchar from implicit to assignment.
   This eliminates ambiguous function resolutions.
* Eliminated superflous functions, per Tom Lane.
* Removed unnecessary `OPERATOR()` calls in NEGATORs and the like.
* Added binary in/out functions. Per Tom Lane
* Added an explicit shell type to make the output a bit quieter.
* Converted tests to pure SQL and omitted multibyte tests (though a
   few remain commented-out).
* Reorganized and expanded the documentation a bit.

This version is far better than I started with, and I'm very grateful  
for the feedback.

Now, I have a few remaining questions to ask, mostly just to get your  
opinions:

* The README for citext 1.0 on pgFoundry says:

> I had to make a decision on casting between types for regular  
> expressions and
> decided that if any parameter is of citext type then case  
> insensitive applies.
> For example applying regular expressions with a varchar and a citext  
> will
> produce a case-insensitive result.
>
> Having thought about this afterwards I realised that since we have  
> the option
> to use case-insensitive results with regular expressions I should  
> have left the
> behaviour exactly as text and then you have the best of both  
> worlds... oh well
> not hard to change for any of you perfectionists!

I followed the original and made all the regex and LIKE comparisons  
case-insensitive. But maybe I should not have? Especially since the  
regular expression functions (e.g., regexp_replace()) and a few non- 
regex functions (e.g., replace()) still don't behave case-insensitively?

* If the answer is "no", how can I make those functions behave case- 
insensitively? (See the "TODO" tests.)

* Should there be any other casts? To and from name, perhaps?

Thanks!

David