--enable-debug is used, to avoid complaints about debugging and
optimization being mutually exclusive. Patch from Stefan Kaltenbrunner.

name.  With this patch, it is always possible for the user to qualify a
plpgsql variable name if needed to avoid ambiguity.  While there is much more
work to be done in this area, this simple change removes one unnecessary
incompatibility with Oracle.  Per discussion.

int not unsigned int.  Third try to get grebe building without warnings...

spaces in them.

ISHIDA Akio

nattering about casting away volatile.  Losers.

TAS support are properly declared.

for 'bool'.  Per buildfarm warnings.

a warning but was outright wrong.

error messages look at least a little bit like the message style
guidelines say.

not-too-bright compilers.  Per buildfarm results.

three slightly different copies of this file?)

theoretically vary depending on what the compile-time locale setting is.
Hence, force it to see LC_CTYPE=C to ensure consistent build results.
(It's likely that this makes no difference in practice, since our
specification for "identifier" surely includes both ends of any possible
uppercase/lowercase pair anyway.  But it should silence warnings about
ambiguous character classes that are reported by some buildfarm members.)

sanely if the loop value overflows int32 on the way to the end value.
Avoid useless computation of "SELECT 1" when BY is omitted.  Avoid some
type-punning between Datum and int4 that dates from the original coding.

Andrew Dunstan.  Minor other improvements in documentation of integer
FOR loops.

of variable substitution and plan caching behavior in dedicated sections.
(A lot of this material existed already, but was scattered in various places
in the chapter.)  Reorganize material a little bit, mostly to try to avoid
diving into deep details in the first introductory sections.  Document some
fine points that had escaped treatment before, notably the ability to qualify
plpgsql variable names with block labels.  Some minor wordsmithing here and
there.

by Solaris 10 and possibly others.

Stefan Kaltenbrunner

from old versions of gcc.  It's not clear to me that this is really
necessary for correctness, but less warnings are always good.
Per buildfarm results and local testing.

columns, per my gripe earlier today.  Make it look a bit less like
someone's first effort at backend coding.

unsigned char).  Fortunately we still have buildfarm machines that
will flag this.  Seems to be new in CVS HEAD, so no backpatch.

on Cygwin (answer: we don't).  Also try to unwind the #ifdef spaghetti
a little bit.  Untested but hopefully I didn't break anything.

add dashes to the start/end of the comment block to try to prevent
this happening in the future.

may or may not be harmless.  Report from Stefan, patch from Heikki.

define pg_dlsym() as returning a PGFunction pointer, not just any
pointer-to-function.  But many are not.  Suppress compiler warnings
on platforms that aren't careful by inserting explicit casts at the
two call sites that didn't have a cast already.  Per Stefan.

suppressing Sun Studio compiler warnings.  Per Stefan.

when built on a 64-bit machine.  Per buildfarm results extracted by Stefan.

enabled, because the only Kerberos library supported always contains it.

such as OpenBSD (possibly all Heimdal).

Stefan Kaltenbrunner

free function provided in the Kerberos library.
This fixes a very hard to track down heap corruption on windows
when using debug runtimes.

Per suggestion from Alvaro.