]> git.ipfire.org Git - thirdparty/postgresql.git/commitdiff
Fix a set of typos and grammar issues across the tree
authorMichael Paquier <michael@paquier.xyz>
Tue, 21 Apr 2026 05:46:22 +0000 (14:46 +0900)
committerMichael Paquier <michael@paquier.xyz>
Tue, 21 Apr 2026 05:46:22 +0000 (14:46 +0900)
This batch is similar to 462fe0ff6215 and addresses a variety of code
style issues, including grammar mistakes, typos, inconsistent variable
names in function declarations, and incorrect function names in comments
and documentation.  These fixes have accumulated on the community
mailing lists since the commit mentioned above.

Notably, Alexander Lakhin previously submitted a patch identifying many
of the trivial typos and grammar issues that had been reported on
pgsql-hackers.  His patch covered a somewhat large portion of the issues
addressed here, though not all of them.

The documentation changes only affect HEAD.

68 files changed:
contrib/btree_gist/btree_utils_var.h
contrib/pg_overexplain/pg_overexplain.c
contrib/pg_plan_advice/pgpa_ast.c
contrib/pg_plan_advice/pgpa_ast.h
contrib/pg_plan_advice/pgpa_identifier.c
contrib/pg_plan_advice/pgpa_join.c
contrib/pg_plan_advice/pgpa_output.c
contrib/pg_plan_advice/pgpa_planner.c
contrib/pg_stash_advice/stashfuncs.c
contrib/pg_stash_advice/stashpersist.c
contrib/pg_trgm/trgm_op.c
contrib/postgres_fdw/postgres_fdw.c
doc/src/sgml/pgstashadvice.sgml
doc/src/sgml/ref/alter_property_graph.sgml
doc/src/sgml/ref/alter_table.sgml
doc/src/sgml/ref/cluster.sgml
doc/src/sgml/release-19.sgml
src/backend/access/common/indextuple.c
src/backend/access/common/tidstore.c
src/backend/access/heap/heapam_visibility.c
src/backend/access/rmgrdesc/gindesc.c
src/backend/access/transam/commit_ts.c
src/backend/access/transam/multixact.c
src/backend/access/transam/xlog.c
src/backend/commands/explain_state.c
src/backend/commands/repack.c
src/backend/commands/vacuumparallel.c
src/backend/executor/execTuples.c
src/backend/executor/nodeModifyTable.c
src/backend/lib/hyperloglog.c
src/backend/libpq/be-secure-openssl.c
src/backend/optimizer/path/indxpath.c
src/backend/parser/analyze.c
src/backend/parser/parse_graphtable.c
src/backend/postmaster/autovacuum.c
src/backend/postmaster/datachecksum_state.c
src/backend/replication/logical/origin.c
src/backend/replication/logical/worker.c
src/backend/rewrite/rewriteGraphTable.c
src/backend/statistics/extended_stats_funcs.c
src/backend/storage/aio/method_worker.c
src/backend/storage/buffer/bufmgr.c
src/backend/storage/ipc/ipci.c
src/backend/storage/lmgr/lock.c
src/backend/storage/lmgr/predicate.c
src/backend/utils/adt/network.c
src/backend/utils/adt/ruleutils.c
src/backend/utils/error/elog.c
src/bin/pg_combinebackup/t/011_ib_truncation.pl
src/bin/pg_dump/pg_dump.c
src/bin/pg_dump/pg_restore.c
src/bin/pg_resetwal/pg_resetwal.c
src/bin/pg_upgrade/check.c
src/bin/pg_upgrade/controldata.c
src/common/wchar.c
src/include/access/tableam.h
src/include/postmaster/datachecksum_state.h
src/interfaces/libpq/fe-protocol3.c
src/test/modules/test_checksums/t/003_standby_restarts.pl
src/test/modules/test_checksums/t/005_injection.pl
src/test/modules/test_checksums/test_checksums.c
src/test/modules/test_extensions/t/001_extension_control_path.pl
src/test/recovery/t/026_overwrite_contrecord.pl
src/test/regress/expected/fast_default.out
src/test/regress/sql/fast_default.sql
src/tools/pg_bsd_indent/indent.c
src/tools/pg_bsd_indent/io.c
src/tools/pg_bsd_indent/pr_comment.c

