]> granicus.if.org Git - postgresql/commitdiff
Rename EnumValuesCreate() single-letter variable names to useful
authorBruce Momjian <bruce@momjian.us>
Thu, 24 Dec 2009 22:17:58 +0000 (22:17 +0000)
committerBruce Momjian <bruce@momjian.us>
Thu, 24 Dec 2009 22:17:58 +0000 (22:17 +0000)
variable names.

src/backend/catalog/pg_enum.c

index ee075d94c84b130c23dd303ebe3819e2ee0ce33e..9a3d533c146aea26b7b76c60a5d03d065067e379 100644 (file)
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/backend/catalog/pg_enum.c,v 1.10 2009/12/19 00:47:57 momjian Exp $
+ *       $PostgreSQL: pgsql/src/backend/catalog/pg_enum.c,v 1.11 2009/12/24 22:17:58 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -39,14 +39,14 @@ EnumValuesCreate(Oid enumTypeOid, List *vals)
        TupleDesc       tupDesc;
        NameData        enumlabel;
        Oid                *oids;
-       int                     i,
-                               n;
+       int                     elemno,
+                               num_elems;
        Datum           values[Natts_pg_enum];
        bool            nulls[Natts_pg_enum];
        ListCell   *lc;
        HeapTuple       tup;
 
-       n = list_length(vals);
+       num_elems = list_length(vals);
 
        /*
         * XXX we do not bother to check the list of values for duplicates --- if
@@ -64,23 +64,23 @@ EnumValuesCreate(Oid enumTypeOid, List *vals)
         * counter wraps all the way around before we finish. Which seems
         * unlikely.
         */
-       oids = (Oid *) palloc(n * sizeof(Oid));
-       for (i = 0; i < n; i++)
+       oids = (Oid *) palloc(num_elems * sizeof(Oid));
+       for (elemno = 0; elemno < num_elems; elemno++)
        {
                /*
                 *      The pg_enum.oid is stored in user tables.  This oid must be
                 *      preserved by binary upgrades.
                 */
-               oids[i] = GetNewOid(pg_enum);
+               oids[elemno] = GetNewOid(pg_enum);
        }
 
        /* sort them, just in case counter wrapped from high to low */
-       qsort(oids, n, sizeof(Oid), oid_cmp);
+       qsort(oids, num_elems, sizeof(Oid), oid_cmp);
 
        /* and make the entries */
        memset(nulls, false, sizeof(nulls));
 
-       i = 0;
+       elemno = 0;
        foreach(lc, vals)
        {
                char       *lab = strVal(lfirst(lc));
@@ -101,13 +101,13 @@ EnumValuesCreate(Oid enumTypeOid, List *vals)
                values[Anum_pg_enum_enumlabel - 1] = NameGetDatum(&enumlabel);
 
                tup = heap_form_tuple(tupDesc, values, nulls);
-               HeapTupleSetOid(tup, oids[i]);
+               HeapTupleSetOid(tup, oids[elemno]);
 
                simple_heap_insert(pg_enum, tup);
                CatalogUpdateIndexes(pg_enum, tup);
                heap_freetuple(tup);
 
-               i++;
+               elemno++;
        }
 
        /* clean up */