Many more cleanups...

Initial attempt to clean up the code...

Switch sprintf() to snprintf()
Remove any/all #if 0 -or- #ifdef NOT_USED -or- #ifdef FALSE sections of
code

OK, here is a diff for the README file in /usr/src/pgsql/contrib/spi/.
For the 6.5 tree.

Have a great night.
Terry

Current multi-byte related codes have a bug with SQL_ASCII
support. Included patches will solve it and should be applied to
both trees.  Also, it fix the problem with \c command of psql when
switching different encoding databases.

Regression tests passed.
--
Tatsuo Ishii
t-ishii@sra.co.jp

Add support for the CASE statement in the rewrite handling.
Allows (at least some) rules and views.
Still some trouble (crashes) with target CASE columns spanning tables,
 but lots now works.

Improve CASE statement support.
Try to label CASE columns for a SELECT if not specified with an AS clause.

Fix up error messages when looking up functions and operators to not
 make the sometimes misleading claim that more than one candidate was
 identified. Now say "Unable to identify...".

Use the new implicit type coersion techniques for matching up types
 between columns and DEFAULT clauses.

Clean up error messages associated with rules and views.
Fix parser error message when an operator is not found
 to not explicitly claim that there is more than one.

Build pltcl.so correctly on platforms that want dependent
shared libraries to be listed in the link command.

Add routines and synonyms to help with single-byte char type handling.
Fix one usage of substr() which mapped to the "Oracle compatibility" funcs
 rather than the more recent (and closer to SQL92) function in varlena.c.
Add more DESC() entries for conversion functions.

Adjust handling of data type names to suppress double quotes
 unless necessary.
Label internal bpchar types as "character" and varchar types as
 "character varying" to be less Postgres-specific. These types map to
 the SQL92 definitions anyway.
Redefine g_force_quotes to be the local variable force_quotes.
Pass this as an argument to fmtId().
These should help with handling the single-byte internal "char" type.

Add routines to help with single-byte (internal) character type support.

Switch around conditional code so that HAVE_TM_ZONE takes precedence
 over HAVE_INT_TIMEZONE. This may help out linux/glibc2 and Dec Alpha.
