]> granicus.if.org Git - postgresql/commitdiff
Fix inconsistencies and typos in the tree, take 10
authorMichael Paquier <michael@paquier.xyz>
Tue, 13 Aug 2019 04:53:41 +0000 (13:53 +0900)
committerMichael Paquier <michael@paquier.xyz>
Tue, 13 Aug 2019 04:53:41 +0000 (13:53 +0900)
This addresses some issues with unnecessary code comments, fixes various
typos in docs and comments, and removes some orphaned structures and
definitions.

Author: Alexander Lakhin
Discussion: https://postgr.es/m/9aabc775-5494-b372-8bcb-4dfc0bd37c68@gmail.com

74 files changed:
contrib/fuzzystrmatch/dmetaphone.c
contrib/ltree/ltxtquery_io.c
contrib/pgcrypto/internal.c
contrib/pgcrypto/pgp-decrypt.c
contrib/pgcrypto/pgp-encrypt.c
contrib/pgcrypto/pgp.h
contrib/sepgsql/label.c
contrib/sepgsql/selinux.c
contrib/spi/refint.example
doc/src/sgml/ecpg.sgml
doc/src/sgml/plpgsql.sgml
doc/src/sgml/sources.sgml
src/backend/access/brin/brin_revmap.c
src/backend/access/gist/gist.c
src/backend/access/hash/hashpage.c
src/backend/access/heap/heapam.c
src/backend/access/nbtree/nbtutils.c
src/backend/access/transam/slru.c
src/backend/access/transam/xact.c
src/backend/access/transam/xlogreader.c
src/backend/catalog/catalog.c
src/backend/catalog/pg_constraint.c
src/backend/commands/cluster.c
src/backend/commands/sequence.c
src/backend/executor/spi.c
src/backend/libpq/auth.c
src/backend/libpq/pqcomm.c
src/backend/optimizer/path/costsize.c
src/backend/partitioning/partbounds.c
src/backend/partitioning/partprune.c
src/backend/port/posix_sema.c
src/backend/port/win32/signal.c
src/backend/postmaster/checkpointer.c
src/backend/replication/basebackup.c
src/backend/replication/logical/snapbuild.c
src/backend/replication/logical/worker.c
src/backend/replication/slot.c
src/backend/storage/file/fd.c
src/backend/storage/ipc/procarray.c
src/backend/storage/lmgr/lwlock.c
src/backend/storage/lmgr/predicate.c
src/backend/utils/adt/pg_locale.c
src/backend/utils/adt/rangetypes_typanalyze.c
src/backend/utils/adt/txid.c
src/backend/utils/adt/varlena.c
src/backend/utils/adt/xml.c
src/backend/utils/misc/check_guc
src/backend/utils/mmgr/slab.c
src/backend/utils/sort/tuplesort.c
src/bin/pg_dump/pg_backup_archiver.c
src/bin/pg_rewind/pg_rewind.c
src/bin/pg_upgrade/pg_upgrade.h
src/include/access/brin_revmap.h
src/include/access/hash_xlog.h
src/include/access/reloptions.h
src/include/access/xlogreader.h
src/include/catalog/namespace.h
src/include/nodes/nodes.h
src/include/port/win32_port.h
src/include/rewrite/prs2lock.h
src/include/storage/standbydefs.h
src/include/utils/elog.h
src/include/utils/guc.h
src/include/utils/rel.h
src/include/utils/selfuncs.h
src/interfaces/ecpg/compatlib/informix.c
src/interfaces/libpq/libpq-fe.h
src/pl/plpython/plpy_subxactobject.c
src/port/getaddrinfo.c
src/test/isolation/specs/sequence-ddl.spec
src/test/recovery/t/012_subtransactions.pl
src/test/regress/.gitignore
src/timezone/pgtz.c
src/tools/pgindent/README

index bff20b37e686745b3700c36d523cbd3f9557f37e..f2f16bc883a4623a21b7c25b8377f874fc7022cc 100644 (file)
@@ -191,7 +191,7 @@ dmetaphone_alt(PG_FUNCTION_ARGS)
                                          (v = (t*)repalloc((v),((n)*sizeof(t))))
 
 /*
- * Don't do pfree - it seems to cause a segv sometimes - which might have just
+ * Don't do pfree - it seems to cause a SIGSEGV sometimes - which might have just
  * been caused by reloading the module in development.
  * So we rely on context cleanup - Tom Lane says pfree shouldn't be necessary
  * in a case like this.
index b576676746dc56e89674992750ff9c5b2c70aed7..054c5d9335678dd589910b6c2c09b8768a4f6548 100644 (file)
@@ -166,7 +166,7 @@ pushquery(QPRS_STATE *state, int32 type, int32 val, int32 distance, int32 lenval
 }
 
 /*
- * This function is used for query_txt parsing
+ * This function is used for query text parsing
  */
 static void
 pushval_asis(QPRS_STATE *state, int type, char *strval, int lenval, uint16 flag)
index 16dfe725eae381bd50f03be86f8124463139f2a9..db58408d4c718f2cd616931990b85c8f5cd04364 100644 (file)
 #include "blf.h"
 #include "rijndael.h"
 
-/*
- * System reseeds should be separated at least this much.
- */
-#define SYSTEM_RESEED_MIN                      (20*60) /* 20 min */
-/*
- * How often to roll dice.
- */
-#define SYSTEM_RESEED_CHECK_TIME       (10*60) /* 10 min */
-/*
- * The chance is x/256 that the reseed happens.
- */
-#define SYSTEM_RESEED_CHANCE           (4) /* 256/4 * 10min ~ 10h */
-
-/*
- * If this much time has passed, force reseed.
- */
-#define SYSTEM_RESEED_MAX                      (12*60*60)      /* 12h */
-
-
 #ifndef MD5_DIGEST_LENGTH
 #define MD5_DIGEST_LENGTH 16
 #endif
