Code and docs review for ALTER TABLE INHERIT/NO INHERIT patch.

Make unknown-option-warning code more portable.  echo -n is not portable,
and neither is "|" or "\|" in basic regular expressions.

Update Japanese FAQ.

Jun Kuwamura

Fix typo in version number.

Fix test_fsync compile on MinGW(win32)

Hiroshi Saito

Fix infinite sleep and failes of send in Win32.

1) pgwin32_waitforsinglesocket(): WaitForMultipleObjectsEx now called with
finite timeout (100ms) in case of FP_WRITE and UDP socket. If timeout occurs
then pgwin32_waitforsinglesocket() tries to write empty packet goes to
WaitForMultipleObjectsEx again.

2) pgwin32_send(): add loop around WSASend and pgwin32_waitforsinglesocket().
The reason is: for overlapped socket, 'ok' result from
pgwin32_waitforsinglesocket() isn't guarantee that socket is still free,
it can become busy again and following WSASend call will fail with
WSAEWOULDBLOCK error.

See http://archives.postgresql.org/pgsql-hackers/2006-10/msg00561.php

Remove:

<  o Issue a notice if CREATE TABLE ... ON COMMIT { DELETE ROWS |
<    DROP } is issued outside a multi-statement transaction

Add url's for hints:

>   http://archives.postgresql.org/pgsql-hackers/2006-10/msg00517.php
>   http://archives.postgresql.org/pgsql-hackers/2006-10/msg00663.php

Add:

>  o Issue a notice if CREATE TABLE ... ON COMMIT { DELETE ROWS |
>    DROP } is issued outside a multi-statement transaction

Make some incremental improvements and fixes to the documentation on
Continuous Archiving. Plenty of editorial work remains...

Update release notes for SQL functions vs triggers fix.

In the installation docs, add some links to the documentation on LDAP
to the discussion of the "--with-ldap" configure option. Patch from
Albe Laurenz.

Fix mishandling of after-trigger state when a SQL function returns multiple
rows --- if the surrounding query queued any trigger events between the rows,
the events would be fired at the wrong time, leading to bizarre behavior.
Per report from Merlin Moncure.

This is a simple patch that should solve the problem fully in the back
branches, but in HEAD we also need to consider the possibility of queries
with RETURNING clauses.  Will look into a fix for that separately.

Use Min() instead of min() in qsort, for consistency and to avoid
redefined-macro warnings on some platforms.  Per gripe from Hiroshi Saito.

Remove somebody's ill-considered free() call, per report from Chander Ganesan.
No, I do not care whether Coverity considers this a memory leak.
It's entirely not worth the code space to do it correctly.

Update pg_hba.conf comment about documentation section.

A bit of copy-editing on back-branch release notes.