Included #error precompiler macros to catch cases where neither is defined
 but USE_POSIX_TIME is (shouldn't happen). Hopefully this isn't just
 a gcc-ism.

Partial fix for vendor-cc-versus-gcc incompatibility on HPUX:
prefer aCC as c++ compiler if hpux_cc is selected as template.
Doesn't solve the problem if you have g++ and not aCC, however...

Use standard AC_PROG_INSTALL macro to search for install program,
instead of our own halfway-there code.  Add AC_STRUCT_TIMEZONE call
to check whether tm_zone exists in struct tm.  Revise reading of template
file so that templates can define any variables they feel like (and,
indeed, can execute arbitrary shell code) rather than being constrained
to a fixed set of variable names.

Add enable_plpgsql from Oliver.

Cleanup patch from Oleg.

Fix for lo_import crash.

While investigating a user's complaint, I have found some memory
destructions in 6.4 source using purify.

(1) parser/gram.y:fmtId()

It writes n+3 bytes into n+1 byte-long memory area if mixed case or
non-ascii identifiers given.

(2) catalog/index.c:

ATTRIBUTE_TUPLE_SIZE bytes are allocated but
sizeof(FormData_pg_attribute) bytes are written. Note that
ATTRIBUTE_TUPLE_SIZE is smaller than
sizeof(FormData_pg_attribute). (for example, on solaris 2.6,

Tatsuo Ishii

I have a simple patch about the treatment of functions.
But it may be self-satisfied.
Please check my patch at the end of this posting.

Case 1. executor evaluates functions twice

Hiroshi Inoue
Inoue@tpf.co.jp

New ecpg man page.

Change Size from unsigned int to size_t.

This patch fixes the undefined (according to C) and erroneous (under
Digital Uni x with both DEC cc and gcc) behaviour of modifying an
lvalue on the left side an d then using it on the right side of an
assignment. Since this code modifies the
 dbname parameter, it was changing, for example, "dbname=template1"
 into "dbname =emplate1".

David Smith Programmer P

Included patches should fix following problems in the muti-byte
enabled PostgreSQL 6.4.

o binary cursor does not work
o pg_dumpall produces incorrect create database statemnt

Tatsuo Ishii
t-ishii@sra.co.jp

Fix prototype for 64-bit platforms.

Update to pgaccess 0.93.

New moddatetime contrib from Terry Mackintosh.

Add stdio for NULL define.

Clean up a few elog() messages for aggregates and functions.

Define routines and catalog entries for string min()/max() functions.
Extend new type coersion techniques to aggregates.
Clean up a few elog() messages.

Define routines and catalog entries for string min()/max() functions.
Extend new type coersion techniques to aggregates.

Fix coredumping bug in pg_dump -z; also eliminate memory leaks
in the ACL code, and spell "GRANT RULE" correctly.
Apply patch from Oliver Elphick to not dump inherited constraints.
Apply patch from Constantin Teodorescu to dump table definitions with a
readable layout.

These data types were removed before the last release.

Include test for CASE expression.

Implement CASE expression.

Portability fixes found needed for SunOS 4.1.x:
SunOS has tas(), but not memmove or strerror, and its sprintf() doesn't
return int.  Also, older versions of GNU Make don't like rules with
empty left-hand sides...

Use autoconf to determine whether system has POSIX signals,
instead of relying on port's os.h to tell us.  (Needed for HPUX
where system major version is not enough info.)
configure unsets USE_TK if X libraries not found.
doc/Makefile uses gzcat or zcat as found by autoconf.

Change exp() behavior to generate error on underflow rather
than silently returning zero on some machines.  Correct float8 regress test
to agree.  Also fix pow() overflow/underflow check to work correctly on
HPUX.

Portability fix for old SunOS releases: realloc(NULL, ...)
doesn't work there.

Portability fix for old SunOS releases: fflush(NULL)
doesn't work there.  Fortunately the postmaster only has stdout and
stderr to flush.

pq_getstr didn't handle buffer overrun correctly; it would
fail to consume the rest of the input string, and worse it would write
one more byte than it should into the buffer, probably resulting in coredump.
Fortunately there's a correct implementation next door in pqcomprim.c.

Add link to HPUX platform FAQ.

New platform-specific FAQ for HP-UX sites.

New HeapTuple structure/interface.

New HeapTuple structure/interface.

New HeapTuple structure/interface.

csh and sh examples were reversed.
Update the csh example with new comments and fixes from Thomas' linux box.

Update platform FAQ's.

Fixed bug in the unknown module. Module still tried to call
builtin SPI commands by their old SPI_* names. New names are spi_*.

Jan - 11/23/1998

modify configure so that template/.similar entries can be
selected when they match a prefix of the  value.  The previous method,
which stripped all version data from  and then tried to match that
against .similar entries, was entirely useless when .similar contained
several entries for different version numbers of a single OS name.

Fix using indices in OR.
EXPLAIN all indices used.

Added indxqualorig to IndexScan: fix for using indices in OR.

Updating and addign files...

Performance improvement for libpq: avoid calling malloc separately
for each field of each tuple.  Makes more difference than you'd think...

Add text<->float8 and text<->float4 conversion functions.
This will fix the problem reported by Jose' Soares
 when trying to cast a float to text.

Additional fixes for mixed-case table names from Billy Allie.

From: Taral <taral@cyberjunkie.com>

More COS Query Service support.

Start defining the Corba work...

From: Taral <taral@cyberjunkie.com>

Fix a couple of bugs involving calling fmtID() twice in
a single printf() ... it'd work if fmtId() didn't use a static return area...

Add new "-N" flag to force double quotes around identifiers.
Fix missing mention of "-n" flag in synopsis.

Add id attribute to reftitle to allow cross referencing from the new
 vaccumdb reference page.

Add reference page for new vacuumdb utility.

Add "vacuumdb" utility to make it easier to clean databases.
Also supports the "analyze" mode, with or without specifying tables
 and columns.

Add "vacuumdb" utility.

Fix test for table existance to allow mixed-case and whitespace in
 the table name. Problem reported by Billy Allie.

Fix at least a few shared-library problems on Linux.
We have not seen the end of this issue :-(.

Update for version 06-40-0001

Fix for NOT in where clause causing crash.

EXPLAIN VERBOSE had a very high probability of triggering
a backend core dump, because it was concatenating a potentially long
string onto another string that didn't necessarily have enough room.
Shame, shame.

Fix a potential infinite loop in appendStringInfo: would lock
up if first string to be appended to an empty StringInfo was longer
than the initial space allocation.
Also speed it up slightly.

Update todo.

Add "-N" flag to force double quotes around identifiers.
This is the default, but the new flag will allow overriding an alias,
 for example. So psql -n -N will put in the double quotes,
 and psql -n can be an alias for psql.
Also, add a few braces around a nested single-line conditional construct
 to suppress compiler warnings about "an ambiguous else".

August version of docs superceded by sgml version from Jan.

Upgrade to 0.91 ...

Change the README file to show the upcoming 6.5 release...

Test for and handle NULL arguments to mylog() print statements.
Include definition for _IN_ADDR_T if INADDR_NONE is not defined.

Somehow had the solaris template names reversed. Probably was something
 fixed in src/configure.in a while ago.

Fix email addres for Dan Horak per his request.

Move configuration info to after installation procedure.
Include only the current release  in the release notes section.

Include instructions on perl installation.
 Already in product docs since they were built on my machine.

Fix for relname.data from SHIOZAKI Takehiko

slight mods required for freebsd (elf vs aout stuff)

Remove various files that were moved to various subdirectories...

Requested by: Peter T Mount peter@retep.org.uk

Cleanup for v6.4 release.
Make new file current.sgml to hold release info for the current release.
 Should be moved to release.sgml before filling with next release info.

Try to get only html and gif files into tarballs.
 Was picking up subdirectories and other garbage.

New versions for v6.4 release.

Generate from installation.sgml -> installation.rtf -> ApplixWare
 -> INSTALL with some cleanup in vi.

Thanks to Vince Vielhaber <vev@michvhf.com>

Here is an os2 psql client, with libpq.a library

oops, forgot one part of the patch...remove 'false' setting to ELF_SYS...

Minor FreeBSD fixes put in place

From: SHIOZAKI Takehiko <takehi-s@ascii.co.jp>

I tried snapshot(Oct30) and made some patches.
# I think that it is confused to manage both Makefile.shlib and
# makefiles/Makefile.*, don't you?

* configure
        Now FreeBSD 2.X is not supported..., so I added its entry.
        If ELF_SYSTEM is set, gmake treat it defined even though
        it is "false".  So nothing should be set to use "ifdef".
        BSD_SHLIB etc. may have same problems.

* Makefile.shlib
        As you said, FreeBSD entry is much like BSD's.
        I only added ELF_SYSTEM code.

* makefiles/Makefile.freebsd
        Ifdef/else/endif can not be indented with TABs.

s_lock patch from Ryan.

Mention --without-CXX option in online help.

Minor editing and markup changes as a result of preparing the Postscript
 documentation for v6.4.
Bigger updates to the installation instructions (install and config).

Minor cleanup in markup, especially in the Output section.

Updated TODO.

Behave reasonably on a system without shared libraries.

Remove obsolete file listing support resources which has not been updated
 in a year an a half. Much of the info was wrong and is covered more
 correctly elsewhere.

Minor fix for Jan to remove claim in SPI and trigger chapters that
 Postgres is missing a procedural language. He correctly points out
 that we now have two (thanks to him, but he modestly didn't mention
 that).

User's Guide for v6.4.
Has three pages of goofy layout at the end. Can't figure out what Applix
doesn't like about it, so giving up for now.