index eed0f6ad2d5faf1d8d3446e71e961e2f42dd2c1c..4c43eb7e3efe60f847d0be39e9dd2b87af58b04d 100644 (file)
@@ -423,7 +423,7 @@ static struct PullFilterOps mdc_filter = {
 /*
  * Combined Pkt reader and MDC hasher.
  *
- * For the case of SYMENCRYPTED_MDC packet, where
+ * For the case of SYMENCRYPTED_DATA_MDC packet, where
  * the data part has 'context length', which means
  * that data packet ends 22 bytes before end of parent
  * packet, which is silly.
@@ -894,7 +894,10 @@ process_data_packets(PGP_Context *ctx, MBuf *dst, PullFilter *src,
                        break;
                }
 
-               /* context length inside SYMENC_MDC needs special handling */
+               /*
+                * Context length inside SYMENCRYPTED_DATA_MDC packet needs special
+                * handling.
+                */
                if (need_mdc && res == PKT_CONTEXT)
                        res = pullf_create(&pkt, &mdcbuf_filter, ctx, src);
                else
index 8a6cd826755b9b05b7c6a319cc03fd132944a4cd..2938b4b3f5ec46f01f8dd73c17ac8f1fb2f8a8b5 100644 (file)
@@ -618,7 +618,7 @@ pgp_encrypt(PGP_Context *ctx, MBuf *src, MBuf *dst)
                goto out;
 
        /*
-        * initialize symkey
+        * initialize sym_key
         */
        if (ctx->sym_key)
        {
index 7c97fa398fa72d36f7b82d161ae08c56e43111cd..f338523b7a644ab45c97988b05904b446cfb54eb 100644 (file)
@@ -279,7 +279,7 @@ int                 pgp_s2k_process(PGP_S2K *s2k, int cipher, const uint8 *key, int klen);
 
 typedef struct PGP_CFB PGP_CFB;
 int                    pgp_cfb_create(PGP_CFB **ctx_p, int algo,
-                                                  const uint8 *key, int key_len, int recync, uint8 *iv);
+                                                  const uint8 *key, int key_len, int resync, uint8 *iv);
 void           pgp_cfb_free(PGP_CFB *ctx);
 int                    pgp_cfb_encrypt(PGP_CFB *ctx, const uint8 *data, int len, uint8 *dst);
 int                    pgp_cfb_decrypt(PGP_CFB *ctx, const uint8 *data, int len, uint8 *dst);
index 2b6a41509316a7869d61136ccd2d756672966b10..63a2dd5cc1ba43496e4e6d40af4426e4a5abbc1c 100644 (file)
@@ -653,7 +653,7 @@ sepgsql_mcstrans_out(PG_FUNCTION_ARGS)
 }
 
 /*
- * quote_object_names
+ * quote_object_name
  *
  * It tries to quote the supplied identifiers
  */
index bc2be8427af1e55308274753e679963a112a9f74..192aabea0b3411d4ed420c60983b7f6fe5c2681f 100644 (file)
@@ -657,10 +657,8 @@ sepgsql_getenforce(void)
 /*
  * sepgsql_audit_log
  *
- * It generates a security audit record. In the default, it writes out
- * audit records into standard PG's logfile. It also allows to set up
- * external audit log receiver, such as auditd in Linux, using the
- * sepgsql_audit_hook.
+ * It generates a security audit record. It writes out audit records
+ * into standard PG's logfile.
  *
  * SELinux can control what should be audited and should not using
  * "auditdeny" and "auditallow" rules in the security policy. In the
index d0ff7441642cf712cf5d1649d6ca8f8c23a12102..299166d504193e9d4ee1ecfab53c4e67f8e203e9 100644 (file)
@@ -5,7 +5,7 @@ CREATE TABLE A (
 );
 CREATE UNIQUE INDEX AI ON A (ID);
 
---Columns REFB of table B and REFC of C are foreign keys referenting ID of A:
+--Columns REFB of table B and REFC of C are foreign keys referencing ID of A:
 
 CREATE TABLE B (
        REFB    int4
index b6a4fa2600e7fea537f481dfdadd139a14fd8865..a1ebcaa862ed5cf7c0e6d6b1748271d03fb439d0 100644 (file)
@@ -5011,7 +5011,7 @@ struct
 
    <para>
     The fields <structfield>sqlcaid</structfield>,
-    <structfield>sqlcabc</structfield>,
+    <structfield>sqlabc</structfield>,
     <structfield>sqlerrp</structfield>, and the remaining elements of
     <structfield>sqlerrd</structfield> and
     <structfield>sqlwarn</structfield> currently contain no useful
index ef1556be8feba07074706244f2659a94eec69d19..ae73630a489d231b45514a7951e6575322bcb49a 100644 (file)
@@ -3740,9 +3740,9 @@ RAISE unique_violation USING MESSAGE = 'Duplicate user ID: ' || user_id;
    <para>
     If no condition name nor SQLSTATE is specified in a
     <command>RAISE EXCEPTION</command> command, the default is to use
-    <literal>RAISE_EXCEPTION</literal> (<literal>P0001</literal>).  If no message
-    text is specified, the default is to use the condition name or
-    SQLSTATE as message text.
+    <literal>ERRCODE_RAISE_EXCEPTION</literal> (<literal>P0001</literal>).
+    If no message text is specified, the default is to use the condition
+    name or SQLSTATE as message text.
    </para>
 
    <note>
@@ -5626,7 +5626,7 @@ BEGIN
     INSERT INTO cs_active_job(job_id) VALUES (v_job_id);
 
     BEGIN
-        INSERT INTO cs_jobs (job_id, start_stamp) VALUES (v_job_id, sysdate);
+        INSERT INTO cs_jobs (job_id, start_stamp) VALUES (v_job_id, now());
     EXCEPTION
         WHEN dup_val_on_index THEN NULL; -- don't worry if it already exists
     END;
index a339ebbc4ba620e373946e1f0573622efbb5acc1..6294312ab7dfc98a0a2144440640c7e48f3ba609 100644 (file)
@@ -884,7 +884,7 @@ BETTER: unrecognized node type: 42
      features can be used, if a fallback is provided.
     </para>
     <para>
-     For example <literal>_StaticAssert()</literal> and
+     For example <literal>_Static_assert()</literal> and
      <literal>__builtin_constant_p</literal> are currently used, even though
      they are from newer revisions of the C standard and a
      <productname>GCC</productname> extension respectively. If not available
index e2bfbf8a680e1b9c998349cd658e4a3be8baa818..647350c1018f50836c3ee5f76777bcb315bcff2b 100644 (file)
@@ -395,9 +395,10 @@ brinRevmapDesummarizeRange(Relation idxrel, BlockNumber heapBlk)
         */
 
        /*
-        * Because of SUE lock, this function shouldn't run concurrently with
-        * summarization.  Placeholder tuples can only exist as leftovers from
-        * crashed summarization, so if we detect any, we complain but proceed.
+        * Because of ShareUpdateExclusive lock, this function shouldn't run
+        * concurrently with summarization.  Placeholder tuples can only exist as
+        * leftovers from crashed summarization, so if we detect any, we complain
+        * but proceed.
         */
        if (BrinTupleIsPlaceholder(tup))
                ereport(WARNING,
index e9ca4b825279e5e6f48faace1cbc6ee1b3cc271a..0cc87911d6bec1e30498f02cb637accea98ee2ea 100644 (file)
@@ -37,7 +37,7 @@ static bool gistinserttuples(GISTInsertState *state, GISTInsertStack *stack,
                                                         Buffer leftchild, Buffer rightchild,
                                                         bool unlockbuf, bool unlockleftchild);
 static void gistfinishsplit(GISTInsertState *state, GISTInsertStack *stack,
-                                                       GISTSTATE *giststate, List *splitinfo, bool releasebuf);
+                                                       GISTSTATE *giststate, List *splitinfo, bool unlockbuf);
 static void gistprunepage(Relation rel, Page page, Buffer buffer,
                                                  Relation heapRel);
 
