Commit a newer fact for 1.09 from Andrew

Fixes:

I found another bug in btree index.  Looking at the code it seems that NULL
keys are never used to build or scan a btree index (see the explain commands
in the example).  However this is not the case when a null key is retrieved
in an outer loop of a join select and used in an index scan of an inner loop.
This bug causes at least three kinds of problems:

1)  the backend crashes when it tries to compare a text string with a null.

2)  it is not possible to find tuples with null keys in a join.

3)  null is considered equal to 0 when the datum is passed by value, see
    the last query.

Submitted by: Massimo Dal Zotto <dz@cs.unitn.it>

Fix up INSTALL file

From Andrew

Parser Overhaul

Fix small syntax error.

Fix sigaction again to try for portability.

Remove another snprintf() for Solaris.

Slight changes to INSTALL to point ppl at the Linux/IRIX specific
FAQs

Add in Andrew's FAQs

Use strncpy() and local buffers instead of snprintf(), since not everyone
has snprintf().

Make a local isblank() function, since not all standard C libraries have it.
For sparc_solaris, add inet_aton prototype, since it isn't in the header
files included for the standard C library functions.

Add our own copy of inet_aton() for sparc_solaris, which doesn't have it in
the standard C library.

Simplify make files, add full dependencies.

Correct #if statement so it compiles on Solaris.  Thanks Keith Parks.

D'Arcy's cleanups

Make sure the btree patch gets into 2.0 as well...

Still submitted by:  Massimo Dal Zotto <dz@cs.unitn.it>

Add CFLAGS+=-DPOSTPORT=... so we can take it out of the global CFLAGS.

Use EUROPEAN_DATES instead of EUROPEAN_STYLE

E Use EUROPEAN_DATES from config.h instead of -D option

Fixes: Growing backend when using nested function calls
Submitted by: wieck@sapserv.debis.de (Jan Wieck)

Fixes: Errors when PQexec() in backend creates temp
              relations and transaction is aborted

Submitted by: wieck@sapserv.debis.de (Jan Wieck)

Fixes:

It's bug in nodeAgg.c on lines 241, 242:

                null_array = malloc(nagg);
                for (i=0;i<nagg;i++)
                    null_array[i] = 'n';
                oneTuple = heap_formtuple(tupType, tupValue, null_array);

- your query has not only aggregates but also 'group by-ed' fields and so
null_array should contain tupType->natts elements (tupType->natts > nagg in
your case).

Patch follows and it's very simple.

VAdim

Take out the PERFECT_MMGR #ifdefs:

My guess is that the thing had bugs, and the pfree was commented out.
The thing is probabally free'ed anyway at the end, so it was not a bad
thing.

If it does cause a bug, it will generate an error when hit, so I say
unless someone else knows, let's remove it and run the regression test.

-Bruce

Bruce...does this fix it once and for all? *grin*

Major code cleanups from D'arcy (-Wall -Werror)

|I have moved things around so that things happen in the following order:
|
|  1. Set default variables
|  2. Include Makefile.custom to override defaults
|  3. Set CFLAGS, etc. with variables
|
|This fixes the problem of Makefile.custom changes not taking effect.

Submitted by: D'Arcy Cain

Update "planner" directory to "optimizer" in include paths in this dead code.

Cosmetic changes to ordering of #include files

-Wall'd

-Wall'd

-Wall'd ... I missed a subdir under access :(

Make line 159:bp = NULL, to quiet compiler

Suggested by: Randy Terbush <randy@zyzzyva.com>

-Wall'd

