Remove 'Array' node type, which has evidently been dead code for
authorTom Lane
Sat, 22 Jul 2000 04:22:47 +0000 (04:22 +0000)
committerTom Lane
Sat, 22 Jul 2000 04:22:47 +0000 (04:22 +0000)
a very long time.

src/backend/nodes/copyfuncs.c
src/backend/nodes/equalfuncs.c
src/backend/nodes/outfuncs.c
src/backend/nodes/readfuncs.c
src/include/nodes/nodes.h
src/include/nodes/primnodes.h

index 4013a0f77b24d34268ce79c81fa8d075a3e13ab5..2def370e9fb36d432514138652937de5f616ae34 100644 (file)
@@ -19,7 +19,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/nodes/copyfuncs.c,v 1.117 2000/07/17 03:04:58 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/nodes/copyfuncs.c,v 1.118 2000/07/22 04:22:46 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -933,26 +933,6 @@ _copyCaseWhen(CaseWhen *from)
    return newnode;
 }
 
-static Array *
-_copyArray(Array *from)
-{
-   Array      *newnode = makeNode(Array);
-
-   /* ----------------
-    *  copy remainder of node
-    * ----------------
-    */
-   newnode->arrayelemtype = from->arrayelemtype;
-   newnode->arrayelemlength = from->arrayelemlength;
-   newnode->arrayelembyval = from->arrayelembyval;
-   newnode->arrayndim = from->arrayndim;
-   newnode->arraylow = from->arraylow;
-   newnode->arrayhigh = from->arrayhigh;
-   newnode->arraylen = from->arraylen;
-
-   return newnode;
-}
-
 static ArrayRef *
 _copyArrayRef(ArrayRef *from)
 {
@@ -1724,9 +1704,6 @@ copyObject(void *from)
        case T_Func:
            retval = _copyFunc(from);
            break;
-       case T_Array:
-           retval = _copyArray(from);
-           break;
        case T_ArrayRef:
            retval = _copyArrayRef(from);
            break;
index b1772e6436cd829c88b962fe9d5164dd2abcb7b8..b85c410c253fbec3564983078de0d8eec25735bd 100644 (file)
@@ -24,7 +24,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/nodes/equalfuncs.c,v 1.69 2000/07/17 03:05:01 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/nodes/equalfuncs.c,v 1.70 2000/07/22 04:22:46 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -293,25 +293,6 @@ _equalRelabelType(RelabelType *a, RelabelType *b)
    return true;
 }
 
-static bool
-_equalArray(Array *a, Array *b)
-{
-   if (a->arrayelemtype != b->arrayelemtype)
-       return false;
-   /* We need not check arrayelemlength, arrayelembyval if types match */
-   if (a->arrayndim != b->arrayndim)
-       return false;
-   /* XXX shouldn't we be checking all indices??? */
-   if (a->arraylow.indx[0] != b->arraylow.indx[0])
-       return false;
-   if (a->arrayhigh.indx[0] != b->arrayhigh.indx[0])
-       return false;
-   if (a->arraylen != b->arraylen)
-       return false;
-
-   return true;
-}
-
 static bool
 _equalArrayRef(ArrayRef *a, ArrayRef *b)
 {
@@ -800,9 +781,6 @@ equal(void *a, void *b)
        case T_Func:
            retval = _equalFunc(a, b);
            break;
-       case T_Array:
-           retval = _equalArray(a, b);
-           break;
        case T_ArrayRef:
            retval = _equalArrayRef(a, b);
            break;
index c561ad5126841f994012f5f69799570a5117638c..155aae37ad2f7026c74455d3e4a9a24d6466310f 100644 (file)
@@ -6,7 +6,7 @@
  * Portions Copyright (c) 1996-2000, PostgreSQL, Inc
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Header: /cvsroot/pgsql/src/backend/nodes/outfuncs.c,v 1.123 2000/07/17 03:05:01 tgl Exp $
+ * $Header: /cvsroot/pgsql/src/backend/nodes/outfuncs.c,v 1.124 2000/07/22 04:22:46 tgl Exp $
  *
  * NOTES
  *   Every (plan) node in POSTGRES has an associated "out" routine which
@@ -770,29 +770,6 @@ _outRelabelType(StringInfo str, RelabelType *node)
                     node->resulttype, node->resulttypmod);
 }
 
-/*
- * Array is a subclass of Expr
- */
-static void
-_outArray(StringInfo str, Array *node)
-{
-   int         i;
-
-   appendStringInfo(str,
-     " ARRAY :arrayelemtype %u :arrayelemlength %d :arrayelembyval %c ",
-                    node->arrayelemtype,
-                    node->arrayelemlength,
-                    node->arrayelembyval ? 't' : 'f');
-
-   appendStringInfo(str, " :arrayndim %d :arraylow ", node->arrayndim);
-   for (i = 0; i < node->arrayndim; i++)
-       appendStringInfo(str, " %d ", node->arraylow.indx[i]);
-   appendStringInfo(str, " :arrayhigh ");
-   for (i = 0; i < node->arrayndim; i++)
-       appendStringInfo(str, " %d ", node->arrayhigh.indx[i]);
-   appendStringInfo(str, " :arraylen %d ", node->arraylen);
-}
-
 /*
  * ArrayRef is a subclass of Expr
  */
@@ -1508,9 +1485,6 @@ _outNode(StringInfo str, void *obj)
            case T_RelabelType:
                _outRelabelType(str, obj);
                break;
-           case T_Array:
-               _outArray(str, obj);
-               break;
            case T_ArrayRef:
                _outArrayRef(str, obj);
                break;
