Thread

Re: [HACKERS] char types gone.

Darren King <darrenk@insightdist.com> — 1998-03-23T20:39:11Z

> > Have managed to delete char2,4,8 and 16.
> > 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?
> 
> Any reason not to use "text" for these cases? Or even varchar()? "name"
> is probably really only an internal Postgres type. We should have a
> regression test for it if we don't already. Let me know if you want me
> to help...

I think the 'name' type should be considered an internal type only and
mentioned as such wherever it is documented.  It's only purpose seems
to be for a consistent length to internal names and it does that rather
well.  User's would be better served with the other, varlena-based, types.

Any examples I was going to convert from char16 to char(16).  There are a
couple of mentions of the char16 struct that will have to be integrated
or changed to reference the varlena struct.

Finally, I'm redoing the char16 regression test for name.

> > 2. I'll take my best shot at the docs.
> 
> Where? The man pages or the sgml sources? I'm shooting for getting the
> sgml sources right, and obsoleting the man page _sources_, regenerating
> them from the sgml instead.

Do you want patches against doc/src/sgml/*?  The man pages references were
few, so they were easy to change for consistency sake then til they drop
into the bit bucket.  Changed a few references in the userguide too.

Only have to tidy up the docs and I'll submit the patch.

darrenk