index 6cb3aadf3c33e023ff368124caeae1c3c1cb39fd..f28a5d98ae35da4b773ad7a7726c8d32c12d141a 100644 (file)
@@ -42,7 +42,7 @@ typedef struct
 } gbtree_vinfo;
 
 /*
- * Free ptr1 in case its a copy of ptr2.
+ * Free ptr1 in case it's a copy of ptr2.
  *
  * This is adapted from varlena's PG_FREE_IF_COPY, though doesn't require
  * fcinfo access.
index fb277e023089a43a5542e1027c3e5fbe4258e5c7..a93a4dcfed6ef2bb461cbf407741e0079d99b051 100644 (file)
@@ -459,7 +459,7 @@ overexplain_range_table(PlannedStmt *plannedstmt, ExplainState *es)
                char       *relkind;
                SubPlanRTInfo *next_rtinfo;
 
-               /* Advance to next SubRTInfo, if it's time. */
+               /* Advance to next SubPlanRTInfo, if it's time. */
                if (lc_subrtinfo != NULL)
                {
                        next_rtinfo = lfirst(lc_subrtinfo);
@@ -512,8 +512,8 @@ overexplain_range_table(PlannedStmt *plannedstmt, ExplainState *es)
                                /*
                                 * We should not see RTE of this kind here since property
                                 * graph RTE gets converted to subquery RTE in
-                                * RewriteGraphTable(). In case we decide not to do the
-                                * conversion and leave RTEkind unchanged in future, print
+                                * rewriteGraphTable(). In case we decide not to do the
+                                * conversion and leave RTE kind unchanged in future, print
                                 * correct name of RTE kind.
                                 */
                                kind = "graph_table";
index 3c340c6ae7a00540a60feca30f9099b77a4fdd29..01db8d24cd03b05c2458f4f8e05c53c3496e37e8 100644 (file)
@@ -321,7 +321,7 @@ pgpa_identifiers_match_target(int nrids, pgpa_identifier *rids,
  * Returns true if every target or sub-target is matched by at least one
  * identifier, and otherwise false.
  *
- * Also sets rids_used[i] = true for each idenifier that matches at least one
+ * Also sets rids_used[i] = true for each identifier that matches at least one
  * target.
  */
 static bool
index a89f125192959224eed0089dba97a7a2dfcfe6f4..4bd6ffa5e3a58722090d1b73f40bb853fd76ee87 100644 (file)
@@ -116,7 +116,7 @@ typedef struct pgpa_advice_item
 } pgpa_advice_item;
 
 /*
- * Result of comparing an array of pgpa_relation_identifier objects to a
+ * Result of comparing an array of pgpa_identifier objects to a
  * pgpa_advice_target.
  *
  * PGPA_ITM_EQUAL means all targets are matched by some identifier, and
index 0cfc4aa4f7eaa5820aa9b5432a26c0f580bac4ad..21392b8e0d7e449b3fcde04b431eebdbd0a34b13 100644 (file)
@@ -211,7 +211,7 @@ pgpa_compute_identifier_by_rti(PlannerInfo *root, Index rti,
  * RTE_JOIN entries are excluded because they cannot be mentioned by plan
  * advice.
  *
- * The caller is responsible for making sure that the tkeys array is large
+ * The caller is responsible for making sure that the "rids" array is large
  * enough to store the results.
  *
  * The return value is the number of identifiers computed.
index 38e7b91ed7eb876d85208bfad2e22a8c788c2643..067321081e7b4d707b02461287783639134d9cd4 100644 (file)
@@ -93,7 +93,7 @@ pgpa_create_join_unroller(void)
  *
  * pgpa_plan_walker creates a "top level" join unroller object when it
  * encounters a join in a portion of the plan tree in which no join unroller
- * is already active. From there, this function is responsible for determing
+ * is already active. From there, this function is responsible for determining
  * to what portion of the plan tree that join unroller applies, and for
  * creating any subordinate join unroller objects that are needed as a result
  * of non-outer-deep join trees. We do this by returning the join unroller
index ed5a9340671668e23d6f6a0346ad355da70837dd..ffced69664a26e409620be95ebaaa3f0e71b7d58 100644 (file)
@@ -115,7 +115,7 @@ pgpa_output_advice(StringInfo buf, pgpa_plan_walker_context *walker,
 
        /*
         * Each piece of JOIN_ORDER() advice fully describes the join order for a
-        * single unrolled join. Merging is not permitted, because that would
+        * single unrolled join. Merging is not permitted, because that would
         * change the meaning, e.g. SEQ_SCAN(a b c d) means simply that sequential
         * scans should be used for all of those relations, and is thus equivalent
         * to SEQ_SCAN(a b) SEQ_SCAN(c d), but JOIN_ORDER(a b c d) means that "a"
index a86ebf987922933021e67fa00da9da6c44dbec39..b3329b793aa8e28fb1e15714b216bba5c31ed398 100644 (file)
@@ -1657,13 +1657,14 @@ pgpa_planner_apply_scan_advice(RelOptInfo *rel,
                        /*
                         * Currently, PGS_CONSIDER_INDEXONLY can suppress Bitmap Heap
                         * Scans, so don't clear it when such a scan is requested. This
-                        * happens because build_index_scan() thinks that the possibility
-                        * of an index-only scan is a sufficient reason to consider using
-                        * an otherwise-useless index, and get_index_paths() thinks that
-                        * the same paths that are useful for index or index-only scans
-                        * should also be considered for bitmap scans. Perhaps that logic
-                        * should be tightened up, but until then we need to include
-                        * PGS_CONSIDER_INDEXONLY in my_scan_type here.
+                        * happens because build_index_scankeys() thinks that the
+                        * possibility of an index-only scan is a sufficient reason to
+                        * consider using an otherwise-useless index, and
+                        * get_index_paths() thinks that the same paths that are useful
+                        * for index or index-only scans should also be considered for
+                        * bitmap scans. Perhaps that logic should be tightened up, but
+                        * until then we need to include PGS_CONSIDER_INDEXONLY in
+                        * my_scan_type here.
                         */
                        my_scan_type = PGS_BITMAPSCAN | PGS_CONSIDER_INDEXONLY;
                }
@@ -2092,7 +2093,7 @@ pgpa_compute_rt_offsets(pgpa_planner_state *pps, PlannedStmt *pstmt)
 
                /*
                 * It's not guaranteed that every plan name we saw during planning has
-                * a SubPlanInfo, but any that do not certainly don't appear in the
+                * a SubPlanRTInfo, but any that do not certainly don't appear in the
                 * final range table.
                 */
                foreach_node(SubPlanRTInfo, rtinfo, pstmt->subrtinfos)
index 77f8e19e86770c712db5cb88a6d32432088ee332..d7aa9f2223f4984bd593fd3fed8cb383c71800bc 100644 (file)
@@ -286,7 +286,7 @@ pg_set_stashed_advice(PG_FUNCTION_ARGS)
        /*
         * Get and check query ID.
         *
-        * queryID 0 means no query ID was computed, so reject that.
+        * Query ID 0 means no query ID was computed, so reject that.
         */
        queryId = PG_GETARG_INT64(1);
        if (queryId == 0)