@@ -1047,7 +1047,7 @@ gistFindCorrectParent(Relation r, GISTInsertStack *child)
                        {
                                /*
                                 * End of chain and still didn't find parent. It's a very-very
-                                * rare situation when root splited.
+                                * rare situation when root splitted.
                                 */
                                break;
                        }
index defdc9b4085ceefa04afcf7a2aafa2a40fb29d73..838ee68c867396be7c8aab6db57d75d600616aba 100644 (file)
@@ -1510,7 +1510,7 @@ _hash_getcachedmetap(Relation rel, Buffer *metabuf, bool force_refresh)
                 * It's important that we don't set rd_amcache to an invalid value.
                 * Either MemoryContextAlloc or _hash_getbuf could fail, so don't
                 * install a pointer to the newly-allocated storage in the actual
-                * relcache entry until both have succeeeded.
+                * relcache entry until both have succeeded.
                 */
                if (rel->rd_amcache == NULL)
                        cache = MemoryContextAlloc(rel->rd_indexcxt,
index 6b42bdc77b24b159231db007757b59b6436c4c76..718f07f4f65f2336a632be4fcdddbc40bf77c385 100644 (file)
@@ -3029,7 +3029,7 @@ heap_update(Relation relation, ItemPointer otid, HeapTuple newtup,
         * the value ends up being the same, this test will fail and we will use
         * the stronger lock.  This is acceptable; the important case to optimize
         * is updates that don't manipulate key columns, not those that
-        * serendipitiously arrive at the same key values.
+        * serendipitously arrive at the same key values.
         */
        if (!bms_overlap(modified_attrs, key_attrs))
        {
index 93fab264ae4d5c2dd208b87c17c2ca5796e702aa..9b172c1a191f70acd6adea827c5d137e7411f46c 100644 (file)
@@ -664,7 +664,7 @@ _bt_restore_array_keys(IndexScanDesc scan)
  * scan->numberOfKeys is the number of input keys, so->numberOfKeys gets
  * the number of output keys (possibly less, never greater).
  *
- * The output keys are marked with additional sk_flag bits beyond the
+ * The output keys are marked with additional sk_flags bits beyond the
  * system-standard bits supplied by the caller.  The DESC and NULLS_FIRST
  * indoption bits for the relevant index attribute are copied into the flags.
  * Also, for a DESC column, we commute (flip) all the sk_strategy numbers
index 13aa8e1b95af67c8d231ce3a3f93af20916a560d..0fbcb4e6fecc932d665629766b6a9ed1a5bf2f24 100644 (file)
@@ -1364,7 +1364,7 @@ SlruScanDirCbDeleteAll(SlruCtl ctl, char *filename, int segpage, void *data)
 }
 
 /*
- * Scan the SimpleLRU directory and apply a callback to each file found in it.
+ * Scan the SimpleLru directory and apply a callback to each file found in it.
  *
  * If the callback returns true, the scan is stopped.  The last return value
  * from the callback is returned.
index 1bbaeeebf4d6f349cdfb887a0fe239320f98d951..52e9643305995bd32e5fee6638392a0d2e7d1309 100644 (file)
@@ -1232,7 +1232,7 @@ RecordTransactionCommit(void)
        if (!markXidCommitted)
        {
                /*
-                * We expect that every smgrscheduleunlink is followed by a catalog
+                * We expect that every RelationDropStorage is followed by a catalog
                 * update, and hence XID assignment, so we shouldn't get here with any
                 * pending deletes.  Use a real test not just an Assert to check this,
                 * since it's a bit fragile.
index 0d6e968ee62737b2ecc177255b699ed2b6b2a5db..c6faf48d24f398ccfe7d42be2554bb16d21210b8 100644 (file)
@@ -702,7 +702,7 @@ ValidXLogRecordHeader(XLogReaderState *state, XLogRecPtr RecPtr,
  * We assume all of the record (that is, xl_tot_len bytes) has been read
  * into memory at *record.  Also, ValidXLogRecordHeader() has accepted the
  * record's header, which means in particular that xl_tot_len is at least
- * SizeOfXlogRecord.
+ * SizeOfXLogRecord.
  */
 static bool
 ValidXLogRecord(XLogReaderState *state, XLogRecord *record, XLogRecPtr recptr)
index a065419cdb28d321b843baa6496121237c9b208b..1af31c2b9f9f041786a113a1012803000aa50158 100644 (file)
@@ -317,13 +317,6 @@ IsSharedRelation(Oid relationId)
  * consecutive existing OIDs.  This is a mostly reasonable assumption for
  * system catalogs.
  *
- * This is exported separately because there are cases where we want to use
- * an index that will not be recognized by RelationGetOidIndex: TOAST tables
- * have indexes that are usable, but have multiple columns and are on
- * ordinary columns rather than a true OID column.  This code will work
- * anyway, so long as the OID is the index's first column.  The caller must
- * pass in the actual heap attnum of the OID column, however.
- *
  * Caller must have a suitable lock on the relation.
  */
 Oid
index b6145593a38a4c4e4e61652a0cb20fc46b76f6e5..56568b0105f2e3f5684e8b6d4eab5bc2b53775d8 100644 (file)
@@ -580,7 +580,7 @@ RemoveConstraintById(Oid conId)
                rel = table_open(con->conrelid, AccessExclusiveLock);
 
                /*
-                * We need to update the relcheck count if it is a check constraint
+                * We need to update the relchecks count if it is a check constraint
                 * being dropped.  This update will force backends to rebuild relcache
                 * entries when we commit.
                 */
index cedb4ee844d91846b7ca2a3e4fcb238a582b3fdd..28985a07ec1b161989fc79dfa435873a5d2e71a8 100644 (file)
@@ -1522,7 +1522,7 @@ finish_heap_swap(Oid OIDOldHeap, Oid OIDNewHeap,
 
 /*
  * Get a list of tables that the current user owns and
- * have indisclustered set.  Return the list in a List * of rvsToCluster
+ * have indisclustered set.  Return the list in a List * of RelToCluster
  * with the tableOid and the indexOid on which the table is already
  * clustered.
  */
index 0960b339cab023326c95c4567eab685498544dbb..a13322b6938e93ce14b216d92034e5b549bda34a 100644 (file)
@@ -1115,7 +1115,7 @@ init_sequence(Oid relid, SeqTable *p_elm, Relation *p_rel)
        /*
         * Initialize the new hash table entry if it did not exist already.
         *
-        * NOTE: seqtable entries are stored for the life of a backend (unless
+        * NOTE: seqhashtab entries are stored for the life of a backend (unless
         * explicitly discarded with DISCARD). If the sequence itself is deleted
         * then the entry becomes wasted memory, but it's small enough that this
         * should not matter.
index b8eb83b2e0aa2bdba38bca7f237df21780da032b..2c0ae395ba6d2e2abac2fc5b9be6cd21b51b5548 100644 (file)
@@ -2603,7 +2603,7 @@ _SPI_cursor_operation(Portal portal, FetchDirection direction, long count,
 
        /*
         * Think not to combine this store with the preceding function call. If
-        * the portal contains calls to functions that use SPI, then SPI_stack is
+        * the portal contains calls to functions that use SPI, then _SPI_stack is
         * likely to move around while the portal runs.  When control returns,
         * _SPI_current will point to the correct stack entry... but the pointer
         * may be different than it was beforehand. So we must be sure to re-fetch
@@ -2733,7 +2733,7 @@ _SPI_make_plan_non_temp(SPIPlanPtr plan)
                                                                        ALLOCSET_SMALL_SIZES);
        oldcxt = MemoryContextSwitchTo(plancxt);
 
-       /* Copy the SPI_plan struct and subsidiary data into the new context */
+       /* Copy the _SPI_plan struct and subsidiary data into the new context */
        newplan = (SPIPlanPtr) palloc0(sizeof(_SPI_plan));
        newplan->magic = _SPI_PLAN_MAGIC;
        newplan->plancxt = plancxt;
index 5fb1f1b7d7ae5677f56cd701cee35b4325b07ca1..0e0a6d87528b3771057fee92cb10f07ff404fc74 100644 (file)
@@ -1797,7 +1797,7 @@ interpret_ident_response(const char *ident_response,
 /*
  *     Talk to the ident server on "remote_addr" and find out who
  *     owns the tcp connection to "local_addr"
- *     It the username successfully retrieved, check the usermap.
+ *     If the username is successfully retrieved, check the usermap.
  *
  *     XXX: Using WaitLatchOrSocket() and doing a CHECK_FOR_INTERRUPTS() if the
  *     latch was set would improve the responsiveness to timeouts/cancellations.
index d8d863dda2056772149e9c2a613c97c6eb3bccbb..cd517e8bb41de1efca0753ded8f6976ca5fb8020 100644 (file)
@@ -485,10 +485,10 @@ StreamServerPort(int family, char *hostName, unsigned short portNumber,
                 * error on TCP ports.
                 *
                 * On win32, however, this behavior only happens if the
-                * SO_EXLUSIVEADDRUSE is set. With SO_REUSEADDR, win32 allows multiple
-                * servers to listen on the same address, resulting in unpredictable
-                * behavior. With no flags at all, win32 behaves as Unix with
-                * SO_REUSEADDR.
+                * SO_EXCLUSIVEADDRUSE is set. With SO_REUSEADDR, win32 allows
+                * multiple servers to listen on the same address, resulting in
+                * unpredictable behavior. With no flags at all, win32 behaves as Unix
+                * with SO_REUSEADDR.
                 */
                if (!IS_AF_UNIX(addr->ai_family))
                {
index bc6bc9995738ed738643201438528c63e2fa0ca6..c5f6593485982359ddadd00323135171ecc6343a 100644 (file)
@@ -1811,7 +1811,7 @@ append_nonpartial_cost(List *subpaths, int numpaths, int parallel_workers)
                return 0;
 
        /*
-        * Array length is number of workers or number of relevants paths,
+        * Array length is number of workers or number of relevant paths,
         * whichever is less.
         */
        arrlen = Min(parallel_workers, numpaths);
index 5a29b9673b8c984ed0b50276d86e26814f5b5a51..46d03f3b9b3a773d7a43c8ad34a9513dc0a9345f 100644 (file)
@@ -1514,7 +1514,7 @@ partition_rbound_cmp(int partnatts, FmgrInfo *partsupfunc,
 /*
  * partition_rbound_datum_cmp
  *
- * Return whether range bound (specified in rb_datums, rb_kind, and rb_lower)
+ * Return whether range bound (specified in rb_datums and rb_kind)
  * is <, =, or > partition key of tuple (tuple_datums)
  *
  * n_tuple_datums, partsupfunc and partcollation give number of attributes in
index a1bd4efd0b049710ee89246a6625316d9ed65dd3..735311eac45b7fd0c6bb10f39ba5a11483f89ccc 100644 (file)
@@ -2127,7 +2127,7 @@ match_clause_to_partition_key(GeneratePruningStepsContext *context,
 
                /*
                 * Now generate a list of clauses, one for each array element, of the
-                * form saop_leftop saop_op elem_expr
+                * form leftop saop_op elem_expr
                 */
                elem_clauses = NIL;
                foreach(lc1, elem_exprs)
index 5e77cb0c12e97b6341525a7451a14360ce38c627..3370adf35e27c87cc60734f118294533dcc922c4 100644 (file)
@@ -189,7 +189,7 @@ PGSemaphoreShmemSize(int maxSemas)
  * maxSemas parameter is just used to size the arrays.  For unnamed
  * semaphores, there is an array of PGSemaphoreData structs in shared memory.
  * For named semaphores, we keep a postmaster-local array of sem_t pointers,
- * which we use for releasing the semphores when done.
+ * which we use for releasing the semaphores when done.
  * (This design minimizes the dependency of postmaster shutdown on the
  * contents of shared memory, which a failed backend might have clobbered.
  * We can't do much about the possibility of sem_destroy() crashing, but
index 7d8961f4a447778ac74bd6792f3ae0c626b20375..9b5e544febb69c1aef01d9da3076fc9528a3adf7 100644 (file)
@@ -102,7 +102,7 @@ pgwin32_signal_initialize(void)
 /*
  * Dispatch all signals currently queued and not blocked
  * Blocked signals are ignored, and will be fired at the time of
- * the sigsetmask() call.
+ * the pqsigsetmask() call.
  */
 void
 pgwin32_dispatch_queued_signals(void)
index 11bbe2c397f90d36ba6955589ce0783bec27062e..61544f65adaf8e2b658a28327daa7dff33482fcd 100644 (file)
@@ -750,7 +750,7 @@ IsCheckpointOnSchedule(double progress)
         * We compare the current WAL insert location against the location
         * computed before calling CreateCheckPoint. The code in XLogInsert that
         * actually triggers a checkpoint when CheckPointSegments is exceeded
-        * compares against RedoRecptr, so this is not completely accurate.
+        * compares against RedoRecPtr, so this is not completely accurate.
         * However, it's good enough for our purposes, we're only calculating an
         * estimate anyway.
         *
index d5f9b617c84c5f49b8ee2eb3f276cf1fbbf7d78b..c91f66dcbeb7e4173e7e4c0d165b63638e929767 100644 (file)
@@ -792,7 +792,7 @@ SendBackupHeader(List *tablespaces)
        pq_sendint32(&buf, 0);          /* typmod */
        pq_sendint16(&buf, 0);          /* format code */
 
-       /* Second field - spcpath */
+       /* Second field - spclocation */
        pq_sendstring(&buf, "spclocation");
        pq_sendint32(&buf, 0);
        pq_sendint16(&buf, 0);
index dc64b1e0c2fa57930cad5d9995bacbd3340b5deb..0bd1d0f9545160da924489ee331e53865e73c6d7 100644 (file)
@@ -269,7 +269,7 @@ static void SnapBuildSnapIncRefcount(Snapshot snap);
 
 static void SnapBuildDistributeNewCatalogSnapshot(SnapBuild *builder, XLogRecPtr lsn);
 
-/* xlog reading helper functions for SnapBuildProcessRecord */
+/* xlog reading helper functions for SnapBuildProcessRunningXacts */
 static bool SnapBuildFindSnapshot(SnapBuild *builder, XLogRecPtr lsn, xl_running_xacts *running);
 static void SnapBuildWaitSnapshot(xl_running_xacts *running, TransactionId cutoff);
 
index 43edfef0895db6113902567f45f3a5e8f8066508..11e6331f494cfd9e466d9575d555b099f7096081 100644 (file)
@@ -1658,7 +1658,7 @@ ApplyWorkerMain(Datum main_arg)
        {
                char       *syncslotname;
 
-               /* This is table synchroniation worker, call initial sync. */
+               /* This is table synchronization worker, call initial sync. */
                syncslotname = LogicalRepSyncTableStart(&origin_startpos);
 
                /* The slot name needs to be allocated in permanent memory context. */
index 62342a69cbf33b03b7bf60e2fdef19b886ed962e..b1bcd93345dacd459c73e548198d48f7e5c90e17 100644 (file)
@@ -108,7 +108,7 @@ static void CreateSlotOnDisk(ReplicationSlot *slot);
 static void SaveSlotToPath(ReplicationSlot *slot, const char *path, int elevel);
 
 /*
- * Report shared-memory space needed by ReplicationSlotShmemInit.
+ * Report shared-memory space needed by ReplicationSlotsShmemInit.
  */
 Size
 ReplicationSlotsShmemSize(void)
@@ -298,7 +298,7 @@ ReplicationSlotCreate(const char *name, bool db_specific,
         * We need to briefly prevent any other backend from iterating over the
         * slots while we flip the in_use flag. We also need to set the active
         * flag while holding the ControlLock as otherwise a concurrent
-        * SlotAcquire() could acquire the slot as well.
+        * ReplicationSlotAcquire() could acquire the slot as well.
         */
        LWLockAcquire(ReplicationSlotControlLock, LW_EXCLUSIVE);
 
index 60f39031fb5558670ae52b0c3e44cc62f2add3be..a76112d6cde1631db9167a6c601f73207ace5a7a 100644 (file)
@@ -1971,10 +1971,6 @@ retry:
        {
                /*
                 * Maintain fileSize and temporary_files_size if it's a temp file.
-                *
-                * If seekPos is -1 (unknown), this will do nothing; but we could only
-                * get here in that state if we're not enforcing temporary_files_size,
-                * so we don't care.
                 */
                if (vfdP->fdstate & FD_TEMP_FILE_LIMIT)
                {
index 469aac38c403b32d1436a84c35e702da78d52cc1..8abcfdf841fba64fbfca9560f6698664f7ec7d37 100644 (file)
@@ -734,8 +734,6 @@ ProcArrayApplyRecoveryInfo(RunningTransactions running)
        Assert(standbyState == STANDBY_INITIALIZED);
 
        /*
-        * OK, we need to initialise from the RunningTransactionsData record.
-        *
         * NB: this can be reached at least twice, so make sure new code can deal
         * with that.
         */
@@ -750,11 +748,11 @@ ProcArrayApplyRecoveryInfo(RunningTransactions running)
         * sort them first.
         *
         * Some of the new xids are top-level xids and some are subtransactions.
-        * We don't call SubtransSetParent because it doesn't matter yet. If we
+        * We don't call SubTransSetParent because it doesn't matter yet. If we
         * aren't overflowed then all xids will fit in snapshot and so we don't
         * need subtrans. If we later overflow, an xid assignment record will add
-        * xids to subtrans. If RunningXacts is overflowed then we don't have
-        * enough information to correctly update subtrans anyway.
+        * xids to subtrans. If RunningTransactionsData is overflowed then we
+        * don't have enough information to correctly update subtrans anyway.
         */
 
        /*
index c77d47c01c6e2a3d2f8eed510ace4910b130d36f..fb0bf44264056c942570a66ea298893fe8e78602 100644 (file)
@@ -1075,8 +1075,8 @@ LWLockDequeueSelf(LWLock *lock)
                 */
 
                /*
-                * Reset releaseOk if somebody woke us before we removed ourselves -
-                * they'll have set it to false.
+                * Reset RELEASE_OK flag if somebody woke us before we removed
+                * ourselves - they'll have set it to false.
                 */
                pg_atomic_fetch_or_u32(&lock->state, LW_FLAG_RELEASE_OK);
 
index 85a629f4fcef7fac01746c83f4004fcb1be03533..78fb90fb1bd6e655fc8b701ca3ed1cde60688c3a 100644 (file)
@@ -849,7 +849,7 @@ OldSerXidInit(void)
 /*
  * Record a committed read write serializable xid and the minimum
  * commitSeqNo of any transactions to which this xid had a rw-conflict out.
- * An invalid seqNo means that there were no conflicts out from xid.
+ * An invalid commitSeqNo means that there were no conflicts out from xid.
  */
 static void
 OldSerXidAdd(TransactionId xid, SerCommitSeqNo minConflictCommitSeqNo)
@@ -1685,7 +1685,7 @@ SetSerializableTransactionSnapshot(Snapshot snapshot,
 /*
  * Guts of GetSerializableTransactionSnapshot
  *
- * If sourcexid is valid, this is actually an import operation and we should
+ * If sourcevxid is valid, this is actually an import operation and we should
  * skip calling GetSnapshotData, because the snapshot contents are already
  * loaded up.  HOWEVER: to avoid race conditions, we must check that the
  * source xact is still running after we acquire SerializableXactHashLock.
index 2376bda497be1de102c67e7bfd3022bf9492451a..15fda7f122561dc7a5559c4edc7ce792130fdd8c 100644 (file)
@@ -79,7 +79,7 @@
 #undef StrNCpy
 #include <shlwapi.h>
 #ifdef StrNCpy
-#undef STrNCpy
+#undef StrNCpy
 #endif
 #endif
 
index 631e42106262883e955147e07dfefdd419ffc6bb..d01d3032ccacfa5dcae4c219d2f77bc51a6b4580 100644 (file)
@@ -1,6 +1,6 @@
 /*-------------------------------------------------------------------------
  *
- * ragetypes_typanalyze.c
+ * rangetypes_typanalyze.c
  *       Functions for gathering statistics from range columns
  *
  * For a range type column, histograms of lower and upper bounds, and
index 4483db573f3459b3f62d1c6077b077a854d89ba7..90b2c9b69488afe458db1643d3690d0c96d89bcf 100644 (file)
@@ -705,7 +705,7 @@ txid_snapshot_xip(PG_FUNCTION_ARGS)
        TxidSnapshot *snap;
        txid            value;
 
-       /* on first call initialize snap_state and get copy of snapshot */
+       /* on first call initialize fctx and get copy of snapshot */
        if (SRF_IS_FIRSTCALL())
        {
                TxidSnapshot *arg = (TxidSnapshot *) PG_GETARG_VARLENA_P(0);
index 0864838867f6d3a7ff9d4db4b8c55c32ec675d0a..fa08b55eb62b59cc0fe1c837279b09d49e460731 100644 (file)
@@ -2052,7 +2052,7 @@ varstr_sortsupport(SortSupport ssup, Oid typid, Oid collid)
 
        /*
         * If we're using abbreviated keys, or if we're using a locale-aware
-        * comparison, we need to initialize a StringSortSupport object.  Both
+        * comparison, we need to initialize a VarStringSortSupport object. Both
         * cases will make use of the temporary buffers we initialize here for
         * scratch space (and to detect requirement for BpChar semantics from
         * caller), and the abbreviation case requires additional state.
index 5e629d29ea8a4f4046f8519650d7920538400e70..0280c2625c65f21e8de4b5536b4f609170d4e6da 100644 (file)
@@ -3987,7 +3987,7 @@ xml_xpathobjtoxmlarray(xmlXPathObjectPtr xpathobj,
 /*
  * Common code for xpath() and xmlexists()
  *
- * Evaluate XPath expression and return number of nodes in res_items
+ * Evaluate XPath expression and return number of nodes in res_nitems
  * and array of XML values in astate.  Either of those pointers can be
  * NULL if the corresponding result isn't wanted.
  *
index d228bbed685b2ecd7186095f560d0c22c1478c4d..416a0875b6c938ebd06f655e6987304e35f80459 100755 (executable)
@@ -18,7 +18,7 @@
 ## can be ignored
 INTENTIONALLY_NOT_INCLUDED="debug_deadlocks \
 is_superuser lc_collate lc_ctype lc_messages lc_monetary lc_numeric lc_time \
-pre_auth_delay role seed server_encoding server_version server_version_int \
+pre_auth_delay role seed server_encoding server_version server_version_num \
 session_authorization trace_lock_oidmin trace_lock_table trace_locks trace_lwlocks \
 trace_notify trace_userlocks transaction_isolation transaction_read_only \
 zero_damaged_pages"
index bd8d20098296dfa55986e5b4bedf2057050045f4..700a91a2a374c4098f158ef743143dafa47acb1c 100644 (file)
@@ -182,7 +182,7 @@ static const MemoryContextMethods SlabMethods = {
  * chunkSize: allocation chunk size
  *
  * The chunkSize may not exceed:
- *             MAXALIGN_DOWN(SIZE_MAX) - MAXALIGN(sizeof(SlabBlock)) - SLAB_CHUNKHDRSZ
+ *             MAXALIGN_DOWN(SIZE_MAX) - MAXALIGN(sizeof(SlabBlock)) - sizeof(SlabChunk)
  */
 MemoryContext
 SlabContextCreate(MemoryContext parent,
index bda802b447f4d525c22f668bf4e70eaadb2ae195..d2bd2aee9f023307afd946a169b0f94a9b150126 100644 (file)
@@ -1150,7 +1150,7 @@ tuplesort_begin_datum(Oid datumType, Oid sortOperator, Oid sortCollation,
         * to compare.  In a tuple sort, we could support that, because we can
         * always extract the original datum from the tuple as needed.  Here, we
         * can't, because a datum sort only stores a single copy of the datum; the
-        * "tuple" field of each sortTuple is NULL.
+        * "tuple" field of each SortTuple is NULL.
         */
        state->sortKeys->abbreviate = !typbyval;
 
index 0f40328310b330a60d05567d4342ae1030a0efc7..6c739ce6633c5dafdb54c26e2f5dda0d62eb9f5c 100644 (file)
@@ -1989,7 +1989,7 @@ WriteInt(ArchiveHandle *AH, int i)
         * This is a bit yucky, but I don't want to make the binary format very
         * dependent on representation, and not knowing much about it, I write out
         * a sign byte. If you change this, don't forget to change the file
-        * version #, and modify readInt to read the new format AS WELL AS the old
+        * version #, and modify ReadInt to read the new format AS WELL AS the old
         * formats.
         */
 
index d378053de450ce6693f5bb6e38a24c3a908a43a2..15e3eab55050c6328a82e4f01f42f11254fdf321 100644 (file)
@@ -404,7 +404,7 @@ sanityChecks(void)
 {
        /* TODO Check that there's no backup_label in either cluster */
 
-       /* Check system_id match */
+       /* Check system_identifier match */
        if (ControlFile_target.system_identifier != ControlFile_source.system_identifier)
                pg_fatal("source and target clusters are from different systems");
 
index 57c4ed69132b96c1ccc4b2749b8211d78da369e4..b3bbb903fc69c06fba8e7c2ba838cb2adc88e016 100644 (file)
@@ -138,7 +138,7 @@ typedef struct
        char       *nspname;            /* namespace name */
        char       *relname;            /* relation name */
        Oid                     reloid;                 /* relation OID */
-       Oid                     relfilenode;    /* relation relfile node */
+       Oid                     relfilenode;    /* relation file node */
        Oid                     indtable;               /* if index, OID of its table, else 0 */
        Oid                     toastheap;              /* if toast table, OID of base table, else 0 */
        char       *tablespace;         /* tablespace path; "" for cluster default */
index 2ade88bc1e5ae4659b7f70e742befa016cd94d27..4351dcca1f89465e45f1428e889632dc302cdb7d 100644 (file)
@@ -31,7 +31,7 @@ extern void brinRevmapExtend(BrinRevmap *revmap,
                                                         BlockNumber heapBlk);
 extern Buffer brinLockRevmapPageForUpdate(BrinRevmap *revmap,
                                                                                  BlockNumber heapBlk);
-extern void brinSetHeapBlockItemptr(Buffer rmbuf, BlockNumber pagesPerRange,
+extern void brinSetHeapBlockItemptr(Buffer buf, BlockNumber pagesPerRange,
                                                                        BlockNumber heapBlk, ItemPointerData tid);
 extern BrinTuple *brinGetTupleForHeapBlock(BrinRevmap *revmap,
                                                                                   BlockNumber heapBlk, Buffer *buf, OffsetNumber *off,
index 53b682c56ef39d2638a1f09398f606f1f490b72b..7d52145a2a74f2dea77f82069dbae01e82852f94 100644 (file)
 #define XLH_SPLIT_META_UPDATE_MASKS            (1<<0)
 #define XLH_SPLIT_META_UPDATE_SPLITPOINT               (1<<1)
 
-/*
- * This is what we need to know about a HASH index create.
- *
- * Backup block 0: metapage
- */
-typedef struct xl_hash_createidx
-{
-       double          num_tuples;
-       RegProcedure procid;
-       uint16          ffactor;
-}                      xl_hash_createidx;
-#define SizeOfHashCreateIdx (offsetof(xl_hash_createidx, ffactor) + sizeof(uint16))
-
 /*
  * This is what we need to know about simple (without split) insert.
  *
index a1912f41e631dc470e4e7939182cc6fa6d76c8fe..6d392e4d5a4a74d25f43b59e81f14301450ca885 100644 (file)
@@ -237,7 +237,7 @@ typedef struct
 
 /*
  * For use by code reading options already parsed: get a pointer to the string
- * value itself.  "optstruct" is the StdRdOption struct or equivalent, "member"
+ * value itself.  "optstruct" is the StdRdOptions struct or equivalent, "member"
  * is the struct member corresponding to the string option
  */
 #define GET_STRING_RELOPTION(optstruct, member) \
index a12c94cba67db406f80701572a3a5e39cae677bb..aa9bc637259fd6875cfefa695e86f236df0eca08 100644 (file)
@@ -248,7 +248,7 @@ extern bool DecodeXLogRecord(XLogReaderState *state, XLogRecord *record,
 extern FullTransactionId XLogRecGetFullXid(XLogReaderState *record);
 #endif
 
-extern bool RestoreBlockImage(XLogReaderState *recoder, uint8 block_id, char *dst);
+extern bool RestoreBlockImage(XLogReaderState *record, uint8 block_id, char *page);
 extern char *XLogRecGetBlockData(XLogReaderState *record, uint8 block_id, Size *len);
 extern bool XLogRecGetBlockTag(XLogReaderState *record, uint8 block_id,
                                                           RelFileNode *rnode, ForkNumber *forknum,
index 187d76de18a410d96ea5ab6cd62c33cdbaba7428..edb98842fcf2776f67406d6574cbbf1ad770047a 100644 (file)
@@ -105,7 +105,7 @@ extern Oid  ConversionGetConid(const char *conname);
 extern bool ConversionIsVisible(Oid conid);
 
 extern Oid     get_statistics_object_oid(List *names, bool missing_ok);
-extern bool StatisticsObjIsVisible(Oid stxid);
+extern bool StatisticsObjIsVisible(Oid relid);
 
 extern Oid     get_ts_parser_oid(List *names, bool missing_ok);
 extern bool TSParserIsVisible(Oid prsId);
index 4e2fb39105b2e223d02956c43961a9e57142e27a..3cbb08df92a05150224bfdde7d9f77136dae0b51 100644 (file)
@@ -768,8 +768,8 @@ typedef enum AggStrategy
 /* Primitive options supported by nodeAgg.c: */
 #define AGGSPLITOP_COMBINE             0x01    /* substitute combinefn for transfn */
 #define AGGSPLITOP_SKIPFINAL   0x02    /* skip finalfn, return state as-is */
-#define AGGSPLITOP_SERIALIZE   0x04    /* apply serializefn to output */
-#define AGGSPLITOP_DESERIALIZE 0x08    /* apply deserializefn to input */
+#define AGGSPLITOP_SERIALIZE   0x04    /* apply serialfn to output */
+#define AGGSPLITOP_DESERIALIZE 0x08    /* apply deserialfn to input */
 
 /* Supported operating modes (i.e., useful combinations of these options): */
 typedef enum AggSplit
index f4841fb3975d49749629c83dc20a17e064d48270..1cf166a570dc40587fdaf53f593820f8c2138835 100644 (file)
@@ -451,7 +451,7 @@ int                 pgwin32_bind(SOCKET s, struct sockaddr *addr, int addrlen);
 int                    pgwin32_listen(SOCKET s, int backlog);
 SOCKET         pgwin32_accept(SOCKET s, struct sockaddr *addr, int *addrlen);
 int                    pgwin32_connect(SOCKET s, const struct sockaddr *name, int namelen);
-int                    pgwin32_select(int nfds, fd_set *readfs, fd_set *writefds, fd_set *exceptfds, const struct timeval *timeout);
+int                    pgwin32_select(int nfds, fd_set *readfds, fd_set *writefds, fd_set *exceptfds, const struct timeval *timeout);
 int                    pgwin32_recv(SOCKET s, char *buf, int len, int flags);
 int                    pgwin32_send(SOCKET s, const void *buf, int len, int flags);
 int                    pgwin32_waitforsinglesocket(SOCKET s, int what, int timeout);
index 62e756ff6387d8f71ebf3c8ee726fca7c8ba8950..07ca649359f8ef82988a883ac10c26a09c70e2e1 100644 (file)
@@ -43,4 +43,4 @@ typedef struct RuleLock
        RewriteRule **rules;
 } RuleLock;
 
-#endif                                                 /* REWRITE_H */
+#endif                                                 /* PRS2LOCK_H */
index befdbf60bf813d89705736f74ae0ea0d6d6b5e92..9a612a5834baa55487cb8de8e6b81bed1f83f8d4 100644 (file)
@@ -1,6 +1,6 @@
 /*-------------------------------------------------------------------------
  *
- * standbydef.h
+ * standbydefs.h
  *        Frontend exposed definitions for hot standby mode.
  *
  *
index dbfd8efd269656d82e8cb311dc81ed75b608fe91..ba0b7f6f79cd9ba0288c1580468d87a144314792 100644 (file)
@@ -63,7 +63,7 @@
        (PGSIXBIT(ch1) + (PGSIXBIT(ch2) << 6) + (PGSIXBIT(ch3) << 12) + \
         (PGSIXBIT(ch4) << 18) + (PGSIXBIT(ch5) << 24))
 
-/* These macros depend on the fact that '0' becomes a zero in SIXBIT */
+/* These macros depend on the fact that '0' becomes a zero in PGSIXBIT */
 #define ERRCODE_TO_CATEGORY(ec)  ((ec) & ((1 << 12) - 1))
 #define ERRCODE_IS_CATEGORY(ec)  (((ec) & ~((1 << 12) - 1)) == 0)
 
index a93ed77c9c34ec0185aec2c52c1d74eaa1dbde9a..6791e0cbc20ac0b6279565637e44ffdafc9b6ca1 100644 (file)
@@ -365,7 +365,7 @@ extern int  set_config_option(const char *name, const char *value,
                                                          GucContext context, GucSource source,
                                                          GucAction action, bool changeVal, int elevel,
                                                          bool is_reload);
-extern void AlterSystemSetConfigFile(AlterSystemStmt *setstmt);
+extern void AlterSystemSetConfigFile(AlterSystemStmt *altersysstmt);
 extern char *GetConfigOptionByName(const char *name, const char **varname,
                                                                   bool missing_ok);
 extern void GetConfigOptionByNum(int varnum, const char **values, bool *noshow);
index b0fe19ebc549fd8de24f085d9470d0418ab39c8d..2174bbafd14d689396b00bdb80134d43c80a8090 100644 (file)
@@ -75,7 +75,7 @@ typedef struct RelationData
         * when a relation has multiple new relfilenodes within a single
         * transaction, with one of them occurring in a subsequently aborted
         * subtransaction, e.g. BEGIN; TRUNCATE t; SAVEPOINT save; TRUNCATE t;
-        * ROLLBACK TO save; -- rd_newRelfilenode is now forgotten
+        * ROLLBACK TO save; -- rd_newRelfilenodeSubid is now forgotten
         */
        SubTransactionId rd_createSubid;        /* rel was created in current xact */
        SubTransactionId rd_newRelfilenodeSubid;        /* new relfilenode assigned in
index 85d9ecbfc6ea980c5d7602e8cd747e98858a9bdc..9554952a8a1e345658d0bcb28e0d4fa3115ad757 100644 (file)
@@ -109,7 +109,7 @@ typedef struct
        double          numIndexPages;  /* number of leaf pages visited */
        double          numIndexTuples; /* number of leaf tuples visited */
        double          spc_random_page_cost;   /* relevant random_page_cost value */
-       double          num_sa_scans;   /* # indexscans from ScalarArrayOps */
+       double          num_sa_scans;   /* # indexscans from ScalarArrayOpExprs */
 } GenericCosts;
 
 /* Hooks for plugins to get control when we ask for stats */
index 8fc0a467f7f5f1e2b8721c801b92e6e531de8e41..6ef1fadb7fc824f35b5978515009da8d54c8070d 100644 (file)
@@ -679,11 +679,6 @@ intoasc(interval * i, char *str)
        return 0;
 }
 
-/*
- *     rfmt.c  -  description
- *     by Carsten Wolff <carsten.wolff@credativ.de>, Wed Apr 2 2003
- */
-
 static struct
 {
        long            val;
@@ -745,7 +740,7 @@ initValue(long lng_val)
        return 0;
 }
 
-/* return the position oft the right-most dot in some string */
+/* return the position of the right-most dot in some string */
 static int
 getRightMostDot(const char *str)
 {
index 3151975564af72548518ee2018e02b05109cd977..22c4954f2b71535c29721422348d6c3028723c92 100644 (file)
@@ -287,7 +287,7 @@ extern PQconninfoOption *PQconninfo(PGconn *conn);
 extern void PQconninfoFree(PQconninfoOption *connOptions);
 
 /*
- * close the current connection and restablish a new one with the same
+ * close the current connection and reestablish a new one with the same
  * parameters
  */
 /* Asynchronous (non-blocking) */
index 53fd36edba166207a8908e32a7fc7772d32ae651..5dfb8a4cb5467c1c751ea59d8b0fac9b1b6e0ebb 100644 (file)
@@ -132,7 +132,7 @@ PLy_subtransaction_enter(PyObject *self, PyObject *unused)
  *
  * Exit an explicit subtransaction. exc_type is an exception type, exc
  * is the exception object, tb is the traceback.  If exc_type is None,
- * commit the subtransactiony, if not abort it.
+ * commit the subtransaction, if not abort it.
  *
  * The method signature is chosen to allow subtransaction objects to
  * be used as context managers as described in
index 7f53afbea2630b65132a96bb723db54cb70aa7e1..536c5f686e2aba1bc1497c486623793bbb947a4f 100644 (file)
@@ -49,8 +49,8 @@ typedef void (__stdcall * freeaddrinfo_ptr_t) (struct addrinfo *ai);
 
 typedef int (__stdcall * getnameinfo_ptr_t) (const struct sockaddr *sa,
                                                                                         int salen,
-                                                                                        char *host, int hostlen,
-                                                                                        char *serv, int servlen,
+                                                                                        char *node, int nodelen,
+                                                                                        char *service, int servicelen,
                                                                                         int flags);
 
 /* static pointers to the native routines, so we only do the lookup once. */
index 5c51fcdae686c44854e8bbef000320ac24bcf6b9..a04fd1cc7e8cb1d9fbcc26f1d0d57d7c7ea8ad7e 100644 (file)
@@ -29,7 +29,7 @@ permutation "s1alter" "s1commit" "s2nv"
 # change, but now it waits.
 permutation "s1alter" "s2nv" "s1commit"
 
-# Prior to PG10, the s2nv step would see the uncommitted s1reset
+# Prior to PG10, the s2nv step would see the uncommitted s1restart
 # change, but now it waits.
 permutation "s1restart" "s2nv" "s1commit"
 
index c1845536942b0ded357a94372eb54cc32fc117ec..292cd40fe2d4addcadba7e11011d2b19e2cced25 100644 (file)
@@ -76,7 +76,7 @@ is($psql_out, '6', "Check nextXid handling for prepared subtransactions");
 
 ###############################################################################
 # Check that replay will correctly set 2PC with more than
-# PGPROC_MAX_CACHED_SUBXIDS subtransations and also show data properly
+# PGPROC_MAX_CACHED_SUBXIDS subtransactions and also show data properly
 # on promotion
 ###############################################################################
 $node_master->psql('postgres', "DELETE FROM t_012_tbl");
index 173f995d240f0c195aa1fa47cb4230a21c010c0c..89129d7358a116b4ad69aa83aa406029007efaa4 100644 (file)
@@ -6,6 +6,6 @@
 /results/
 /log/
 
-# Note: regreesion.* are only left behind on a failure; that's why they're not ignored
+# Note: regression.* are only left behind on a failure; that's why they're not ignored
 #/regression.diffs
 #/regression.out
index 1d5bd90afb733e52d9c831b36221d29354fa4b9d..4b3829b0b5361a123d39c175145196eb7084faca 100644 (file)
@@ -357,7 +357,7 @@ pg_tzset_offset(long gmtoffset)
  * is to ensure that log_timezone has a valid value before any logging GUC
  * variables could become set to values that require elog.c to provide
  * timestamps (e.g., log_line_prefix).  We may as well initialize
- * session_timestamp to something valid, too.
+ * session_timezone to something valid, too.
  */
 void
 pg_timezone_initialize(void)
index 00bffcf005ad74da557761852786babd62b12571..8eb15fafb996eccf96e3978abecf91568b0c44c5 100644 (file)
@@ -31,7 +31,7 @@ DOING THE INDENT RUN:
 
        wget -O src/tools/pgindent/typedefs.list https://buildfarm.postgresql.org/cgi-bin/typedefs.pl
 
-   (See https://www.pgbuildfarm.org/cgi-bin/typedefs.pl?show_list for a full
+   (See https://buildfarm.postgresql.org/cgi-bin/typedefs.pl?show_list for a full
    list of typedef files, if you want to indent some back branch.)
 
 3) Run pgindent on the C files: