More improvements;  still disabled.  (Don't panic.)

The result of getopt() should be compared to -1, not EOF, per
pgsql-hackers discussion of this date.

Describe type casts under the heading of Value Expressions, and explain
the difference between a run-time type cast and casting a literal string
to a specific type.  Minor editorial work in same area.

Update with new features.  Still disabled.

Fix use of 'char' to hold result of getc, per bug report forwarded by
Oliver Elphick.  A few other minor cleanups while at it.

Fix portability problem (result of getopt is int, not char).

Remove clog copy.

Make improvements to pg_upgrade; is still disabled.

Add quotes around $bindir.

Make spelling of operating system names match official version closely.
And listing QNX 6 as both supported and unsupported is probably not helpful.

Fix include paths for case of VPATH build.

Adjust spelling of article titles to match the originals, add some links.

make install failed in VPATH build.

Fix copy-and-paste mistake exposed by gcc warning.

In a VPATH build, resultmap must be one of the symlinked files, else
pg_regress doesn't see it and you don't get any port-specific
comparisons.

Revert last change (CFLAGS+=-g).  Probably was a mistake...

Update Lockhart's email address.

< * Thomas is Thomas Lockhart <lockhart@alumni.caltech.edu>
---
> * Thomas is Thomas Lockhart <lockhart@fourpalms.org>

Update Thomas Lockhart's email address to lockhart@fourpalms.org.

Remove shift/reduce conflicts introduced by last change.

Repair the GEQO bibliography.  It was marked up all wrong.  Move some of
the info to the main bibliography.

Fix lpad() and rpad() to produce correct results in variable-length
multibyte encodings.

Point to main CPAN site, and reword a little.

Minor wordsmithing.

Make FreeBSD URL point to the right chapter again.

Fixed array pointers, no longer using void * in arithmetics.

Email address no longer valid.

Update bad URL's with good ones.

Update my2pg to version 1.22.

Add variants of digest() and hmac() that accept text inputs.
 Marko Kreen says:
This is so obvious that I would like to make it 'official'.

Seems like the theology around bytea<>text casting kept me from
seeing the simple :)

Tweak LWLock algorithms so that an awakened waiter for a lock is not
granted the lock when awakened; the signal now only means that the lock
is potentially available.  The waiting process must retry its attempt
to get the lock when it gets to run.  This allows the lock releasing
process to re-acquire the lock later in its timeslice.  Since LWLocks
are usually held for short periods, it is possible for a process to
acquire and release the same lock many times in a timeslice.  The old
spinlock-based implementation of these locks allowed for that; but the
original coding of LWLock would force a process swap for each acquisition
if there was any contention.  Although this approach reopens the door to
process starvation (a waiter might repeatedly fail to get the lock),
the odds of that being a big problem seem low, and the performance cost
of the previous approach is considerable.

Fixed parser to accept initializing expressions starting with "(".

Editorial review

If we fail to fork a new backend process, (try to) report the failure
to the client before closing the connection.  Before 7.2 this was done
correctly, but new code would simply close the connection with no report
to the client.

Fix ecpg wording to be consistent.

Remove ecpg TODO section:

> Looking at this I also found an ecpg TODO list in the docs:
>
>
http://candle.pha.pa.us/main/writings/pgsql/sgml/ecpg-develop.html
>
> Seems that TODO section should be removed.  Some items are done,
others
> are on the main TODO list.

That's correct. I did not fix the docs for quite some time.

Michael
--
Michael Meskes

Fix copy-and-paste brain damage.  (These commands do not create a
database.)

Add CHECK_FOR_INTERRUPTS() in various strategic spots, per comments
from Hiroshi.

Bugfix for bug reported by Marcus Better (marcus@dactylis.com).  When preforming
a get on a bytea value the code was running the raw value from the server
through character set conversion, which if the character set was SQL_ASCII
would cause all 8bit characters to become ?'s.

Spelling fix:

< * Consider use of open/fctl(O_DIRECT) to minimize OS caching
---
> * Consider use of open/fcntl(O_DIRECT) to minimize OS caching