index 07a4da65b7e09a253072a7a3e14d76210e0a3d58..00a0a74f04df3f175d391cb8c14f6254cee610e8 100644 (file)
@@ -85,7 +85,7 @@ static void pgsa_write_to_disk(void);
 /*
  * Background worker entry point for pg_stash_advice persistence.
  *
- * On startup, if load_from_disk_pending is set, we load previously saved
+ * On startup, if stashes_ready is set, we load previously saved
  * stash data from disk.  Then we enter a loop, periodically checking whether
  * any changes have been made (via the change_count atomic counter) and
  * writing them to disk.  On shutdown, we perform a final write.
index 0aca9b5826f23cfc69031660219b1013cfcaf247..3fd087852e7828b5d58c047897aeb8f4fb46106d 100644 (file)
@@ -242,7 +242,7 @@ CMPTRGM_CHOOSE(const void *a, const void *b)
 #define ST_DECLARE
 #include "lib/sort_template.h"
 
-/* Sort an array of trigrams, handling signedess correctly */
+/* Sort an array of trigrams, handling signedness correctly */
 static void
 trigram_qsort(trgm *array, size_t n)
 {
index 0f20f38c83e49a6843da9665c64d3c2d045d4930..c42cb690c7bc9e976adbbd5755af29335d5b395c 100644 (file)
@@ -5798,7 +5798,7 @@ fetch_remote_statistics(Relation relation,
                                                                                                          remote_relname,
                                                                                                          column_list.data);
 
-                       /* If any attribute statsare missing, fallback to sampling. */
+                       /* If any attribute stats are missing, fallback to sampling. */
                        if (!match_attrmap(attstats,
                                                           local_schemaname, local_relname,
                                                           remote_schemaname, remote_relname,
index c9b1e078382f9071be4c3afef73bf8a6bdac0349..7813d63d91e97ee50969da5ad8b0776a0f371012 100644 (file)
@@ -84,7 +84,7 @@
   <literal>pg_stash_advice.stash_name</literal> for their session, and this
   may reveal the contents of any advice stash with that name. Users should
   assume that information embedded in stashed advice strings may become visible
-  to nonprivileged users.
+  to non-privileged users.
  </para>
 
  <sect2 id="pgstashadvice-functions">
index 19352c06305f36bd43dacda3d6b0fd08e330c1c9..f517f2b2d7a7d0b7772995033b2839eb42558afa 100644 (file)
@@ -78,7 +78,7 @@ ALTER PROPERTY GRAPH [ IF EXISTS ] <replaceable class="parameter">name</replacea
       <para>
        This form removes vertex or edge tables from the property graph.  (Only
        the association of the tables with the graph is removed.  The tables
-       themself are not dropped.)
+       themselves are not dropped.)
       </para>
      </listitem>
     </varlistentry>
index 453395c5c7365a0a8bac60fa0d26622997881376..54f38cb964ab53cba5b1bd3a997bd4e9fdcefc2a 100644 (file)
@@ -279,7 +279,7 @@ WITH ( MODULUS <replaceable class="parameter">numeric_literal</replaceable>, REM
       This form replaces the expression of a generated column.  Existing data
       in a stored generated column is rewritten and all the future changes
       will apply the new generation expression.
-      Replacing the expression of a virtual generated column do not require a
+      Replacing the expression of a virtual generated column does not require a
       table rewrite, but if the column is used in a constraint, the table will
       be scanned to check that existing rows meet the constraint.
      </para>
index 17778e9471c426ee7bd3ede8ce69a3230685f246..ffb3ff898c69a7d34fb71a5fccf1c70d2508e675 100644 (file)
@@ -34,7 +34,7 @@ CLUSTER [ ( <replaceable class="parameter">option</replaceable> [, ...] ) ] [ <r
 
   <para>
    The <command>CLUSTER</command> command is equivalent to
-   <xref linkend="sql-repack"/> with an <literal>USING INDEX</literal>
+   <xref linkend="sql-repack"/> with a <literal>USING INDEX</literal>
    clause.  See there for more details.
   </para>
  </refsect1>
index 9636b41ac1e64b9cc5b136ccf4cd0b5ada39052d..6e519a8fdf799d3aa951f799f5f51c8113f515fc 100644 (file)
@@ -3231,7 +3231,7 @@ Add functions to pg_buffercache to mark buffers as dirty (Nazir Bilal Yavuz)
 </para>
 
 <para>
-The functions are pg_buffercache_mark_dirty(), pg_buffercache_mark_dirt_relation(), and pg_buffercache_mark_dirty_all().
+The functions are pg_buffercache_mark_dirty(), pg_buffercache_mark_dirty_relation(), and pg_buffercache_mark_dirty_all().
 </para>
 </listitem>
 
index a2db55e9b7367ae76276373fd78ef7d6ffd40bbb..60bba0a21459a490630cc18cabb8a9ee32dfad17 100644 (file)
@@ -222,7 +222,7 @@ index_form_tuple_context(TupleDesc tupleDescriptor,
  *             nocache_index_getattr
  *
  *             This gets called from index_getattr() macro, and only in cases
- *             where we can't use cacheoffset and the value is not null.
+ *             where we can't use attcacheoff and the value is not null.
  * ----------------
  */
 Datum
index f208c7938c20bc7c3a750542d5fd6d08d3433929..de3e44d0f5d66567e1a45668c5c88a7d2c6cf717 100644 (file)
@@ -254,7 +254,7 @@ TidStoreAttach(dsa_handle area_handle, dsa_pointer handle)
 
        area = dsa_attach(area_handle);
 
-       /* Find the shared the shared radix tree */
+       /* Find the shared radix tree */
        ts->tree.shared = shared_ts_attach(area, handle);
        ts->area = area;
 
index 3a6a1e5a0849daa508e90514a8d2ceb0ead6937b..361b76e50652896e8035c58dea646aece793aa36 100644 (file)
@@ -192,7 +192,7 @@ SetHintBitsExt(HeapTupleHeader tuple, Buffer buffer,
 }
 
 /*
- * Simple wrapper around SetHintBitExt(), use when operating on a single
+ * Simple wrapper around SetHintBitsExt(), use when operating on a single
  * tuple.
  */
 static inline void
@@ -1671,7 +1671,7 @@ HeapTupleSatisfiesHistoricMVCC(HeapTuple htup, Snapshot snapshot,
 }
 
 /*
- * Perform HeaptupleSatisfiesMVCC() on each passed in tuple. This is more
+ * Perform HeapTupleSatisfiesMVCC() on each passed in tuple. This is more
  * efficient than doing HeapTupleSatisfiesMVCC() one-by-one.
  *
  * To be checked tuples are passed via BatchMVCCState->tuples. Each tuple's
index 66a0bf0e0e7672c50ecf2576ce5471dc5a5d310c..4e24a1cdfe7257f091309f80f3975cbc1a39fafe 100644 (file)
@@ -126,7 +126,7 @@ gin_desc(StringInfo buf, XLogReaderState *record)
                                ginxlogSplit *xlrec = (ginxlogSplit *) rec;
 
                                appendStringInfo(buf, "isrootsplit: %c",
-                                                                (((ginxlogSplit *) rec)->flags & GIN_SPLIT_ROOT) ? 'T' : 'F');
+                                                                (xlrec->flags & GIN_SPLIT_ROOT) ? 'T' : 'F');
                                appendStringInfo(buf, " isdata: %c isleaf: %c",
                                                                 (xlrec->flags & GIN_INSERT_ISDATA) ? 'T' : 'F',
                                                                 (xlrec->flags & GIN_INSERT_ISLEAF) ? 'T' : 'F');
index 2625cbf93bf92b79a7fef69a75b4a11e1f26c8d5..9e6fd5d465722b707268c7bc60e0bdf92c148834 100644 (file)
@@ -299,7 +299,7 @@ TransactionIdGetCommitTsData(TransactionId xid, TimestampTz *ts,
                /* frozen and bootstrap xids are always committed far in the past */
                *ts = 0;
                if (nodeid)
-                       *nodeid = 0;
+                       *nodeid = InvalidReplOriginId;
                return false;
        }
 
index cb78ba0842d1511b65b1d62b24bfef76a67e6f30..10cbc0d76bd7a9dd8b4c3d6cbfea80284dfc6c20 100644 (file)
@@ -341,8 +341,8 @@ static void ExtendMultiXactMember(MultiXactOffset offset, int nmembers);
 static void SetOldestOffset(void);
 static bool find_multixact_start(MultiXactId multi, MultiXactOffset *result);
 static void WriteMTruncateXlogRec(Oid oldestMultiDB,
-                                                                 MultiXactId endTruncOff,
-                                                                 MultiXactOffset endTruncMemb);
+                                                                 MultiXactId oldestMulti,
+                                                                 MultiXactOffset oldestOffset);
 
 
 /*
index f85b5286086e55fca5cefb6e27631176c8ccee15..e39af79c03b5e3dc23a7a2235819b586c9a069a3 100644 (file)
@@ -668,7 +668,7 @@ static TimeLineID LocalMinRecoveryPointTLI;
 static bool updateMinRecoveryPoint = true;
 
 /*
- * Local state for Controlfile data_checksum_version.  After initialization
+ * Local state for ControlFile data_checksum_version.  After initialization
  * this is only updated when absorbing a procsignal barrier during interrupt
  * processing.  The reason for keeping a copy in backend-private memory is to
  * avoid locking for interrogating the data checksum state.  Possible values
@@ -5045,7 +5045,7 @@ check_wal_buffers(int *newval, void **extra, GucSource source)
        {
                /*
                 * If we haven't yet changed the boot_val default of -1, just let it
-                * be.  We'll fix it when XLOGShmemSize is called.
+                * be.  We'll fix it when XLOGShmemRequest is called.
                 */
                if (XLOGbuffers == -1)
                        return true;
index 0e07a63fca6e5c3f21b5301603ddda9a02c6958b..a0ee0a664be0ef5371c8b319aa469638bf99a8f4 100644 (file)
@@ -435,7 +435,7 @@ GUCCheckExplainExtensionOption(const char *option_name,
  * for an EXPLAIN extension option, the caller is entitled to assume that
  * a suitably constructed DefElem passed to the main option handler will
  * not cause an error. To construct this DefElem, the caller should set
- * the DefElem's defname to option_name. If option_values is NULL, arg
+ * the DefElem's defname to option_name. If option_value is NULL, arg
  * should be NULL. Otherwise, arg should be of the type given by
  * option_type, with option_value as the associated value. The only option
  * types that should be passed are T_String, T_Float, and T_Integer; in
index 67364cc60e3fbdcd89fd5908a61c340f6ffdd5c3..4a9dc7b164da2e033899c960231b59ed1d906794 100644 (file)
@@ -104,7 +104,7 @@ typedef struct ChangeContext
        /* The relation the changes are applied to. */
        Relation        cc_rel;
 
-       /* Needed to update indexes of rel_dst. */
+       /* Needed to update indexes of cc_rel. */
        ResultRelInfo *cc_rri;
        EState     *cc_estate;
 
index 979c2be4abd741bd2bf634bbe658c6346a207332..41cefcfde54fed40b0f9738484ca2e05f2187616 100644 (file)
@@ -545,7 +545,7 @@ parallel_vacuum_end(ParallelVacuumState *pvs, IndexBulkDeleteResult **istats)
 /*
  * DSM detach callback. This is invoked when an autovacuum worker detaches
  * from the DSM segment holding PVShared. It ensures to reset the local pointer
- * to the shared state even if paralell vacuum raises an error and doesn't
+ * to the shared state even if parallel vacuum raises an error and doesn't
  * call parallel_vacuum_end().
  */
 static void
index f08982a43cc07f89a10553b80c74ab744ab4faa5..b0a0028b165bd7fb449c9979e9357b6dc620dafa 100644 (file)
@@ -512,7 +512,7 @@ tts_minimal_init(TupleTableSlot *slot)
 
        /*
         * Initialize the heap tuple pointer to access attributes of the minimal
-        * tuple contained in the slot as if its a heap tuple.
+        * tuple contained in the slot as if it's a heap tuple.
         */
        mslot->tuple = &mslot->minhdr;
 }
@@ -1403,7 +1403,7 @@ MakeTupleTableSlot(TupleDesc tupleDesc,
                 * Precalculate the maximum guaranteed attribute that has to exist in
                 * every tuple which gets deformed into this slot.  When the
                 * TTS_FLAG_OBEYS_NOT_NULL_CONSTRAINTS flag is enabled, we simply take
-                * the precalculated value from the tupleDesc, otherwise the
+                * the pre-calculated value from the tupleDesc, otherwise the
                 * optimization is disabled, and we set the value to 0.
                 */
                if ((flags & TTS_FLAG_OBEYS_NOT_NULL_CONSTRAINTS) != 0)
index e8a044f56c61e3440f74f112090acc60431d9fab..4cb057ca4f94c8b2270d074a7ce176ee55c7dba5 100644 (file)
@@ -1432,7 +1432,7 @@ ExecForPortionOfLeftovers(ModifyTableContext *context,
                /*
                 * If we don't have a ForPortionOfState yet, we must be a partition
                 * child being hit for the first time. Make a copy from the root, with
-                * our own tupleTableSlot. We do this lazily so that we don't pay the
+                * our own TupleTableSlot. We do this lazily so that we don't pay the
                 * price of unused partitions.
                 */
                ForPortionOfState *leafState = makeNode(ForPortionOfState);
index c74f11217effcbfc46df1eaf8621743c600e7d03..3bc6aa0548fff21e39be5c3b16a27b71cf0f48ca 100644 (file)
@@ -228,7 +228,7 @@ estimateHyperLogLog(hyperLogLogState *cState)
  * starting from the first, reading from most significant to least significant
  * bits.
  *
- * Example (when considering fist 10 bits of x):
+ * Example (when considering first 10 bits of x):
  *
  * rho(x = 0b1000000000)   returns 1
  * rho(x = 0b0010000000)   returns 3
index a3e222f3a3d9062f01725d56d874a645acc7af33..f64b2787f668b4557b0eaebd915d8580c4724af2 100644 (file)
@@ -200,7 +200,7 @@ be_tls_init(bool isServerStart)
         *
         * The reason for not doing everything in this if-else conditional is that
         * we want to use the same processing of postgresql.conf for when ssl_sni
-        * is off as well as when it's on but the hostsfile is missing etc.  Thus
+        * is off as well as when it's on but the hosts file is missing etc.  Thus
         * we set res to the state and continue with a new conditional instead of
         * duplicating logic and risk it diverging over time.
         */
@@ -208,7 +208,7 @@ be_tls_init(bool isServerStart)
        {
                /*
                 * The GUC check hook should have already blocked this but to be on
-                * the safe side we doublecheck here.
+                * the safe side we double-check here.
                 */
 #ifndef HAVE_SSL_CTX_SET_CLIENT_HELLO_CB
                ereport(isServerStart ? FATAL : LOG,
index 430e06dcaaac25090823c4445af0f10a1aedf825..f76a5373c4bea2d20bb7abd9f0da1fa543ba0a40 100644 (file)
@@ -1187,7 +1187,7 @@ typedef struct
        Oid                     inputcollid;    /* OID of the OpClause input collation */
        int                     argindex;               /* index of the clause in the list of
                                                                 * arguments */
-       int                     groupindex;             /* value of argindex for the fist clause in
+       int                     groupindex;             /* value of argindex for the first clause in
                                                                 * the group of similar clauses */
 } OrArgIndexMatch;
 
index 84deed9aaa629f1581af91d19adb0beb52092916..cb4e5019c2f04bf6bb96c4ef1b62597db4bf50f5 100644 (file)
@@ -80,7 +80,7 @@ static OnConflictExpr *transformOnConflictClause(ParseState *pstate,
                                                                                                 OnConflictClause *onConflictClause);
 static ForPortionOfExpr *transformForPortionOfClause(ParseState *pstate,
                                                                                                         int rtindex,
-                                                                                                        const ForPortionOfClause *forPortionOfClause,
+                                                                                                        const ForPortionOfClause *forPortionOf,
                                                                                                         bool isUpdate);
 static int     count_rowexpr_columns(ParseState *pstate, Node *expr);
 static Query *transformSelectStmt(ParseState *pstate, SelectStmt *stmt,
index 30ddce5aa9f42855241e41689d444efef3539c22..f889c8df4e314df94a2d114ca47d0a7cf5a57c25 100644 (file)
@@ -157,7 +157,7 @@ transformGraphTablePropertyRef(ParseState *pstate, ColumnRef *cref)
  * A label expression is parsed as either a ColumnRef with a single field or a
  * label expression like label disjunction. The single field in the ColumnRef is
  * treated as a label name and transformed to a GraphLabelRef node. The label
- * expression is recursively transformed into an expression tree containg
+ * expression is recursively transformed into an expression tree containing
  * GraphLabelRef nodes corresponding to the names of the labels appearing in the
  * expression. If any label name cannot be resolved to a label in the property
  * graph, an error is raised.
index 680db664be4ca256451d1253b3a5005d97e5708a..a5a8db2ff88793ffcd5d8ce06a3864aadefb1c46 100644 (file)
@@ -3047,7 +3047,7 @@ table_recheck_autovac(Oid relid, HTAB *table_toast_map,
  *
  * One exception to the previous paragraph is for tables nearing wraparound,
  * i.e., those that have surpassed the effective failsafe ages.  In that case,
- * the relfrozen/relminmxid-based score is scaled aggressively so that the
+ * the relfrozenxid/relminmxid-based score is scaled aggressively so that the
  * table has a decent chance of sorting to the front of the list.
  *
  * To adjust how strongly each component contributes to the score, the
index 213a909aa921e847835c9532dacea4c00b5b2a34..b3e991706699dec1b99af2cd66a3a039e2594b05 100644 (file)
  * state will also be set to "off".
  *
  * Backends transition Bd -> Bi via a procsignalbarrier which is emitted by the
- * DataChecksumsLauncher.  When all backends have acknowledged the barrier then
- * Bd will be empty and the next phase can begin: calculating and writing data
- * checksums with DataChecksumsWorkers.  When the DataChecksumsWorker processes
- * have finished writing checksums on all pages, data checksums are enabled
- * cluster-wide via another procsignalbarrier. There are four sets of backends
- * where Bd shall be an empty set:
+ * DataChecksumsWorkerLauncherMain.  When all backends have acknowledged the
+ * barrier then Bd will be empty and the next phase can begin: calculating and
+ * writing data checksums with DataChecksumsWorkers.  When the
+ * DataChecksumsWorker processes have finished writing checksums on all pages,
+ * data checksums are enabled cluster-wide via another procsignalbarrier.
+ * There are four sets of backends where Bd shall be an empty set:
  *
  * Bg: Backend updating the global state and emitting the procsignalbarrier
  * Bd: Backends in "off" state
@@ -634,7 +634,7 @@ ProcessSingleRelationFork(Relation reln, ForkNumber forkNum, BufferAccessStrateg
 
        relns = get_namespace_name(RelationGetNamespace(reln));
 
-       /* Report the current relation to pgstat_activity */
+       /* Report the current relation to pg_stat_activity */
        snprintf(activity, sizeof(activity) - 1, "processing: %s.%s (%s, %u blocks)",
                         (relns ? relns : ""), RelationGetRelationName(reln), forkNames[forkNum], numblocks);
        pgstat_report_activity(STATE_RUNNING, activity);
@@ -659,7 +659,7 @@ ProcessSingleRelationFork(Relation reln, ForkNumber forkNum, BufferAccessStrateg
                 * re-write the page to WAL even if the checksum hasn't changed,
                 * because if there is a replica it might have a slightly different
                 * version of the page with an invalid checksum, caused by unlogged
-                * changes (e.g. hintbits) on the primary happening while checksums
+                * changes (e.g. hint bits) on the primary happening while checksums
                 * were off. This can happen if there was a valid checksum on the page
                 * at one point in the past, so only when checksums are first on, then
                 * off, and then turned on again.  TODO: investigate if this could be
@@ -1262,7 +1262,7 @@ ProcessAllDatabases(void)
 }
 
 /*
- * DataChecksumShmemRequest
+ * DataChecksumsShmemRequest
  *             Request datachecksumsworker-related shared memory
  */
 static void
index 372d77c475ed5065621b848a6705c4c6db2eb682..c9dfb094c2b1699ee39298a125f92cb413b89157 100644 (file)
@@ -1371,7 +1371,7 @@ replorigin_session_get_progress(bool flush)
 /*
  * Clear the per-transaction replication origin state.
  *
- * replorigin_session_origin is also cleared if clear_origin is set.
+ * replorigin_xact_state.origin is also cleared if clear_origin is set.
  */
 void
 replorigin_xact_clear(bool clear_origin)
index b38170f0fbe99861e55e7f914a3c13770ab79664..9f2a16b17fa99faccec9ec5619087c18a669f3ee 100644 (file)
@@ -4980,7 +4980,7 @@ adjust_xid_advance_interval(RetainDeadTuplesData *rdt_data, bool new_xid_found)
                /*
                 * Retention has been stopped, so double the interval-capped at a
                 * maximum of 3 minutes. The wal_receiver_status_interval is
-                * intentionally not used as a upper bound, since the likelihood of
+                * intentionally not used as an upper bound, since the likelihood of
                 * retention resuming is lower than that of general activity resuming.
                 */
                rdt_data->xid_advance_interval = Min(rdt_data->xid_advance_interval * 2,
index 2c3199d32302a3db40bef76adfbd5c958dfd26b2..7b64818e369601c11b6e23e00af39f42b60a3161 100644 (file)
@@ -91,10 +91,10 @@ struct path_element
 
 static Node *replace_property_refs(Oid propgraphid, Node *node, const List *mappings);
 static List *build_edge_vertex_link_quals(HeapTuple edgetup, int edgerti, int refrti, Oid refid, AttrNumber catalog_key_attnum, AttrNumber catalog_ref_attnum, AttrNumber catalog_eqop_attnum);
-static List *generate_queries_for_path_pattern(RangeTblEntry *rte, List *element_patterns);
+static List *generate_queries_for_path_pattern(RangeTblEntry *rte, List *path_pattern);
 static Query *generate_query_for_graph_path(RangeTblEntry *rte, List *path);
 static Node *generate_setop_from_pathqueries(List *pathqueries, List **rtable, List **targetlist);
-static List *generate_queries_for_path_pattern_recurse(RangeTblEntry *rte, List *pathqueries, List *cur_path, List *path_pattern_lists, int elempos);
+static List *generate_queries_for_path_pattern_recurse(RangeTblEntry *rte, List *pathqueries, List *cur_path, List *path_elem_lists, int elempos);
 static Query *generate_query_for_empty_path_pattern(RangeTblEntry *rte);
 static Query *generate_union_from_pathqueries(List **pathqueries);
 static List *get_path_elements_for_path_factor(Oid propgraphid, struct path_factor *pf);
index 9279904b46537b6fd97ea91c29b5688f32666be2..26a78913cc89977ccc8a5ec3231d796e0aa3d493 100644 (file)
@@ -571,7 +571,7 @@ extended_statistics_update(FunctionCallInfo fcinfo)
 
        /*
         * Either of these statistic types requires that we supply a semi-filled
-        * VacAttrStatP array.
+        * VacAttrStatsP array.
         *
         * It is not possible to use the existing lookup_var_attr_stats() and
         * examine_attribute() because these functions will skip attributes where
@@ -586,8 +586,8 @@ extended_statistics_update(FunctionCallInfo fcinfo)
 
                /*
                 * The leading stxkeys are attribute numbers up through numattnums.
-                * These keys must be in ascending AttNumber order, but we do not rely
-                * on that.
+                * These keys must be in ascending AttrNumber order, but we do not
+                * rely on that.
                 */
                for (int i = 0; i < numattnums; i++)
                {
@@ -724,7 +724,7 @@ extended_statistics_update(FunctionCallInfo fcinfo)
                /*
                 * Generate the expressions array.
                 *
-                * The attytypids, attytypmods, and atttypcolls arrays have all the
+                * The atttypids, atttypmods, and atttypcolls arrays have all the
                 * regular attributes listed first, so we can pass those arrays with a
                 * start point after the last regular attribute.  There are numexprs
                 * elements remaining.
@@ -1091,7 +1091,7 @@ array_in_safe(FmgrInfo *array_in, const char *s, Oid typid, int32 typmod,
  * still return a legit tuple datum.
  *
  * Set pg_statistic_ok to true if all of the values found in the container
- * were imported without issue.  pg_statistic_ok is swicthed to "true" once
+ * were imported without issue.  pg_statistic_ok is switched to "true" once
  * the full pg_statistic tuple has been built and validated.
  */
 static Datum
@@ -1307,10 +1307,6 @@ import_pg_statistic(Relation pgsd, JsonbContainer *cont,
         * if they aren't then we need to reject that stakind completely.
         * Currently we go a step further and reject the expression array
         * completely.
-        *
-        * Once it is established that the pairs are in NULL/NOT-NULL alignment,
-        * we can test either expr_nulls[] value to see if the stakind has
-        * value(s) that we can set or not.
         */
 
        if (found[MOST_COMMON_VALS_ELEM])
index 061a93d90d475989c9c221d50c9a9d6976012c93..63e34d66690d45b74f646614dc90166c3c1ada30 100644 (file)
@@ -814,7 +814,7 @@ IoWorkerMain(const void *startup_data, size_t startup_data_len)
                         * it.  Crossing the current worker count is a useful signal
                         * because it's clearly too deep to avoid queuing latency already,
                         * but still leaves a small window of opportunity to improve the
-                        * situation before the queue oveflows.
+                        * situation before the queue overflows.
                         *
                         * 2. The worker pool is keeping up, no latency is being
                         * introduced and an extra worker would be a waste of resources.
@@ -830,10 +830,10 @@ IoWorkerMain(const void *startup_data, size_t startup_data_len)
                         *
                         * On its own, this is an extremely crude signal.  When combined
                         * with the wakeup propagation test that precedes it (but on its
-                        * own tends to overshoot) and io_worker_launch_delay, the result
-                        * is that we gradually test each pool size until we find one that
-                        * doesn't trigger further expansion, and then hold it for at
-                        * least io_worker_idle_timeout.
+                        * own tends to overshoot) and io_worker_launch_interval, the
+                        * result is that we gradually test each pool size until we find
+                        * one that doesn't trigger further expansion, and then hold it
+                        * for at least io_worker_idle_timeout.
                         *
                         * XXX Perhaps ideas from queueing theory or control theory could
                         * do a better job of this.
index 3cc0b0bdd929f9b9238cf0eaf82e8d3851407d06..32f501bbd218b25b1d9e8d549718a4a097852aa5 100644 (file)
@@ -5844,7 +5844,7 @@ MarkBufferDirtyHint(Buffer buffer, bool buffer_std)
  * Used to clean up after errors.
  *
  * Currently, we can expect that resource owner cleanup, via
- * ResOwnerReleaseBufferPin(), took care of releasing buffer content locks per
+ * ResOwnerReleaseBuffer(), took care of releasing buffer content locks per
  * se; the only thing we need to deal with here is clearing any PIN_COUNT
  * request that was in progress.
  */
@@ -5993,7 +5993,7 @@ BufferLockAcquire(Buffer buffer, BufferDesc *buf_hdr, BufferLockMode mode)
 
                pgstat_report_wait_end();
 
-               /* Retrying, allow BufferLockRelease to release waiters again. */
+               /* Retrying, allow BufferLockReleaseSub to release waiters again. */
                pg_atomic_fetch_and_u64(&buf_hdr->state, ~BM_LOCK_WAKE_IN_PROGRESS);
        }
 
index bf6b81e621b0d2fe2746cbd5025967ec07c0008c..e149a738c8d28a98f4a6972c3840272a6ea056b2 100644 (file)
@@ -169,7 +169,7 @@ RegisterBuiltinShmemCallbacks(void)
 {
        /*
         * Call RegisterShmemCallbacks(...) on each subsystem listed in
-        * subsystemslist.h
+        * subsystemlist.h
         */
 #define PG_SHMEM_SUBSYSTEM(subsystem_callbacks) \
        RegisterShmemCallbacks(&(subsystem_callbacks));
index c221fe9688981454765785e494e153d8b29e79e1..8d246ed5a4e3206a67f7c3d2b7d047c5ee8ef8ee 100644 (file)
@@ -453,8 +453,7 @@ LockManagerShmemRequest(void *arg)
        int64           max_table_size;
 
        /*
-        * Compute sizes for lock hashtables.  Note that these calculations must
-        * agree with LockManagerShmemSize!
+        * Compute sizes for lock hashtables.
         */
        max_table_size = NLOCKENTS();
 
index 899a4ef06e479099da1a5490338670ef3766dc93..0ae85b7d5b4be19bf10033fa89e4b70958a67727 100644 (file)
@@ -1156,8 +1156,7 @@ PredicateLockShmemRequest(void *arg)
                );
 
        /*
-        * Compute size for serializable transaction hashtable. Note these
-        * calculations must agree with PredicateLockShmemSize!
+        * Compute size for serializable transaction hashtable.
         *
         * Assume an average of 10 predicate locking transactions per backend.
         * This allows aggressive cleanup while detail is present before data must
index c7e0828764e7b1a6b595af9b4355e6f4222da581..f4bf9c3b532854d0f0bb6bb54e614fb33559faa9 100644 (file)
@@ -551,7 +551,7 @@ network_abbrev_abort(int memtupcount, SortSupport ssup)
  * all their subnet bits *must* be zero (1.2.3.0/24).
  *
  * IPv4 and IPv6 are identical in this makeup, with the difference being that
- * IPv4 addresses have a maximum of 32 bits compared to IPv6's 64 bits, so in
+ * IPv4 addresses have a maximum of 32 bits compared to IPv6's 128 bits, so in
  * IPv6 each part may be larger.
  *
  * inet/cidr types compare using these sorting rules. If inequality is detected
index 090e8cc28c118fb67127b86e2d254d01a66e7ca9..78587d223cb6285c9a02743bf5ecec5530011291 100644 (file)
@@ -13352,7 +13352,7 @@ get_for_portion_of(ForPortionOfExpr *forPortionOf, deparse_context *context)
 
                /*
                 * Try to write it as FROM ... TO ... if we received it that way,
-                * otherwise (targetExpr).
+                * otherwise (targetRange).
                 */
                if (forPortionOf->targetFrom && forPortionOf->targetTo)
                {
index ad960336e8dd7b9688e37c62eee148aab6fd83dc..1b8a73f589a126a037151f54860d87bedc42a3e2 100644 (file)
@@ -2353,7 +2353,7 @@ DebugFileOpen(void)
  * GUC check_hook for log_min_messages
  *
  * This value is parsed as a comma-separated list of zero or more TYPE:LEVEL
- * elements.  For each element, TYPE corresponds to a bk_category value (see
+ * elements.  For each element, TYPE corresponds to a bkcategory value (see
  * postmaster/proctypelist.h); LEVEL is one of server_message_level_options.
  *
  * In addition, there must be a single LEVEL element (with no TYPE part)
index c5e0124c04deb1e048d4e50e3ab14b4a10de7695..e1ce45215582cc51d592e6384541bde6c04dbf3a 100644 (file)
@@ -102,7 +102,8 @@ is($vm_limits, '1',
        'WAL summary has correct VM fork truncation limit');
 
 # Combine full and incremental backups.  Before the fix, this failed because
-# the INCREMENTAL file header contained an incorrect truncation_block value.
+# the INCREMENTAL file header contained an incorrect truncation_block_length
+# value.
 my $restored = PostgreSQL::Test::Cluster->new('node2');
 $restored->init_from_backup($primary, 'incr', combine_with_prior => ['full']);
 $restored->start();
index 1ca03d6b2787e142b72bfcae1b081df4955480bc..d56dcc701ce8db9e3cbca34757488493983aa025 100644 (file)
@@ -18861,7 +18861,7 @@ dumpStatisticsExtStats(Archive *fout, const StatsExtInfo *statsextinfo)
                {
                        /*
                         * There is no ordering column in pg_stats_ext_exprs.  However, we
-                        * can rely on the unnesting of pg_statistic.ext_data.stxdexpr to
+                        * can rely on the unnesting of pg_statistic_ext_data.stxdexpr to
                         * maintain the desired order of expression elements.
                         */
                        appendPQExpBufferStr(pq,
index f016b336308921e7d6594441dcd334aa5f5cf328..95f4ac110b93ae7aea665686497018efa442053c 100644 (file)
@@ -704,7 +704,7 @@ restore_one_database(const char *inputFileSpec, RestoreOptions *opts,
         * We don't have a connection yet but that doesn't matter. The connection
         * is initialized to NULL and if we terminate through exit_nicely() while
         * it's still NULL, the cleanup function will just be a no-op. If we are
-        * restoring multiple databases, then only update AX handle for cleanup as
+        * restoring multiple databases, then only update AH handle for cleanup as
         * the previous entry was already in the array and we had closed previous
         * connection, so we can use the same array slot.
         */
index 44f2b446e5d04f1387da4cb26bab34bb11503cd4..f8d25afed9de401d97318432b14236f51128e4e9 100644 (file)
@@ -307,7 +307,7 @@ main(int argc, char *argv[])
                                break;
 
                        case 'l':
-                               if (strspn(optarg, "01234567890ABCDEFabcdef") != XLOG_FNAME_LEN)
+                               if (strspn(optarg, "0123456789ABCDEFabcdef") != XLOG_FNAME_LEN)
                                {
                                        pg_log_error("invalid argument for option %s", "-l");
                                        pg_log_error_hint("Try \"%s --help\" for more information.", progname);
index 06134cf5d2e2ba698538453a8d720f78cd184b24..5a7afe62eab2de3d084fafb93bea6d13873456c9 100644 (file)
@@ -2601,7 +2601,7 @@ check_old_cluster_global_names(ClusterInfo *cluster)
        conn_template1 = connectToServer(cluster, "template1");
 
        /*
-        * Get database, user/role and tablespacenames from cluster.  Can't use
+        * Get database, user/role and tablespace names from cluster.  Can't use
         * pg_authid because only superusers can view it.
         */
        res = executeQueryOrDie(conn_template1,
index e18687226aebc437fafeb1efbadd1d73a06b27f6..cffcd4b0ebabe13bd06b8b7bdedd1771db8e6cf5 100644 (file)
@@ -361,7 +361,7 @@ get_control_data(ClusterInfo *cluster)
                        p = strchr(p, ':');
                        if (p == NULL || strlen(p) <= 1)
                                pg_fatal("%d: controldata retrieval problem", __LINE__);
-                       p = strpbrk(p, "01234567890ABCDEF");
+                       p = strpbrk(p, "0123456789ABCDEF");
                        if (p == NULL || strlen(p) <= 1)
                                pg_fatal("%d: controldata retrieval problem", __LINE__);
 
index a44ee73accfe6c05515655350d7ed29f5250dd63..4c77e3e1dc81ce509af2b920f99ae2643ad20ce5 100644 (file)
@@ -26,9 +26,9 @@
  * this pair specifically.  Byte pair range constraints, in encoding
  * originator documentation, always excluded this pair.  No core conversion
  * could translate it.  However, longstanding verifychar implementations
- * accepted any non-NUL byte.  big5_to_euc_tw and big5_to_mic even translate
- * pairs not valid per encoding originator documentation.  To avoid tightening
- * core or non-core conversions in a security patch, we sought this one pair.
+ * accepted any non-NUL byte.  big5_to_euc_tw even translates pairs not
+ * valid per encoding originator documentation.  To avoid tightening core
+ * or non-core conversions in a security patch, we sought this one pair.
  *
  * PQescapeString() historically used spaces for BYTE1; many other values
  * could suffice for BYTE1.
index c13f05d39db251eb04bf20fed1e774370042735e..f2c36696bcad0f64ceaf0c3e5528da62cd733826 100644 (file)
@@ -1570,8 +1570,6 @@ table_tuple_delete(Relation rel, ItemPointer tid, CommandId cid,
  *     crosscheck - if not InvalidSnapshot, also check old tuple against this
  *     options - These allow the caller to specify options that may change the
  *     behavior of the AM. The AM will ignore options that it does not support.
- *             TABLE_UPDATE_WAIT -- set if should wait for any conflicting update to
- *             commit/abort
  *             TABLE_UPDATE_NO_LOGICAL -- force-disables the emitting of logical
  *             decoding information for the tuple.
  *
index 05625539604c3f9dfac8b19ad19db07413c395d0..7acb3b43ab83dc48d1c06306461e98357621c2f6 100644 (file)
@@ -17,7 +17,7 @@
 
 #include "storage/procsignal.h"
 
-/* Possible operations the Datachecksumsworker can perform */
+/* Possible operations the DataChecksumsWorker can perform */
 typedef enum DataChecksumsWorkerOperation
 {
        ENABLE_DATACHECKSUMS,
index 8c1fda5caf02f3d7310062edff5e12a44c322f95..b0638bd3254840efdbe835d8cf950819ba8d1718 100644 (file)
@@ -1612,7 +1612,7 @@ getParameterStatus(PGconn *conn)
 
 /*
  * parseInput subroutine to read a BackendKeyData message.
- * Entry: 'v' message type and length have already been consumed.
+ * Entry: 'K' message type and length have already been consumed.
  * Exit: returns 0 if successfully consumed message.
  *              returns EOF if not enough data.
  */
index 7ad11417ca6518611f4fe52a42fc480cdfe27344..11e15c9d73471bf0113f9c63ef797ea2a06ff1f5 100644 (file)
@@ -110,7 +110,7 @@ $node_primary->wait_for_catchup($node_standby, 'replay');
 # Ensure that the primary and standby has switched to off
 wait_for_checksum_state($node_primary, 'off');
 wait_for_checksum_state($node_standby, 'off');
-# Doublecheck reading data without errors
+# Double-check reading data without errors
 $result =
   $node_primary->safe_psql('postgres', "SELECT count(a) FROM t WHERE a > 1");
 is($result, "19998", 'ensure we can safely read all data without checksums');
index a37a24dbbadb1b0ce8419371bf0a74544ccee96f..7240b93bdd135a9e61be0be5811cd2031e453166 100644 (file)
@@ -25,7 +25,7 @@ if ($ENV{enable_injection_points} ne 'yes')
 # Test cluster setup
 #
 
-# Initiate testcluster
+# Initiate test cluster
 my $node = PostgreSQL::Test::Cluster->new('injection_node');
 $node->init(no_data_checksums => 1);
 $node->start;
index c2eabc2821c5ca636d81d904ffbf66b66293de3f..621cf788dadbd818a06b413c89dd53f2671911d7 100644 (file)
@@ -22,8 +22,6 @@
 PG_MODULE_MAGIC;
 
 extern PGDLLEXPORT void dc_delay_barrier(const char *name, const void *private_data, void *arg);
-extern PGDLLEXPORT void dc_modify_db_result(const char *name, const void *private_data, void *arg);
-extern PGDLLEXPORT void dc_fake_temptable(const char *name, const void *private_data, void *arg);
 
 /*
  * Test for delaying emission of procsignalbarriers.
index 0660b0a38db75b2d8d07c0e1b69964b49752dea3..f8302503b4bfa92bb24bff5ea28c87edee918225 100644 (file)
@@ -40,7 +40,7 @@ extension_control_path = '\$system$sep@{[ $windows_os ? ($ext_dir =~ s/\\/\\\\/g
 # Start node
 $node->start;
 
-# Create an user to test permissions to read extension locations.
+# Create a user to test permissions to read extension locations.
 my $user = "user01";
 $node->safe_psql('postgres', "CREATE USER $user");
 
index 4f501169f42dba6c14e0761e2739a0a5595a5d98..82567ca6bfcb08363f78cfc02ecfd424c1ba8b98 100644 (file)
@@ -58,7 +58,7 @@ my $initfile = $node->safe_psql('postgres',
 $node->safe_psql('postgres',
        qq{SELECT pg_logical_emit_message(true, 'test 026', repeat('xyzxz', 123456))}
 );
-#$node->safe_psql('postgres', qq{create table foo ()});
+
 my $endfile = $node->safe_psql('postgres',
        'SELECT pg_walfile_name(pg_current_wal_insert_lsn())');
 ok($initfile ne $endfile, "$initfile differs from $endfile");
index ffbc47089b1290a2c0703e8c57fe167cc502ce3a..bd142ed481cbb84c11b7bfb7b5a10a3744b50b91 100644 (file)
@@ -429,7 +429,7 @@ SELECT attname, atthasmissing, attmissingval FROM pg_attribute
 
 DROP TABLE T;
 DROP FUNCTION foolme(timestamptz);
--- Simple querie
+-- Simple queries
 CREATE TABLE T (pk INT NOT NULL PRIMARY KEY);
 SELECT set('t');
  set 
index 8ff29cf26973768d08ebaa405254b4806c69b8bc..8b31d317d73d8c694a6df862d418518a4eee3652 100644 (file)
@@ -375,7 +375,7 @@ SELECT attname, atthasmissing, attmissingval FROM pg_attribute
 DROP TABLE T;
 DROP FUNCTION foolme(timestamptz);
 
--- Simple querie
+-- Simple queries
 CREATE TABLE T (pk INT NOT NULL PRIMARY KEY);
 
 SELECT set('t');
index 421592db92816b2e7f203080b50b644ce0fdfced..736f350f145c47f891756046c40ebb2239c0e993 100644 (file)
@@ -229,7 +229,7 @@ main(int argc, char **argv)
     }
 
     if (ps.com_ind <= 1)
-       ps.com_ind = 2;         /* dont put normal comments before column 2 */
+       ps.com_ind = 2;         /* don't put normal comments before column 2 */
     if (block_comment_max_col <= 0)
        block_comment_max_col = max_col;
     if (ps.local_decl_indent < 0)      /* if not specified by user, set this */
@@ -483,7 +483,7 @@ check_type:
                if (verbose)
                    diag2(0, "Line broken");
                dump_line();
-               ps.want_blank = false;  /* dont insert blank at line start */
+               ps.want_blank = false;  /* don't insert blank at line start */
                force_nl = false;
            }
            ps.in_stmt = true;  /* turn on flag which causes an extra level of
@@ -604,7 +604,7 @@ check_type:
                force_nl = true;/* must force newline after if */
                ps.last_u_d = true;     /* inform lexi that a following
                                         * operator is unary */
-               ps.in_stmt = false;     /* dont use stmt continuation
+               ps.in_stmt = false;     /* don't use stmt continuation
                                         * indentation */
 
                parse(hd_type); /* let parser worry about if, or whatever */
@@ -750,7 +750,7 @@ check_type:
                if (sp_sw) {    /* this is a check for an if, while, etc. with
                                 * unbalanced parens */
                    sp_sw = false;
-                   parse(hd_type);     /* dont lose the if, or whatever */
+                   parse(hd_type);     /* don't lose the if, or whatever */
                }
            }
            *e_code++ = ';';
@@ -765,7 +765,7 @@ check_type:
            break;
 
        case lbrace:            /* got a '{' */
-           ps.in_stmt = false; /* dont indent the {} */
+           ps.in_stmt = false; /* don't indent the {} */
            if (!ps.block_init)
                force_nl = true;/* force other stuff on same line as '{' onto
                                 * new line */
@@ -803,7 +803,7 @@ check_type:
                }
            }
            if (s_code == e_code)
-               ps.ind_stmt = false;    /* dont put extra indentation on line
+               ps.ind_stmt = false;    /* don't put extra indentation on line
                                         * with '{' */
            if (ps.in_decl && ps.in_or_st) {    /* this is either a structure
                                                 * declaration or an init */
@@ -1014,7 +1014,7 @@ check_type:
        case period:            /* treat a period kind of like a binary
                                 * operation */
            *e_code++ = '.';    /* move the period into line */
-           ps.want_blank = false;      /* dont put a blank after a period */
+           ps.want_blank = false;      /* don't put a blank after a period */
            break;
 
        case comma:
index 9d64ca1ee56daae026911a923c472f58ff34d1fc..62d600bbb1167669644d3e88e4832ae8c9675bec 100644 (file)
@@ -97,7 +97,7 @@ dump_line(void)
            putc('\n', output);
        n_real_blanklines = 0;
        if (ps.ind_level == 0)
-           ps.ind_stmt = 0;    /* this is a class A kludge. dont do
+           ps.ind_stmt = 0;    /* this is a class A kludge. don't do
                                 * additional statement indentation if we are
                                 * at bracket level 0 */
 
index a9572b39ffe68ba8fb9d6a790eafe6c114a4cbd5..7dd0f59e3f7d6d386642f1b751ddef7175bf0dec 100644 (file)
@@ -117,7 +117,7 @@ pr_comment(void)
        if ( /* ps.bl_line && */ (s_lab == e_lab) && (s_code == e_code)) {
            /* klg: check only if this line is blank */
            /*
-            * If this (*and previous lines are*) blank, dont put comment way
+            * If this (*and previous lines are*) blank, don't put comment way
             * out at left
             */
            ps.com_col = (ps.ind_level - ps.unindent_displace) * ps.ind_size + 1;
@@ -234,7 +234,7 @@ pr_comment(void)
            last_bl = NULL;
            CHECK_SIZE_COM(4);
            if (ps.box_com || ps.last_nl) {     /* if this is a boxed comment,
-                                                * we dont ignore the newline */
+                                                * we don't ignore the newline */
                if (s_com == e_com)
                    *e_com++ = ' ';
                if (!ps.box_com && e_com - s_com > 3) {