index b9916ce6b06ee6e8352d7448cde0a7f29a8a3780..c663ba304fc285c591b5a17b3a014666ce5783db 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/nodes/readfuncs.c,v 1.93 2000/07/17 03:05:01 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/nodes/readfuncs.c,v 1.94 2000/07/22 04:22:46 tgl Exp $
  *
  * NOTES
  *   Most of the read functions for plan nodes are tested. (In fact, they
@@ -814,48 +814,6 @@ _readVar()
    return local_node;
 }
 
-/* ----------------
- * _readArray
- *
- * Array is a subclass of Expr
- * ----------------
- */
-static Array *
-_readArray()
-{
-   Array      *local_node;
-   char       *token;
-   int         length;
-
-   local_node = makeNode(Array);
-
-   token = lsptok(NULL, &length);      /* eat :arrayelemtype */
-   token = lsptok(NULL, &length);      /* get arrayelemtype */
-   local_node->arrayelemtype = strtoul(token, NULL, 10);
-
-   token = lsptok(NULL, &length);      /* eat :arrayelemlength */
-   token = lsptok(NULL, &length);      /* get arrayelemlength */
-   local_node->arrayelemlength = atoi(token);
-
-   token = lsptok(NULL, &length);      /* eat :arrayelembyval */
-   token = lsptok(NULL, &length);      /* get arrayelembyval */
-   local_node->arrayelembyval = (token[0] == 't') ? true : false;
-
-   token = lsptok(NULL, &length);      /* eat :arraylow */
-   token = lsptok(NULL, &length);      /* get arraylow */
-   local_node->arraylow.indx[0] = atoi(token);
-
-   token = lsptok(NULL, &length);      /* eat :arrayhigh */
-   token = lsptok(NULL, &length);      /* get arrayhigh */
-   local_node->arrayhigh.indx[0] = atoi(token);
-
-   token = lsptok(NULL, &length);      /* eat :arraylen */
-   token = lsptok(NULL, &length);      /* get arraylen */
-   local_node->arraylen = atoi(token);
-
-   return local_node;
-}
-
 /* ----------------
  * _readArrayRef
  *
@@ -1835,8 +1793,6 @@ parsePlanString(void)
        return_value = _readExpr();
    else if (length == 8 && strncmp(token, "ARRAYREF", length) == 0)
        return_value = _readArrayRef();
-   else if (length == 5 && strncmp(token, "ARRAY", length) == 0)
-       return_value = _readArray();
    else if (length == 3 && strncmp(token, "VAR", length) == 0)
        return_value = _readVar();
    else if (length == 4 && strncmp(token, "ATTR", length) == 0)
index f919221b637fb54f32b6c706418cbb8c38d19f12..08a44675fc8c9d03758bd10dda9181256dec9504 100644 (file)
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2000, PostgreSQL, Inc
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Id: nodes.h,v 1.71 2000/07/14 15:43:51 thomas Exp $
+ * $Id: nodes.h,v 1.72 2000/07/22 04:22:47 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -64,7 +64,7 @@ typedef enum NodeTag
    T_Aggref,
    T_SubLink,
    T_Func,
-   T_Array,
+   T_ArrayXXX,                 /* not used anymore; this tag# is available */
    T_ArrayRef,
    T_Iter,
    T_RelabelType,
index 2cf59ca50c21641c134ce31be436f9337990a1ae..47de5779ae57f1e7f23016615bacccaf8ffc89f8 100644 (file)
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1996-2000, PostgreSQL, Inc
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- * $Id: primnodes.h,v 1.44 2000/07/17 03:05:27 tgl Exp $
+ * $Id: primnodes.h,v 1.45 2000/07/22 04:22:47 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -389,33 +389,6 @@ typedef struct SubLink
    Node       *subselect;
 } SubLink;
 
-/* ----------------
- * Array
- *     arrayelemtype   - type of the array's elements (homogenous!)
- *     arrayelemlength - length of that type
- *     arrayelembyval  - is the element type pass-by-value?
- *     arrayndim       - number of dimensions of the array
- *     arraylow        - base for array indexing
- *     arrayhigh       - limit for array indexing
- *     arraylen        - total length of array object
- * ----------------
- *
- * memo from mao:  the array support we inherited from 3.1 is just
- * wrong.  when time exists, we should redesign this stuff to get
- * around a bunch of unfortunate implementation decisions made there.
- */
-typedef struct Array
-{
-   NodeTag     type;
-   Oid         arrayelemtype;
-   int         arrayelemlength;
-   bool        arrayelembyval;
-   int         arrayndim;
-   IntArray    arraylow;
-   IntArray    arrayhigh;
-   int         arraylen;
-} Array;
-
 /* ----------------
  * ArrayRef: describes an array subscripting operation
  *
@@ -423,11 +396,12 @@ typedef struct Array
  * fetching a subarray (array slice), storing a single element into
  * an array, or storing a slice.  The "store" cases work with an
  * initial array value and a source value that is inserted into the
- * appropriate part of the array.
+ * appropriate part of the array; the result of the operation is an
+ * entire new modified array value.
  *
- *     refattrlength   - total length of array object
- *     refelemtype     - type of the result of the subscript operation
- *     refelemlength   - length of the array element type
+ *     refattrlength   - typlen of array type
+ *     refelemtype     - type of the result of the ArrayRef operation
+ *     refelemlength   - typlen of the array element type
  *     refelembyval    - is the element type pass-by-value?
  *     refupperindexpr - expressions that evaluate to upper array indexes
  *     reflowerindexpr - expressions that evaluate to lower array indexes
@@ -449,7 +423,7 @@ typedef struct Array
  * Note: currently, refelemtype is NOT the element type, but the array type,
  * when doing subarray fetch or either type of store.  It would be cleaner
  * to add more fields so we can distinguish the array element type from the
- * result type of the subscript operator...
+ * result type of the ArrayRef operator...
  * ----------------
  */
 typedef struct ArrayRef