Re-order items:

> * Consider use of open/fctl(O_DIRECT) to minimize OS caching
> * Make blind writes go through the file descriptor cache
391d392
< * Make blind writes go through the file descriptor cache
409d409
< * Consider use of open/fctl(O_DIRECT) to minimize OS caching

Added:

> * Consider use of open/fctl(O_DIRECT) to minimize OS caching

ecpg.sgml manual page updated to ON.

---------------------------------------------------------------------------

When you run 'ecpg --help' you get the following:

          -t   turn on autocommit of transactions

amongst the other options... Shouldn't this be OFF as per the
documentation?

Best regards, Lee.

--
 Lee Kindness,   Senior Software Engineer,   lkindness@csl.co.uk

Change:

<   perhaps CREATE DATABASE dbname WITH USER = "user" (Gavin)
---
>   perhaps CREATE DATABASE dbname WITH OWNER = "user" (Gavin)

Add from Vadim:

> * Allow command blocks that can ignore certain types of errors

Fix some incorrect and obsolete commentary.

Add more complete details on date/time keywords and parsing rules.
Move some tabular information on these from the chapter on data types to
 the appendix on dates and times.

Have to_date() call timestamptz_date() per Karel's email instructions.
Fixes time zone problems introduced by Thomas' implementation of
 TIMESTAMP WITHOUT TIME ZONE which caused the behavior of the previously
 appropriate routine, timestamp_date(), to change for the worse in this
 context.

Replace #ifdef, #endif with # requires comment.

Update FAQ.

Give attribution:

<   perhaps CREATE DATABASE dbname WITH USER = "user"
---
>   perhaps CREATE DATABASE dbname WITH USER = "user" (Gavin)

Remove:

< * Allow only owner to create indexes

Require ownership permission for CREATE INDEX, per bug report.
Disallow CREATE INDEX on system catalogs, non-tables (views, sequences, etc).
Disallow CREATE/DROP TRIGGER on system catalogs, non-tables.
Disallow ALTER TABLE ADD/DROP CONSTRAINT on system catalogs.
Disallow FOREIGN KEY reference to non-table.
None of these things can actually work in the present system structure,
but the code was letting them pass without complaint.

Instead of waiting a fixed amount of time for the test postmaster to
start up, wait until a psql connection attempt succeeds.  Per gripe
from Jason Tishler.

Prefer savepoints term:

<    allow error codes to be specified; requires nested transactions
---
>    allow error codes to be specified; requires savepoints
321c321
< * Allow nested transactions / savepoints [transactions]
---
> * Allow savepoints / nested transactions [transactions]

Update:

<    allow error codes to be specified
---
>    allow error codes to be specified; requires nested transactions

SPI_cursor_open must copy by-reference parameter values into the
portal's memory context, so that they will live as long as the portal does.

Add:

>  o Allow COPY to report error lines and continue;  optionally
>    allow error codes to be specified

Add:

> * Make equals sign optional in CREATE DATABASE WITH param = 'val'

Added:

> * Prevent create/drop scripts from allowing extra args (Bruce)

Shouldn't try to copy null datums with datumCopy.

Added:

> * Allow only owner to create indexes

Updated:

< * Add config file check for $ODBCINI, $HOME/.odbc.ini, and /etc/odbc.ini
---
> * Add config file check for $ODBCINI, $HOME/.odbc.ini, installpath/etc/odbc.ini

Add to java mention of pl/j.

Droplang didn't have test for -l, list.  This was really broken too.
Took test from createlang.

Add mention that createlang.sh indirection not needed but kept for later.

Update FAQ_DEV.

Update FAQ_DEV.

Added

> * Add config file check for $ODBCINI, $HOME/.odbc.ini, and /etc/odbc.ini

Added suggested syntax:

> * Make it easier to create a database owned by someone who can't createdb,
>   perhaps CREATE DATABASE dbname WITH USER = "user"

