diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2000-12-27 23:59:14 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2000-12-27 23:59:14 +0000 |
| commit | 8609d4abf248f0eede4ed9505226da3f7e3e7c84 (patch) | |
| tree | 39e5a813099835056d76dd385478069d01a8dcbf /src/backend/executor/execUtils.c | |
| parent | 97799fc4757fd2699e0238254875994253659582 (diff) | |
| download | postgresql-8609d4abf248f0eede4ed9505226da3f7e3e7c84.tar.gz postgresql-8609d4abf248f0eede4ed9505226da3f7e3e7c84.zip | |
Fix portability problems recently exposed by regression tests on Alphas.
1. Distinguish cases where a Datum representing a tuple datatype is an OID
from cases where it is a pointer to TupleTableSlot, and make sure we use
the right typlen in each case.
2. Make fetchatt() and related code support 8-byte by-value datatypes on
machines where Datum is 8 bytes. Centralize knowledge of the available
by-value datatype sizes in two macros in tupmacs.h, so that this will be
easier if we ever have to do it again.
Diffstat (limited to 'src/backend/executor/execUtils.c')
| -rw-r--r-- | src/backend/executor/execUtils.c | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/src/backend/executor/execUtils.c b/src/backend/executor/execUtils.c index 3b05a78e83a..5d4d7f145b3 100644 --- a/src/backend/executor/execUtils.c +++ b/src/backend/executor/execUtils.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/executor/execUtils.c,v 1.69 2000/11/16 22:30:20 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/executor/execUtils.c,v 1.70 2000/12/27 23:59:11 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -274,16 +274,10 @@ ExecAssignResultTypeFromTL(Plan *node, CommonState *commonstate) { List *targetList; TupleDesc tupDesc; - int len; targetList = node->targetlist; tupDesc = ExecTypeFromTL(targetList); - len = ExecTargetListLength(targetList); - - if (len > 0) - ExecAssignResultType(commonstate, tupDesc); - else - ExecAssignResultType(commonstate, (TupleDesc) NULL); + ExecAssignResultType(commonstate, tupDesc); } /* ---------------- @@ -582,8 +576,8 @@ ExecSetTypeInfo(int index, } /* ---------------- - * ExecFreeTypeInfo frees the array of attrbutes - * created by ExecMakeTypeInfo and returned by ExecTypeFromTL... + * ExecFreeTypeInfo frees the array of attributes + * created by ExecMakeTypeInfo and returned by ExecTypeFromTL * ---------------- */ void |