From: Bruce Momjian <maillist@candle.pha.pa.us>
Message-Id: <199804260403.AAA19745@candle.pha.pa.us>
Subject: Re: [HACKERS] char types gone.
To: darrenk@insightdist.com (Darren King)
Date: Sun, 26 Apr 1998 00:03:42 -0400 (EDT)
Cc: pgsql-hackers@postgreSQL.org
In-Reply-To: <9803210045.AA22642@ceodev> from "Darren King" at Mar 20,
	98 07:45:46 pm
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

Re-applied this patch, now that 6.3.* is closed.

> 
> 
> Have managed to delete char2,4,8 and 16.  The char16 was tricky
> since it was still one of the builtins.  Removed fine everywhere
> but the bootstrap code.  Had to leave a dummy entry in the Procid
> array so all of the types after char16 would be indexed correctly.
> Could remove this dummy by moving types 21-30 down one, but that
> would entail renumbering the catalog.  With that in mind, I can
> live with a filler entry in an array.  Heaven knows there are bigger
> and badder hacks elsewhere in the code dying to be discovered. :)
> 
> Three points now...regression tests, docs and interfaces.
> 
> 1. Regression tests make very extensive use of char16.  Seems to
> me that these should be changed to a name.  Same for tutorial and
> any other example references in the code.  Seem ok?
> 
> 2. I'll take my best shot at the docs.
> 
> 3. Also take a look/see at the interfaces that reference the char16.
> Perhaps would be better for me to contact the maintainers of those
> interfaces as they would know what to do better than I.
> 
> Once I do one and two, I'll submit the patch.
> 
> darrenk
> 
> 


-- 
Bruce Momjian                          |  830 Blythe Avenue
maillist@candle.pha.pa.us              |  Drexel Hill, Pennsylvania 19026
  +  If your life is a hard drive,     |  (610) 353-9879(w)
  +  Christ can be your backup.        |  (610) 853-3000(h)