pgsql: Modify tuptoaster's API so that it does not try to modify the

[tgl(at)svr1(dot)postgresql(dot)org (Tom Lane)]

Log Message:
-----------
Modify tuptoaster's API so that it does not try to modify the passed
tuple in-place, but instead passes back an all-new tuple structure if
any changes are needed.  This is a much cleaner and more robust solution
for the bug discovered by Alexey Beschiokov; accordingly, revert the
quick hack I installed yesterday.
With this change, HeapTupleData.t_datamcxt is no longer needed; will
remove it in a separate commit in HEAD only.

Tags:
----
REL8_1_STABLE

Modified Files:
--------------
    pgsql/src/backend/access/heap:
        heapam.c (r1.200 -> r1.200.2.1)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/access/heap/heapam.c.diff?r1=1.200&r2=1.200.2.1)
        tuptoaster.c (r1.53 -> r1.53.2.1)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/access/heap/tuptoaster.c.diff?r1=1.53&r2=1.53.2.1)
    pgsql/src/backend/executor:
        execMain.c (r1.256.2.2 -> r1.256.2.3)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/backend/executor/execMain.c.diff?r1=1.256.2.2&r2=1.256.2.3)
    pgsql/src/include/access:
        tuptoaster.h (r1.23 -> r1.23.2.1)
        (http://developer.postgresql.org/cvsweb.cgi/pgsql/src/include/access/tuptoaster.h.diff?r1=1.23&r2=1.23.2.1)