Repair incorrect check for coercion of unknown literal to ANYARRAY, a bug
I introduced in 7.4.1 :-(.  It's correct to allow unknown to be coerced to
ANY or ANYELEMENT, since it's a real-enough data type, but it most certainly
isn't an array datatype.  This can cause a backend crash but AFAICT is not
exploitable as a security hole.  Per report from Michael Fuhr.

Note: as fixed in HEAD, this changes a constant in the pg_stats view,
resulting in a change in the expected regression outputs.  The back-branch
patches have been hacked to avoid that, so that pre-existing installations
won't start failing their regression tests.

Code review for LIKE INCLUDING CONSTRAINTS patch --- improve comments,
don't cheat on the raw-vs-cooked status of a constraint.

Rename function 'isexists' and 'isdefined' to
'exist' and 'defined' accordingly. Old names
are saved not mentioned in docs - for compatibility
with old applications.
Per discussion
http://archives.postgresql.org/pgsql-hackers/2006-10/msg00571.php

Cleanup pass

MSVC/BCC Win32 compiler fixes.

Hiroshi Saito

Typo fix.

Add release checklist item:

        o copy FAQs from HEAD to top-most branches

Update FAQ for latest release 8.1.5.

Revise psql pattern-matching switches as per discussion.  The rule is now
to process all inclusion switches then all exclusion switches, so that the
behavior is independent of switch ordering.
Use of -T does not cause non-table objects to be suppressed.  And
the patterns are now interpreted the same way psql's \d commands do it,
rather than as pure regex commands; this allows for example -t schema.tab
to do what it should have been doing all along.  Re-enable the --blobs
switch to do something useful, ie, add back blobs into a dump they were
otherwise suppressed from.

Improve description of the pattern matching rules used by psql's \d
commands (and soon by pg_dump).

Move processNamePattern into dumputils.c in preparation for using it in
pg_dump as well as psql.  Since psql already uses dumputils.c, while there's
not any code sharing in the other direction, this seems the easiest way.
Also, fix misinterpretation of patterns using regex | by adding parentheses
(same bug found previously in similar_escape()).  This should be backpatched.

Update release notes for releases 7.3.16, 7.4.14, 8.0.9, and 8.1.5.

Updated keyword table for 8.2

Update Darwin dlopen() support to avoid deprecation warnings with latest
Apple developer tools.  We now use dlopen directly if available, and fall
back to the older code if not.  Chris Campbell

Update multi-line editing wording in release notes.

In release notes, mention that multi-line history saving does not work
on Win32.

Add include needed for new getrusage() call.

On Windows, we know the backend stack size limit because we have to
specify it explicitly in backend/Makefile.  Arrange for this value to
be known by get_stack_depth_rlimit() too.  Per suggestion from Magnus.

Update comments for pgcvslog, -r BASE does not work with cvs log.

Simplify <link> regex cleanup code.

Allow HISTORY/history.html to be generated by removing <link> elements,
per suggestion from Peter.  Without this, the main doc links prevent
those files from being built.

Fix ancient oversight in psql's \d pattern processing code: when seeing two
quote chars inside quote marks, should emit one quote *and stay in inquotes
mode*.  No doubt the lack of reports of this have something to do with the
poor documentation of the feature ...

In release notes, always use </link> rather than </>.

Added missing entry (CASCADED) in keywords table.

Fix thinko in comment.

Updated config.guess and config.sub

The -X option in pg_dump was supposed to be a workaround for the lack of
portable long options.  But we have had portable long options for a long
time now, so this is obsolete.  Now people have added options which *only*
work with -X but not as regular long option, so I'm putting a stop to this:
-X is deprecated; it still works, but it has been removed from the
documentation, and please don't add more of them.

When planning a query at Bind time, be careful to pass the correct
query_list into the Portal, ie, the one seen and possibly modified by
the planner.  My fault :-(  Per report from Sergey Koposov.

On platforms that have getrlimit(RLIMIT_STACK), use it to ensure that
max_stack_depth is not set to an unsafe value.

This commit also provides configure-time checking for <sys/resource.h>,
and cleans up some perhaps-unportable code associated with use of that
include file and getrlimit().

Adjust HINT for stack depth limit to mention checking the underlying
platform limit, rather than just blindly raising max_stack_depth.
Also, tweak the code to work properly if someone sets max_stack_depth
to more than 2Gb, which guc.c will allow on a 64-bit machine.

Remove tabs added accidentally.

Fix a typo.

Add main documentation link tags to the release notes.

Fix string_to_array() to correctly handle the case where there are
overlapping possible matches for the separator string, such as
string_to_array('123xx456xxx789', 'xx').
Also, revise the logic of replace(), split_part(), and string_to_array()
to avoid O(N^2) work from redundant searches and conversions to pg_wchar
format when there are N matches to the separator string.
Backpatched the full patch as far as 8.0.  7.4 also has the bug, but the
code has diverged a lot, so I just went for a quick-and-dirty fix of the
bug itself in that branch.

Fix bcc32.mak to create and remove pg_config_os.h properly.  Apparently
win32.mak got patched for this, but not bcc32.mak.

Fix SysCacheGetAttr() to handle the case where the specified syscache has not
been initialized yet.  This can happen because there are code paths that call
SysCacheGetAttr() on a tuple originally fetched from a different syscache
(hopefully on the same catalog) than the one specified in the call.  It
doesn't seem useful or robust to try to prevent that from happening, so just
improve the function to cope instead.  Per bug#2678 from Jeff Trout.  The
specific example shown by Jeff is new in 8.1, but to be on the safe side
I'm backpatching 8.0 as well.  We could patch 7.x similarly but I think
that's probably overkill, given the lack of evidence of old bugs of this ilk.

Message style improvements

Cleanup for pglz_compress code: remove dead code, const-ify API of
remaining functions, simplify pglz_compress's API to not require a useless
data copy when compression fails.  Also add a check in pglz_decompress that
the expected amount of data was decompressed.

Make use of qsort_arg in several places that were formerly using klugy
static variables.  This avoids any risk of potential non-reentrancy,
and in particular offers a much cleaner workaround for the Intel compiler
bug that was affecting ginutil.c.

Add indenting to Solaris FAQ, for clarity.

Update DB2X texinfo example code.

Andreas Seltenreich

Work around buggy strtod on (some versions of?) IRIX.  Combination of
proposed patches from John Jorgensen and Steve Singer.

Second try at fixing libedit configuration for Bob Friesenhahn.
Buildfarm results from 'gazelle' show that there are indeed libedit
versions for which history.h is a needed header, even though it's
apparently been dropped entirely in other versions.  Grumble.

Update Solaris strtod() bug comment.

Update Solaris strtol() bug comment.

Adjust configure's handling of libedit to not look for a history.h file.
Per Bob Friesenhahn's report, this file is not supplied by some versions
of libedit, and even when it is supplied it seems to be just a link to
readline.h, so we don't need to include it anyway.
Also, ensure that we won't try to use a too-old version of Bison.
The previous coding would bleat but then use it anyway; better to invoke
the 'missing' script if any grammar files need to be rebuilt.

Update Solaris FAQ.

Zdenek Kotala

Exclude pgindent from affecting the ecpg regression directory.

Fix initdb's logic for picking shared_buffers/max_fsm_pages, which had
gotten rather thoroughly whacked out by careless recent changes: the
intended ratio between the two was off by a lot, and the minimum number
of shared buffers tried had increased by a lot.  Problem exposed by
failures on buildfarm members with smaller SHMMAX values.

Fix comment to agree with presentation.

Improve error messages from to_tsquery per yesterday's discussion:
provide the bad input, and be sure to mention that we are talking about
a tsearch query.

Add qsort and qsort_arg to MSVC list of src/port files.

Reverted changes made by pgindent
Made show.pgc set all variables before displaying them.

pgindent run for 8.2.

Udpate typedefs for pgindent.

Switch over to using our own qsort() all the time, as has been proposed
repeatedly.  Now that we don't have to worry about memory leaks from
glibc's qsort, we can safely put CHECK_FOR_INTERRUPTS into the tuplesort
comparators, as was requested a couple months ago.  Also, get rid of
non-reentrancy and an extra level of function call in tuplesort.c by
providing a variant qsort_arg() API that passes an extra void * argument
through to the comparison routine.  (We might want to use that in other
places too, I didn't look yet.)

pgevent fixes:

1) Make vcbuild actually build the pgevent dll.
2) Change the pgevent DLL file so it doens't specify ordinal for the
functions. You're not supposed to do that. You're actually supposed to
declare them as PRIVATE as well, but mingw doesn't support that. VC++
will throw a warning and not an error though, so we can live with it.

Magnus Hagander

Update typedef list for 8.2 pgindent run.

Fix yesno_prompt() memory leak in /script tools, reported by Converity.

Improve numeric overflow error message.

David Fetter

Make some sentences consistent with similar ones.

Euler Taveira de Oliveira

Improve readline psql prompt suggestion.

Andreas Seltenreich

Add experimental support for db2x texinfo

Andreas Seltenreich

The attached patch changes units of the some default values in
postgresql.conf.

- shared_buffers = 32000kB => 32MB
- temp_buffers = 8000kB => 8MB
- wal_buffers = 8 => 64kB

The code of initdb was a bit modified to write MB-unit values.
Values greater than 8000kB are rounded out to MB.

GUC_UNIT_XBLOCKS is added for wal_buffers. It is like GUC_UNIT_BLOCKS,
but uses XLOG_BLCKSZ instead of BLCKSZ.

Also, I cleaned up the test of GUC_UNIT_* flags in preparation to
add more unit flags in less bits.

ITAGAKI Takahiro

Cleanup pgwin32_open() 'if' test, and avoid possible error.

Rearrange MSVC errcode hack, fix incorrect _MSC_VER test.  Magnus

Move WIN32_ONLY_COMPILER define from c.h to win32.h because it was being
defined too late.

This patch fixes a small typo in information_schema.sgml.

Guillaume Lelarge

Mention in postgresql.conf that autovacuum also needs
stats_start_collector and stats_row_level to also be on

David Wheeler

Add URL for strlcpy():

>
>   http://archives.postgresql.org/pgsql-hackers/2006-09/msg02108.php

Add:

>  * Fix SSL retry to avoid useless repeated connection attempts and
>    ensuing misleading error messages

Add URL about strlcpy() creation history.

Update wording:

< * Use strlcpy() rather than StrNCpy() macro
> * Use strlcpy() rather than our StrNCpy() macro

Add:

>
> * Use strlcpy() rather than StrNCpy() macro

Remove mention of the -fast option from the Solaris FAQ.

Kenneth Marshall

Update recovery documentation.

Simon Riggs

Update tsearch2 README.

Robert Treat

Add URL for cached copy of paper as SGML comment.

Add release note item:

        Drop privileges on startup so servers can be started from
        an administrative account (Magnus)

Move "Exotic Features" to the bottom of TODO.

Properly handle the case where strlcpy() exists in libc but isn't
declared in the system headers.  Per report from Bruce than some BSDen
are like this.

Improve documentation of configure's readline/libedit switches.

Make some marginal performance improvements in reportErrorPosition(),
which turns out to be a dominant part of the runtime in scenarios
involving lots of parse-time warnings (such as Stephen Frost's example
of an INSERT with a lot of backslash-containing strings).  There's not
a whole lot we can do about the character-at-a-time scanning, but we
can at least avoid traversing the query twice.