Fix two typos in comments.

Re-add brace removal code but comment it out so we know why we removed
it and have it in case we need it for some special case.

Remove code that delete braces around single statements.

When LockAcquire fails at the stage of creating a proclock object, be
sure to clean up the already-created lock object, if it has no other
references.  Avoids possibly-permanent leak of shared memory.

Renumber SnapshotNow and the other special snapshot codes so that
((Snapshot) NULL) can no longer be confused with a valid snapshot,
as per my recent suggestion.  Define a macro InvalidSnapshot for 0.
Use InvalidSnapshot instead of SnapshotAny as the do-nothing special
case for heap_update and heap_delete crosschecks; this seems a little
cleaner even though the behavior is really the same.

Ensure that pg_largeobject references opened by lo_import() or lo_export()
will be cleaned up at end of transaction, even when there is no other LO
operation in the transaction.  Per bug report from Daniel Schuchardt.

Send thread test output to file descriptor 5 like configure does rather
than /dev/null, which Win32 doesn't have.

Check for ignored thread compiler options to reduce compiler noise.

Done:

< * Have AFTER triggers execute after the appropriate SQL statement in a
> * -Have AFTER triggers execute after the appropriate SQL statement in a

Fix some problems with restoring databases owned by non-superusers,
as per bug #1249; and remove the last vestiges of using \connect to
change authorization.

Fire non-deferred AFTER triggers immediately upon query completion,
rather than when returning to the idle loop.  This makes no particular
difference for interactively-issued queries, but it makes a big difference
for queries issued within functions: trigger execution now occurs before
the calling function is allowed to proceed.  This responds to numerous
complaints about nonintuitive behavior of foreign key checking, such as
http://archives.postgresql.org/pgsql-bugs/2004-09/msg00020.php, and
appears to be required by the SQL99 spec.
Also take the opportunity to simplify the data structures used for the
pending-trigger list, rename them for more clarity, and squeeze out a
bit of space.

Add:

> * Make row-wise comparisons work per SQL spec
>

Fix #if define

Fix palloc call from /port for Cygwin.

Properly include port file for Cygwin.

Minor cleanup.

More cleanup.

Use $PORTNAME consistently instead of $host_os, where appropriate.

Add:

> * Remove Win32 rename/unlink looping if unnecessary

Add mention loops over unlink/rename might not be needed.

Move undef of rename/unlink so system declarations are unmodified.

Adjust quotes for win32 build of psql.

Fix two typos in comments.

Apply patch from Steven Singer for contrib/dbmirror. Changes:

-It fixes up some bugs with handling setval calls
-Adds upgrade instructions from prior versions
-Improved the sample config file
-Fixed some things in the clean_pending script

psql consistency fixes from Greg Sabino Mullane: have \df show schema and
name first, make sure \di+ shows description last, and make \dl SQL look
a little more standard.

Create pg_config_paths in port/ for win32 builds of psql.

Clean up Cygwin test.

Move TIMEZONE_GLOBAL out into the proper place.

We don't use java in our compiles anymore so remove the filter test from
pgtest.

Added a call to gettext() to fix broken translated error messages.

Translation updates

Some source files was not scanned for translatable strings.

Translation updates

Fix Cygwin defines to be consistent.

Make TZNAME_GLOBAL for reference to tzname global variable.

Minor efficiency improvements in keeping track of trigger deferred
status.  In particular, I see no reason for deferredTriggerCheckState
to make an explicit entry to note that a particular trigger has its
default state --- that just clutters a list that should normally be
empty or very short.  I have plans to revise this module much more
heavily, but this is a simple separable improvement.

Clarify description of SET CONSTRAINTS.  Point out that it is still
missing the ability to schema-qualify constraint names.

Use _timezone global on Cygwin instead of timezone.

Improve README with mention of new functions.

Fix a couple of small errors in trigger-list management, as per recent
discussion.

Fix places where WaitForxxx can block, to eliminate failure to detect
deadlock on Win32.  Magnus Hagander

Fix error in Windows version of CHECK_FOR_INTERRUPTS macro.
This prevented statement_timeout from working.
Magnus Hagander

Fix a number of places where brittle data structures or overly strong
Asserts would lead to a server core dump if an error occurred while
trying to abort a failed subtransaction (thereby leading to re-execution
of whatever parts of AbortSubTransaction had already run).  This of course
does not prevent such an error from creating an infinite loop, but at
least we don't make the situation worse.  Responds to an open item on
the subtransactions to-do list.

Guard against transaction control statements in SQL functions.  This
never worked, but it particularly doesn't work now.

Fix incorrect ordering of smgr cleanup relative to buffer pin cleanup
during transaction abort.  Add a regression test case to catch related
mistakes in future.  Alvaro Herrera and Tom Lane.

I can't see any good reason for DropRelFileNodeBuffers to be issuing
FATAL when it detects a nonzero reference count.  Reduce to ERROR.

Fixed segfault in Informix mode.

Downgrade LOG messages to DEBUG1 for normal recycling of xlog, clog,
subtrans segments.  Per Greg Mullane and Chris K-L.

Simplify IsXactIsoLevelSerializable test.  A cycle saved is a cycle
earned ...

On further consideration, there's another problem here: the existing
elog() emulation code always calls errstart with ERROR error level.
This means that a recursive error call triggered by elog would do
MemoryContextReset(ErrorContext), whether or not this was actually
appropriate.  I'm surprised we haven't seen this in the field...

Tweak elog.c's logic for promoting errors into more severe errors.
Messages of less than ERROR severity should never be promoted (this
fixes Gaetano Mendola's problem with a COMMERROR becoming a PANIC,
and is obvious in hindsight anyway).  Do all promotion in errstart
not errfinish, to ensure that output decisions are made correctly;
the former coding could suppress logging of promoted errors, which
doesn't seem like a good idea.  Eliminate some redundant code too.

