Cleanup Name usage.

diff --git a/src/backend/access/gist/gist.c b/src/backend/access/gist/gist.c
index dc2b57d4df2ede8e36753b35401cf60f635bbd92..efa5286c1f44ed81a2a7ee873e7b5e3db33401e8 100644 (file)
--- a/src/backend/access/gist/gist.c
+++ b/src/backend/access/gist/gist.c
@@ -130,7 +130,7 @@ gistbuild(Relation heap,
         */
 
        if (oldPred == NULL && (nb = RelationGetNumberOfBlocks(index)) != 0)
-               elog(ERROR, "%.16s already contains data", &(index->rd_rel->relname.data[0]));
+               elog(ERROR, "%s already contains data", index->rd_rel->relname.data);
 
        /* initialize the root page (if this is a new index) */
        if (oldPred == NULL)

diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c
index 78fe6b22a43edde6941402aaa65eb2ce65a36d5d..1dcd282d6c4e32a7be108e15b7ea90568627211f 100644 (file)
--- a/src/backend/executor/execMain.c
+++ b/src/backend/executor/execMain.c
@@ -26,7 +26,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/executor/execMain.c,v 1.49 1998/07/19 03:46:29 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/executor/execMain.c,v 1.50 1998/07/20 16:14:13 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -606,16 +606,6 @@ InitPlan(CmdType operation, Query *parseTree, Plan *plan, EState *estate)
 
        estate->es_into_relation_descriptor = intoRelationDesc;
 
-       /* ----------------
-        *      return the type information..
-        * ----------------
-        */
-/*
-       attinfo = (AttrInfo *)palloc(sizeof(AttrInfo));
-       attinfo->numAttr = len;
-       attinfo->attrs = tupType->attrs;
-*/
-
        return tupType;
 }
 

diff --git a/src/backend/executor/execUtils.c b/src/backend/executor/execUtils.c
index 1db49d391c7b6ef3a17ad7e61453ce013b9ca311..0502036f9a12de48ff57c72fcd58773de26c5ae4 100644 (file)
--- a/src/backend/executor/execUtils.c
+++ b/src/backend/executor/execUtils.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/executor/execUtils.c,v 1.32 1998/06/15 19:28:20 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/executor/execUtils.c,v 1.33 1998/07/20 16:14:13 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -309,8 +309,7 @@ ExecFreeResultType(CommonState *commonstate)
        slot = commonstate->cs_ResultTupleSlot;
        tupType = slot->ttc_tupleDescriptor;
 
-/*       ExecFreeTypeInfo(tupType); */
-       pfree(tupType);
+       ExecFreeTypeInfo(tupType);
 }
 
 #endif
@@ -407,8 +406,7 @@ ExecFreeScanType(CommonScanState *csstate)
        slot = csstate->css_ScanTupleSlot;
        tupType = slot->ttc_tupleDescriptor;
 
-/*       ExecFreeTypeInfo(tupType); */
-       pfree(tupType);
+       ExecFreeTypeInfo(tupType);
 }
 
 #endif

diff --git a/src/backend/storage/smgr/md.c b/src/backend/storage/smgr/md.c
index 2545523850d3864db4bbbd86486b3863d8023a37..30246281ebbc669edcc0de8c31dddbe4b3744eb0 100644 (file)
--- a/src/backend/storage/smgr/md.c
+++ b/src/backend/storage/smgr/md.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/storage/smgr/md.c,v 1.31 1998/06/15 19:29:23 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/storage/smgr/md.c,v 1.32 1998/07/20 16:14:14 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -139,7 +139,7 @@ mdcreate(Relation reln)
                                vfd;
        char       *path;
 