> > On Fri, Dec 21, 2001 at 11:43:21AM +0800, Christopher Kings-Lynne
wrote:
> > > Just testing pgcrypto on freebsd/alpha.  I get some warnings:
> > They should be harmless, although I should fix them.
>
> The actual code is:
>
>     if ((dlen & 15) || (((unsigned) res) & 3))
>         return -1;

> Hard to imagine how (uint *) & 3 makes any sense, unless res isn't
> always a (uint8 *).  Is that true?

At some point it was casted to (uint32*) so I wanted to be sure its ok.
ATM its pointless.  Please apply the following patch.

--
marko

Add mention of index on foreign key for frequently updated primary key
tables.

droplang had same problem as createlang;  default are backwards and did
not work.  Fixed to match createlang.

Add to pool discussion.

Added

<  o Move to using 'make' rather than 'ant'(?)
---
>  o Support both 'make' and 'ant'

Add mention of PRIMARY in ALTER TABLE ... [ UNIQUE | PRIMARY ]

In createlang, make dbname optional, not langname, or rather, make
existing code of dbname optional actually work.

Remove:

< * Change 'createlang [langname] dbname' to 'createlang langname [dbname]'

Added:

> * Change 'createlang [langname] dbname' to 'createlang langname [dbname]'

Guard against roundoff errors in new selectivity-estimation code,
per bug report from Laurette Cisneros.

Fix to_timestamp/to_date so that zero year input for Y, YY, or YYY
formats will be taken as 2000, not year zero.  Per bug report from
Aasmund Midttun Godal.  Fix from Karel Zak.

Fix incorrect description of rowtype variable declarations.  Eventually
(IMHO) the code should be fixed to do what the documentation formerly
claimed, and then revert this change.

Do not accept interrupts in RESUME_INTERRUPTS() and END_CRIT_SECTION()
macros, but only at explicit CHECK_FOR_INTERRUPTS() calls.  Not clear
whether overenthusiastic acceptance of interrupts accounts for any real
bugs, but it definitely seems risky and unnecessary.

Fix race condition that could allow two concurrent transactions
to insert the same key into a supposedly unique index.  The bug is of
low probability, and may not explain any of the recent reports of
duplicated rows; but a bug is a bug.

Rename TIME token to ISOTIME to eliminate conflict with gram.y parser
 token. Seems to be isolated to datetime.c and datetime.h.

Update dbase for Maarten.Boekhold.

Make sure that all <ctype.h> routines are called with unsigned char
values; it's not portable to call them with signed chars.  I recall doing
this for the last release, but a few more uncasted calls have snuck in.

Repair example, per note in interactive docs.

Update FAQ_DEV.

*** empty log message ***

Add

> * Make locking of shared data structures more fine-grained

Oops, only wanted datetime.c changes in there.  lock stuff reversed out.

Fix newly introduced datetime.c compile failure;  not enough parens.

Added

> * Change factorial to return a numeric

Add missing <varlistentry>;  shown in build logs.

Check a bit more carefully for preceeding ISO field tags
 when decoding date fields.

Document the precision arguments for date/time types per SQL9x.
Update the list of recognized time zones.
Document the range of arguments allowed for SET TIME ZONE.
Still need to add info on other date/time symbols (e.g. "AM", "T")
 and to freshen the docs on the date/time parsing rules.

Rework the date/time parsing to tighten up some cases and to enable other
 cases which should have worked but did not.
Now supports julian day (J2452271), ISO time labels (T040506) and various
 combinations of spaces and run-togethers of dates, times, and time zones.
All regression tests pass, and I have more tests to add after the 7.2
 release (don't want to require changes to the ancillary horology result
 files until after then).

Add crossdb TODO.detail item.

Add crossdb TODO.detail item:

---
> * Allow queries across multiple databases [crossdb]

Update server DEBUG startup tag fixes:

> * Change DEBUG startup tag to NOTICE;  change NOTICE to output to client
>   only if client exists (Bruce)

This item is not done:

< * -Make elog(LOG) in WAL its own output type, distinct from DEBUG (Peter E)
---
> * Make elog(LOG) in WAL its own output type, distinct from DEBUG