Fix shared library linking operations to work properly on all (or at
least more) combinations of HP-UX hardware, compiler, and linker.
Shinji Teragaito and Tom Lane.

Apparently HPUX runs the IA64 in big-endian mode.

Fix ancient declaration inconsistency in cube & seg ... came across a
compiler that warns about it.

Some versions of lex will drop a lex.yy.c file when we probe to see if
they are flex.  Clean up after them.

Yet another place where someone was being careless about the arguments
of <ctype.h> macros.

Spelling correction, per Greg Mullane.

Fix getrusage() emulation on Windows.  Magnus Hagander

FAQ_HPUX updates from Shinji Teragaito.

Cope with recent HPUX versions providing isfinite() instead of finite().

Add s_lock support for HPUX on IA64, per Shinji Teragaito.

Remove useless variable.

Use $PATH_SEPARATOR like the rest of the autoconf code, instead of
hardwiring IFS=: when searching paths.

Remove line specifying Andreas as author.

Remove pg_config_paths.h from libpq *.mak builds so later MinGW builds
will succeed.

Add:

> * Consider using hash buckets to do DISTINCT, rather than sorting
>
>   This would be beneficial when there are few distinct values.

Back out timezone detection patch.  Tom already applied it.

This patch attempts to fix the issue with localized timezones on
Windows.

Recap: When running on a localized windows version, the timezone name
returned is also localized, and therefor does not match our lookup
table.

Solution: The registry contains both the name of the timezone in english
and the localized name. The patch adds code to scan the registry for the
localized name and gets the english name from that, and then rescans the
table.

I have tested this on a Swedish WinXP, and it works without problems.
The registry layout is the same in Win2k, but I haven't specifically
tested it. It's also the same on different languages but again only
Swedish is tested.

Magnus Hagander

The current implementation of dbsize doesn't handle tables in
tablespaces correctly, and is quite restricted on objects covered (only
tables and databases, but not tablespaces and indexes).

The attached patch contributes:

- database_size(name)
- relation_size(text)
These are the well-known functions, tablespace-aware.

- pg_tablespace_size(oid)
- pg_database_size(oid)
- pg_relation_size(oid)
Tablespace-aware implementations, used by the upper functions.
pg_relation_size will report sizes of indexes as well.

- pg_size_pretty(bigint)
Formatting of sizes, to display '146MB' instead of '152885668'

Andreas Pflug

Remove obsolete comment.

Tweak prettyprinting rules for saner indenting of UNION, INTERSECT,
EXCEPT constructs.

Fix bcc to compile libpq by creating include file for path.c.

Realign libpq and psql for need for path.c under Win32 VC and BCC.

Back out Cygwin timezone change until we have a more global solution.

Fix mistaken comment.

Whack Wisconsin benchmark around until it actually works again.
It's still useless because it tests a standalone backend, but at least
the bit rot is repaired.

Add code to be able to match the timezone name on localized Windows
systems.  Magnus Hagander.

Remove a stray reference to sequences as having tablespaces.

Translation updates

Add a note about the difference between Postgres' treatment of the rights
of an object owner and the SQL spec's treatment of these rights.

Cast _timezone to int from time_t for Cygwin.

Add index mention:

    <P>In pre-8.0 releases, indexes often can not be used unless the data
    types exactly match the index's column types.  This is particularly
    true of int2, int8, and numeric column indexes.</P>

src/bin/psql/tab-complete.c =~ s/CONVERSATION/CONVERSION/;

Greg Sabino Mullane

needs_toast_table() should ignore dropped columns.

Code review for recent changes in guc-file.l.  Avoid multiple frees,
use of already-freed strings, other silliness.  Also fix reporting of
config file syntax errors so that it actually works reasonably well
(eg, points at the correct line).  Use palloc instead of malloc for
temporary storage to reduce code clutter.

Code review for various recent GUC hacking.  Don't elog(ERROR) when
not supposed to (fixes problem with postmaster aborting due to mistaken
postgresql.conf change); don't call superuser() when not inside a
transaction (fixes coredump when, eg, try to set log_statement from
PGOPTIONS); some message style guidelines enforcement.

Fix unintended assignment of sequences to the containing schema's
default tablespace --- they should always go in the database's default
tablespace.  Adjust heap_create() API so that it is passed the relkind
to make this easier; should simplify any further tweaking of the same
sort.

FlushRelationBuffers was also being a bit cavalier about whether the
relation is already opened by smgr.

copy_relation_data was mistakenly assuming that the source relation
would always be already open at the smgr level.  Per bug report from
Fabien Coelho.

Define lstat with parameters, rather than just redefining the symbol.

Fix typo in lstat() macro exposed by new tablespace code.

Replace log_filename_prefix with more general log_filename parameter,
to allow DBA to choose the form in which log filenames reflect the
current time.  Also allow for truncating instead of appending to
pre-existing files --- this is convenient when the log filename pattern
rewrites the same names cyclically.  Per Ed L.

tag configure beta2

Improve spinlock selftest to make it able to detect misdeclaration of
the slock_t datatype (ie, declared type smaller than what the hardware
TAS instruction needs).

slock_t must be int not char for MIPS.  7.4 got this right, but the
info was apparently mistranscribed in s_lock code rearrangement.

Avoid ambiguity, as per suggestion from Peter Stricker.

Rather than referring to src/tutorial/README, put the build instructions
for the tutorial files right here.

Fix obviously-utterly-untested noTocComments code.

Ensure that the remainder of the current pg_clog page is zeroed during
startup, just to be sure that there's no leftover junk there.

Fix failure to advance nextXID beyond subtransactions whose XIDs appear
only within COMMIT or ABORT records.