-       path = relpath(&(reln->rd_rel->relname.data[0]));
+       path = relpath(reln->rd_rel->relname.data);
        fd = FileNameOpenFile(path, O_RDWR | O_CREAT | O_EXCL, 0600);
 
        /*
@@ -288,7 +288,7 @@ mdopen(Relation reln)
        int                     fd;
        int                     vfd;
 
-       path = relpath(&(reln->rd_rel->relname.data[0]));
+       path = relpath(reln->rd_rel->relname.data);
 
        fd = FileNameOpenFile(path, O_RDWR, 0600);
 
@@ -718,7 +718,7 @@ mdtruncate(Relation reln, int nblocks)
        if (curnblk / RELSEG_SIZE > 0)
        {
                elog(NOTICE, "Can't truncate multi-segments relation %s",
-                        &(reln->rd_rel->relname.data[0]));
+                        reln->rd_rel->relname.data);
                return (curnblk);
        }
 #endif

diff --git a/src/backend/storage/smgr/smgr.c b/src/backend/storage/smgr/smgr.c
index f96e7aa0988a577ee3de02e9388691a470ee931e..0adc16ed78e7ae0d7315325429479fe09e59ecfb 100644 (file)
--- a/src/backend/storage/smgr/smgr.c
+++ b/src/backend/storage/smgr/smgr.c
@@ -10,7 +10,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/storage/smgr/smgr.c,v 1.17 1998/06/27 04:53:40 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/storage/smgr/smgr.c,v 1.18 1998/07/20 16:14:15 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -132,8 +132,7 @@ smgrcreate(int16 which, Relation reln)
        int                     fd;
 
        if ((fd = (*(smgrsw[which].smgr_create)) (reln)) < 0)
-               elog(ERROR, "cannot create %s",
-                        &(reln->rd_rel->relname.data[0]));
+               elog(ERROR, "cannot create %s", reln->rd_rel->relname.data);
 
        return (fd);
 }
@@ -149,8 +148,7 @@ smgrunlink(int16 which, Relation reln)
        int                     status;
 
        if ((status = (*(smgrsw[which].smgr_unlink)) (reln)) == SM_FAIL)
-               elog(ERROR, "cannot unlink %s",
-                        &(reln->rd_rel->relname.data[0]));
+               elog(ERROR, "cannot unlink %s", reln->rd_rel->relname.data);
 
        return (status);
 }
@@ -169,8 +167,7 @@ smgrextend(int16 which, Relation reln, char *buffer)
        status = (*(smgrsw[which].smgr_extend)) (reln, buffer);
 
        if (status == SM_FAIL)
-               elog(ERROR, "%s: cannot extend",
-                        &(reln->rd_rel->relname.data[0]));
+               elog(ERROR, "%s: cannot extend", reln->rd_rel->relname.data);
 
        return (status);
 }
@@ -187,8 +184,7 @@ smgropen(int16 which, Relation reln)
        int                     fd;
 
        if ((fd = (*(smgrsw[which].smgr_open)) (reln)) < 0)
-               elog(ERROR, "cannot open %s",
-                        &(reln->rd_rel->relname.data[0]));
+               elog(ERROR, "cannot open %s", reln->rd_rel->relname.data);
 
        return (fd);
 }
@@ -209,8 +205,7 @@ int
 smgrclose(int16 which, Relation reln)
 {
        if ((*(smgrsw[which].smgr_close)) (reln) == SM_FAIL)
-               elog(ERROR, "cannot close %s",
-                        &(reln->rd_rel->relname.data[0]));
+               elog(ERROR, "cannot close %s", reln->rd_rel->relname.data);
 
        return (SM_SUCCESS);
 }
@@ -234,7 +229,7 @@ smgrread(int16 which, Relation reln, BlockNumber blocknum, char *buffer)
 
        if (status == SM_FAIL)
                elog(ERROR, "cannot read block %d of %s",
-                        blocknum, &(reln->rd_rel->relname.data[0]));
+                        blocknum, reln->rd_rel->relname.data);
 
        return (status);
 }
@@ -256,7 +251,7 @@ smgrwrite(int16 which, Relation reln, BlockNumber blocknum, char *buffer)
 
        if (status == SM_FAIL)
                elog(ERROR, "cannot write block %d of %s",
-                        blocknum, &(reln->rd_rel->relname.data[0]));
+                       blocknum, reln->rd_rel->relname.data);
 
        return (status);
 }
@@ -273,7 +268,7 @@ smgrflush(int16 which, Relation reln, BlockNumber blocknum, char *buffer)
 
        if (status == SM_FAIL)
                elog(ERROR, "cannot flush block %d of %s to stable store",
-                        blocknum, &(reln->rd_rel->relname.data[0]));
+                        blocknum, reln->rd_rel->relname.data);
 
        return (status);
 }
@@ -332,8 +327,7 @@ smgrnblocks(int16 which, Relation reln)
        int                     nblocks;
 
        if ((nblocks = (*(smgrsw[which].smgr_nblocks)) (reln)) < 0)
-               elog(ERROR, "cannot count blocks for %s",
-                        &(reln->rd_rel->relname.data[0]));
+               elog(ERROR, "cannot count blocks for %s", reln->rd_rel->relname.data);
 
        return (nblocks);
 }
@@ -355,7 +349,7 @@ smgrtruncate(int16 which, Relation reln, int nblocks)
        {
                if ((newblks = (*(smgrsw[which].smgr_truncate)) (reln, nblocks)) < 0)
                        elog(ERROR, "cannot truncate %s to %d blocks",
-                                &(reln->rd_rel->relname.data[0]), nblocks);
+                               reln->rd_rel->relname.data, nblocks);
        }
 
        return (newblks);

diff --git a/src/backend/utils/cache/relcache.c b/src/backend/utils/cache/relcache.c
index 686cc610f2b5293fc2a11a0a0f95df7fdff68524..3196a12bf7d37862aaa65d69fd6af1656ac5ff8a 100644 (file)
--- a/src/backend/utils/cache/relcache.c
+++ b/src/backend/utils/cache/relcache.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/utils/cache/relcache.c,v 1.40 1998/06/15 19:29:40 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/utils/cache/relcache.c,v 1.41 1998/07/20 16:14:16 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -604,8 +604,8 @@ build_tupdesc_ind(RelationBuildDescInfo buildinfo,
                atttup = (HeapTuple) AttributeNumIndexScan(attrel, relation->rd_id, i);
 
                if (!HeapTupleIsValid(atttup))
-                       elog(ERROR, "cannot find attribute %d of relation %.*s", i,
-                                NAMEDATALEN, &(relation->rd_rel->relname.data[0]));
+                       elog(ERROR, "cannot find attribute %d of relation %s", i,
+                               relation->rd_rel->relname.data);
                attp = (AttributeTupleForm) GETSTRUCT(atttup);
 
                relation->rd_att->attrs[i - 1] =
@@ -1712,40 +1712,39 @@ AttrDefaultFetch(Relation relation)
                        if (adform->adnum != attrdef[i].adnum)
                                continue;
                        if (attrdef[i].adsrc != NULL)
-                               elog(ERROR, "AttrDefaultFetch: second record found for attr %.*s in rel %.*s",
-                                        NAMEDATALEN, relation->rd_att->attrs[adform->adnum - 1]->attname.data,
-                                        NAMEDATALEN, relation->rd_rel->relname.data);
+                               elog(ERROR, "AttrDefaultFetch: second record found for attr %s in rel %s",
+                                       relation->rd_att->attrs[adform->adnum - 1]->attname.data,
+                                       relation->rd_rel->relname.data);
 
                        val = (struct varlena *) fastgetattr(tuple,
                                                                                                 Anum_pg_attrdef_adbin,
                                                                                                 adrel->rd_att, &isnull);
                        if (isnull)
-                               elog(ERROR, "AttrDefaultFetch: adbin IS NULL for attr %.*s in rel %.*s",
-                                        NAMEDATALEN, relation->rd_att->attrs[adform->adnum - 1]->attname.data,
-                                        NAMEDATALEN, relation->rd_rel->relname.data);
+                               elog(ERROR, "AttrDefaultFetch: adbin IS NULL for attr %s in rel %s",
+                                       relation->rd_att->attrs[adform->adnum - 1]->attname.data,
+                                       relation->rd_rel->relname.data);
                        attrdef[i].adbin = textout(val);
                        val = (struct varlena *) fastgetattr(tuple,
                                                                                                 Anum_pg_attrdef_adsrc,
                                                                                                 adrel->rd_att, &isnull);
                        if (isnull)
-                               elog(ERROR, "AttrDefaultFetch: adsrc IS NULL for attr %.*s in rel %.*s",
-                                        NAMEDATALEN, relation->rd_att->attrs[adform->adnum - 1]->attname.data,
-                                        NAMEDATALEN, relation->rd_rel->relname.data);
+                               elog(ERROR, "AttrDefaultFetch: adsrc IS NULL for attr %s in rel %s",
+                                       relation->rd_att->attrs[adform->adnum - 1]->attname.data,
+                                       relation->rd_rel->relname.data);
                        attrdef[i].adsrc = textout(val);
                        break;
                }
 
                if (i >= ndef)
-                       elog(ERROR, "AttrDefaultFetch: unexpected record found for attr %d in rel %.*s",
+                       elog(ERROR, "AttrDefaultFetch: unexpected record found for attr %d in rel %s",
                                 adform->adnum,
-                                NAMEDATALEN, relation->rd_rel->relname.data);
+                                relation->rd_rel->relname.data);
                ReleaseBuffer(buffer);
        }
 
        if (found < ndef)
-               elog(ERROR, "AttrDefaultFetch: %d record not found for rel %.*s",
-                        ndef - found,
-                        NAMEDATALEN, relation->rd_rel->relname.data);
+               elog(ERROR, "AttrDefaultFetch: %d record not found for rel %s",
+                        ndef - found, relation->rd_rel->relname.data);
 
        index_endscan(sd);
        pfree(sd);
@@ -1795,29 +1794,29 @@ RelCheckFetch(Relation relation)
                if (!HeapTupleIsValid(tuple))
                        continue;
                if (found == ncheck)
-                       elog(ERROR, "RelCheckFetch: unexpected record found for rel %.*s",
-                                NAMEDATALEN, relation->rd_rel->relname.data);
+                       elog(ERROR, "RelCheckFetch: unexpected record found for rel %s",
+                               relation->rd_rel->relname.data);
 
                rcname = (Name) fastgetattr(tuple,
                                                                        Anum_pg_relcheck_rcname,
                                                                        rcrel->rd_att, &isnull);
                if (isnull)
-                       elog(ERROR, "RelCheckFetch: rcname IS NULL for rel %.*s",
-                                NAMEDATALEN, relation->rd_rel->relname.data);
+                       elog(ERROR, "RelCheckFetch: rcname IS NULL for rel %s",
+                                relation->rd_rel->relname.data);
                check[found].ccname = nameout(rcname);
                val = (struct varlena *) fastgetattr(tuple,
                                                                                         Anum_pg_relcheck_rcbin,
                                                                                         rcrel->rd_att, &isnull);
                if (isnull)
-                       elog(ERROR, "RelCheckFetch: rcbin IS NULL for rel %.*s",
-                                NAMEDATALEN, relation->rd_rel->relname.data);
+                       elog(ERROR, "RelCheckFetch: rcbin IS NULL for rel %s",
+                                relation->rd_rel->relname.data);
                check[found].ccbin = textout(val);
                val = (struct varlena *) fastgetattr(tuple,
                                                                                         Anum_pg_relcheck_rcsrc,
                                                                                         rcrel->rd_att, &isnull);
                if (isnull)
-                       elog(ERROR, "RelCheckFetch: rcsrc IS NULL for rel %.*s",
-                                NAMEDATALEN, relation->rd_rel->relname.data);
+                       elog(ERROR, "RelCheckFetch: rcsrc IS NULL for rel %s",
+                                relation->rd_rel->relname.data);
                check[found].ccsrc = textout(val);
                found++;
 
@@ -1825,9 +1824,9 @@ RelCheckFetch(Relation relation)
        }
 
        if (found < ncheck)
-               elog(ERROR, "RelCheckFetch: %d record not found for rel %.*s",
+               elog(ERROR, "RelCheckFetch: %d record not found for rel %s",
                         ncheck - found,
-                        NAMEDATALEN, relation->rd_rel->relname.data);
+                        relation->rd_rel->relname.data);
 
        index_endscan(sd);
        pfree(sd);

diff --git a/src/backend/utils/misc/database.c b/src/backend/utils/misc/database.c
index a2ab2f3ba76fa28220fff7b37937f28c48acf86b..9c9a0a63fa8b0f5be80d3ee9cba90400d8aa061c 100644 (file)
--- a/src/backend/utils/misc/database.c
+++ b/src/backend/utils/misc/database.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/utils/misc/Attic/database.c,v 1.10 1998/06/15 19:29:50 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/utils/misc/Attic/database.c,v 1.11 1998/07/20 16:14:18 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -267,9 +267,7 @@ GetRawDatabaseInfo(char *name, Oid *owner, Oid *db_id, char *path)
                         */
                        tup_db = (Form_pg_database) GETSTRUCT(tup);
 
-                       if (strncmp(name,
-                                               &(tup_db->datname.data[0]),
-                                               16) == 0)
+                       if (strcmp(name, tup_db->datname.data) == 0)
                        {
                                *db_id = tup->t_oid;
                                strncpy(path, VARDATA(&(tup_db->datpath)),