That finishes access/*

Force user to set PORTNAME.  Remove NAMEDATALEN from CFLAGS (don't need it
there because it's in config.h, which is cleaner).

-Wall'd

-Wall'd

-Wall Cleanup of gist subdirectory

-Wall cleanup of gist subdirectory:
- cleaned out unused variables
- added missing prototype headers

One outstanding -Werror failure:

/home/staff/scrappy/postgres/2.0/cvs/postgres95/src/backend/access/gist/giststra
t.c:117: warning: missing braces around initializer for `GISTEvaluationData.expr
ession'

Define prototypes from giststrat.c required in gistscan.c

Other then:

indextuple.c:159: warning: `bp' might be used uninitialized in this function

this directory passes -Wall -Werror under FreeBSD

remove an #ifdef/#else/#endif that doesn't actually have anything inside
of it

Make default HAVE_MEMMOVE defined, undef inside of definitions for sparc

change '#if defined(PORTNAME_sparc)' to '#if !defined(HAVE_MEMMOVE)'

Add #define HAVE_MEMMOVE for systems that do have it.

Clean up comments (-Wall -Werror)

#include file cleanup

More #include cleanups

Once access/* is cleaned out, will redo using -Wall on compile to
make sure that all prototyping is correct

First pass at fixing my own mistakes

Mainly...fix up the includes I removed, as well as prototypes

Pointed out by D'Arcy

Another directory cleaned out

More include files cleaned up

Okay...another directory cleaned out and compilable

Add HAVE_Cplusplus= true to BSD44_derived Makefile.inc, as I know that
FreeBSD/NetBSD both ship with gcc 2.7.x

Not quite the cleanest way to do it, but add an option to Makefile.global
that allows one to compile libpq++

this should be set on a port-by-port basis

Welp, another subdirectory cleaned out of redundant/unused #include
files

There...that pretty much cleans up redundant/unused #includes in
access/common...how many more directories to go? :)

Almost have access/common cleaned up...

A few more yet...

wish there was an easier way to clean this up...

another small one

Hopefully that cleans up the cruft from rel.h

another...

its coming...

step by step...file by file...

...same

slow work...

and another

another one cleaned up

and another one..

another another one

anothre one reduced to its component parts

clean out include files of including other includes files except those
absolutely required

Another cleaned up

One at a time...another one is complete...

another one with #include file  clean'd up

All #include's removed from *.h files, so cleaning up the .c #includes...

First file of, what...1000's?

#include "postgres.h" is a standard include for all .c files, auto-including
config.h, c.h *and* elog.h...added elog.h

Add prototype for heap_getattr() to quiet compiler

Reap deleted/unused tuples on page-base

1. Reap deleted/unused tuples on page-base
2. Reap unused tuples too
3. Reap empty pages
4. Check if a page is initialized, initialize it if not
   and reap it
5. Binary search in list of reapped pages/tids to check
   is the heap' tid pointed by a index' tuple on this list
   (it's mu-u-uch faster)

Move #define PageIsNew (true if page is not initialize) to bufpage.h

Move definition of PageIsNew from hio.c to here

Document some more #ifdef's

Document more #ifdef's into config.h

Get rid of ESCAPE_PATCH ifdef, as its on by default, and there is no
apparent reason for turning it off...it fixes a bug

Moved definition of MAXPATHLEN to config.h

Added a #define NEED_SIG_JMP to config.h

Removed #ifdef's for OPENLINK_PATCH.  We enable it by default, and nobody
*has* to enable the functionality, but no reason to retain the "old code"

Document #ifdef's from backend/access subdirs so that we know what
features(?) are available

Moved '#define *DEBUG' defines to config.h, and document where first
found.

Document any '#ifdef' segments found in config.h

Moved '#define *DEBUG' defines to config.h, and document where first
found.

Document any '#ifdef' segments found in config.h

Remove gistold.c, since it was older copy of gist.c

#define cleanup: moved #define IPORTAL_DEBUG to config.h

Moved all the CFLAG -D's from the makefiles to config.h towards better
centralized configurations/porting...

Moved more -D's out of the port specific Makefiles straight into the
config.h file

Centralize more of the defines from backend/port under config.h

Further ports should *really* avoid backend/port and make use of
config.h

Move "port specific" #ifdefs out of here and into config.h
- created a HAVE_TZSET define for this purpose

Add a HAVE_TZSET define to clean up defines in backend/utils/adt/nabstime.c

Change treatment of sigaction structure again to include even more platforms.
Thanks D'Arcy.

Remove 4th element of sigaction structure so it works on more platforms.

New FAQ's.

More closing of relations left open by parser.

Change display of char(5) to (bp)char 5 so psql \d can
distinguish between char and char(1).

Removed const warning.