Fix typos in comments, code and documentation
authorMichael Paquier <[email protected]>
Tue, 3 Jan 2023 07:26:14 +0000 (16:26 +0900)
committerMichael Paquier <[email protected]>
Tue, 3 Jan 2023 07:26:14 +0000 (16:26 +0900)
While on it, newlines are removed from the end of two elog() strings.
The others are simple grammar mistakes.  One comment in pg_upgrade
referred incorrectly to sequences since a7e5457.

Author: Justin Pryzby
Discussion: https://postgr.es/m/20221230231257[email protected]
Backpatch-through: 11

23 files changed:
.cirrus.yml
doc/src/sgml/auto-explain.sgml
doc/src/sgml/parallel.sgml
doc/src/sgml/ref/alter_table.sgml
doc/src/sgml/ref/create_subscription.sgml
doc/src/sgml/ref/grant.sgml
doc/src/sgml/sources.sgml
doc/src/sgml/syntax.sgml
meson.build
src/backend/access/common/bufmask.c
src/backend/access/spgist/spgutils.c
src/backend/jit/llvm/llvmjit.c
src/backend/optimizer/util/tlist.c
src/backend/utils/adt/ruleutils.c
src/backend/utils/misc/guc_internal.h
src/bin/pg_dump/pg_dumpall.c
src/bin/pg_upgrade/info.c
src/include/lib/ilist.h
src/test/regress/expected/copy.out
src/test/regress/expected/expressions.out
src/test/regress/sql/copy.sql
src/test/regress/sql/expressions.sql
src/test/ssl/t/SSL/Server.pm

index 354102613d803ac017dd3e39bc61439d9e9e1eba..69837bcd5ad7e6b0789120e781e3fd7bb51c13af 100644 (file)
@@ -11,7 +11,7 @@ env:
   CONTAINER_REPO: us-docker.pkg.dev/${GCP_PROJECT}/ci
 
   # The lower depth accelerates git clone. Use a bit of depth so that
-  # concurrent tasks and retrying older jobs has a chance of working.
+  # concurrent tasks and retrying older jobs have a chance of working.
   CIRRUS_CLONE_DEPTH: 500
   # Useful to be able to analyse what in a script takes long
   CIRRUS_LOG_TIMESTAMP: true
@@ -47,7 +47,7 @@ on_failure_meson: &on_failure_meson
 
   # In theory it'd be nice to upload the junit files meson generates, so that
   # cirrus will nicely annotate the commit. Unfortunately the files don't
-  # contain identifieable file + line numbers right now, so the annotations
+  # contain identifiable file + line numbers right now, so the annotations
   # don't end up useful. We could probably improve on that with a some custom
   # conversion script, but ...
   meson_log_artifacts:
@@ -160,7 +160,7 @@ task:
 
   ccache_cache:
     folder: $CCACHE_DIR
-  # Workaround around performance issues due to 32KB block size
+  # Work around performance issues due to 32KB block size
   repartition_script: src/tools/ci/gcp_freebsd_repartition.sh
   create_user_script: |
     pw useradd postgres
@@ -251,7 +251,7 @@ task:
     DEBUGINFOD_URLS: "https://debuginfod.debian.net"
 
     # Enable a reasonable set of sanitizers. Use the linux task for that, as
-    # it one of the fastest tasks (without sanitizers). Also several of the
+    # it's one of the fastest tasks (without sanitizers). Also several of the
     # sanitizers work best on linux.
     #
     # The overhead of alignment sanitizer is low, undefined behaviour has
@@ -600,7 +600,7 @@ task:
     CCACHE_MAXSIZE: "500M"
     CCACHE_SLOPPINESS: pch_defines,time_macros
     CCACHE_DEPEND: 1
-    # for some reason mingw plpython cannot find it's installation without this
+    # for some reason mingw plpython cannot find its installation without this
     PYTHONHOME: C:/msys64/ucrt64
     # prevents MSYS bash from resetting error mode
     MSYS: winjitdebug
index 394fec94e88efcc70a1090c82d88602fe1b54859..0899e55e4ca9d28c03c9ee1b25adaa6c92f53096 100644 (file)
@@ -73,7 +73,7 @@ LOAD 'auto_explain';
     <listitem>
      <para>
       <varname>auto_explain.log_parameter_max_length</varname> controls the
-      logging of query parameter values. A value of<literal>-1</literal> (the
+      logging of query parameter values. A value of <literal>-1</literal> (the
       default) logs the parameter values in full. <literal>0</literal> disables
       logging of parameter values. A value greater than zero truncates each
       parameter value to that many bytes. Only superusers can change this
index e556786e2bd96b6105d12e6d1043cf2960eccd23..5acc9537d6f283046b4b757e0b07256a3e6781a4 100644 (file)
@@ -128,7 +128,7 @@ EXPLAIN SELECT * FROM pgbench_accounts WHERE filler LIKE '%x%';
 
   <para>
     In addition, the system must not be running in single-user mode.  Since
-    the entire database system is running in single process in this situation,
+    the entire database system is running as a single process in this situation,
     no background workers will be available.
   </para>
 
index 43d782fea90ca4ea5fa60766369d27b542d81398..892df57f702dca9cebfb31ec497d4394b50bfd7f 100644 (file)
@@ -1005,7 +1005,7 @@ WITH ( MODULUS <replaceable class="parameter">numeric_literal</replaceable>, REM
       constraint. This does not work, however, if any of the partition keys
       is an expression and the partition does not accept
       <literal>NULL</literal> values. If attaching a list partition that will
-      not accept <literal>NULL</literal> values, also add
+      not accept <literal>NULL</literal> values, also add a
       <literal>NOT NULL</literal> constraint to the partition key column,
       unless it's an expression.
      </para>
@@ -1377,7 +1377,7 @@ WITH ( MODULUS <replaceable class="parameter">numeric_literal</replaceable>, REM
     type or an unconstrained domain over the new type, a table rewrite is not
     needed. However, indexes must always be rebuilt unless the system can
     verify that the new index would be logically equivalent to the existing
-    one.  For example, if the collation for a column has been changed an index
+    one.  For example, if the collation for a column has been changed, an index
     rebuild is always required because the new sort order might be different.
     However, in the absence of a collation change, a column can be changed
     from <type>text</type> to <type>varchar</type> (or vice versa) without
index f9a1776380bfc6058cdf7a1331bcf3c1d77228fa..90bd70050dd75718f4c04d8bb62e85afe55326d8 100644 (file)
@@ -383,7 +383,7 @@ CREATE SUBSCRIPTION <replaceable class="parameter">subscription_name</replaceabl
    <literal>FOR TABLES IN SCHEMA</literal>, rows are always published
    regardless of the definition of the other expressions.
    If the subscriber is a <productname>PostgreSQL</productname> version before
-   15 then any row filtering is ignored during the initial data synchronization
+   15, then any row filtering is ignored during the initial data synchronization
    phase. For this case, the user might want to consider deleting any initially
    copied data that would be incompatible with subsequent filtering.
    Because initial data synchronization does not take into account the publication
index c8ca2b1d6417a030e70930d1de15ebe0fb273888..518bdb32d82ef2c0960e6dce29ec7c11f2ab7e81 100644 (file)
@@ -253,7 +253,7 @@ GRANT <replaceable class="parameter">role_name</replaceable> [, ...] TO <replace
    in a role to one or more other roles.  Membership in a role is significant
    because it potentially allows access to the privileges granted to a role
    to each of its members, and potentially also the ability to make changes
-   to the role itself. However, the actual permisions conferred depend on
+   to the role itself. However, the actual permissions conferred depend on
    the options associated with the grant.
   </para>
 
index 5186d75d6164838c78fa71192fbb0c6720ec7a30..e0a6f775b6e89269f658b00ef1cc25cea998ded0 100644 (file)
@@ -935,8 +935,8 @@ BETTER: unrecognized node type: 42
    <simplesect>
     <title>Function-Like Macros and Inline Functions</title>
     <para>
-     Both, macros with arguments and <literal>static inline</literal>
-     functions, may be used. The latter are preferable if there are
+     Both macros with arguments and <literal>static inline</literal>
+     functions may be used. The latter are preferable if there are
      multiple-evaluation hazards when written as a macro, as e.g., the
      case with
 <programlisting>
index 956182e7c6a8d4dea8525e2159c954b8ae3e939f..0ccddea310dab67463657395041ada28bc8efa04 100644 (file)
@@ -722,7 +722,7 @@ $function$
 
     <note>
      <para>
-      Nondecimal integer constants are currently only supported in the range
+      Non-decimal integer constants are currently only supported in the range
       of the <type>bigint</type> type (see <xref
       linkend="datatype-numeric-table"/>).
      </para>
index eb64cda7e004f7e857ff245d925dd02e2363f1db..8999170b4d2cb1cc8830144d6282a81237a23817 100644 (file)
@@ -21,7 +21,7 @@ project('postgresql',
     'buildtype=release',
     # For compatibility with the autoconf build, set a default prefix. This
     # works even on windows, where it's a drive-relative path (i.e. when on
-    # d:/sompath it'll install to d:/usr/local/pgsql)
+    # d:/somepath it'll install to d:/usr/local/pgsql)
     'prefix=/usr/local/pgsql',
   ]
 )
@@ -1078,7 +1078,7 @@ if not get_option('readline').disabled()
 
     if not at_least_one_header_found
       error('''readline header not found
-If you have @0@ already installed, see see meson-log/meson-log.txt for details on the
+If you have @0@ already installed, see meson-log/meson-log.txt for details on the
 failure. It is possible the compiler isn't looking in the proper directory.
 Use -Dreadline=false to disable readline support.'''.format(readline_dep))
     endif
index e3ca644bd6f06010b5495eaf9bafc6f86c2d75c5..5e392dab1e9d2d6daacac54fa04029e4a74375dd 100644 (file)
@@ -78,7 +78,7 @@ mask_unused_space(Page page)
    if (pd_lower > pd_upper || pd_special < pd_upper ||
        pd_lower < SizeOfPageHeaderData || pd_special > BLCKSZ)
    {
-       elog(ERROR, "invalid page pd_lower %u pd_upper %u pd_special %u\n",
+       elog(ERROR, "invalid page pd_lower %u pd_upper %u pd_special %u",
             pd_lower, pd_upper, pd_special);
    }
 
index 9fd0531589bf273b725b358d0225d7ba5db4c00a..3761f2c193ba4059c7e0005fc1e12bc954c8283d 100644 (file)
@@ -1301,7 +1301,7 @@ spgproperty(Oid index_oid, int attno,
    /*
     * Currently, SP-GiST distance-ordered scans require that there be a
     * distance operator in the opclass with the default types. So we assume
-    * that if such a operator exists, then there's a reason for it.
+    * that if such an operator exists, then there's a reason for it.
     */
 
    /* First we need to know the column's opclass. */
index 1c10c1a55a0484d52cff6b99148edc205bdf56c2..312612115ca2abf33727076e873fa5a39b5bb417 100644 (file)
@@ -816,7 +816,7 @@ llvm_session_initialize(void)
 
    if (LLVMGetTargetFromTriple(llvm_triple, &llvm_targetref, &error) != 0)
    {
-       elog(FATAL, "failed to query triple %s\n", error);
+       elog(FATAL, "failed to query triple %s", error);
    }
 
    /*
index 4d95ffbddb26298eda93beb7d4c8a2b942ae55f7..c672b338c0d033e1e8262f81c16fda88fe9e8fe9 100644 (file)
@@ -865,7 +865,7 @@ apply_pathtarget_labeling_to_tlist(List *tlist, PathTarget *target)
  *
  * The outputs of this function are two parallel lists, one a list of
  * PathTargets and the other an integer list of bool flags indicating
- * whether the corresponding PathTarget contains any evaluatable SRFs.
+ * whether the corresponding PathTarget contains any evaluable SRFs.
  * The lists are given in the order they'd need to be evaluated in, with
  * the "lowest" PathTarget first.  So the last list entry is always the
  * originally given PathTarget, and any entries before it indicate evaluation
index 82f87511a1f262e689a260413081765c28f0cae5..9ac42efdbc358adcb0008b7e09aae94484d509ca 100644 (file)
@@ -11391,7 +11391,7 @@ get_opclass_name(Oid opclass, Oid actual_datatype,
 
 /*
  * generate_opclass_name
- *     Compute the name to display for a opclass specified by OID
+ *     Compute the name to display for an opclass specified by OID
  *
  * The result includes all necessary quoting and schema-prefixing.
  */
index 8d93d347a727aec9758961083da5db57ea1fe5e2..608c8ae47846cffca83edf7879f027d7091fb8c8 100644 (file)
@@ -1,16 +1,16 @@
 /*--------------------------------------------------------------------
- * guc_internals.h
+ * guc_internal.h
  *
  * Declarations shared between backend/utils/misc/guc.c and
  * backend/utils/misc/guc-file.l
  *
  * Copyright (c) 2000-2023, PostgreSQL Global Development Group
  *
- * src/include/utils/guc_internals.h
+ * src/include/utils/guc_internal.h
  *--------------------------------------------------------------------
  */
-#ifndef GUC_INTERNALS_H
-#define GUC_INTERNALS_H
+#ifndef GUC_INTERNAL_H
+#define GUC_INTERNAL_H
 
 #include "utils/guc.h"
 
@@ -23,4 +23,4 @@ extern void record_config_file_error(const char *errmsg,
                                     ConfigVariable **head_p,
                                     ConfigVariable **tail_p);
 
-#endif                         /* GUC_INTERNALS_H */
+#endif                         /* GUC_INTERNAL_H */
index f26a53b286d3b9cc69d7744f5cc1720bc91d990d..fbd1c6fc85b70c23739b36b7daf67eccf3cfcab5 100644 (file)
@@ -1034,7 +1034,7 @@ dumpRoleMembership(PGconn *conn)
        ht = rolename_create(remaining, NULL);
 
        /*
-        * Make repeated passses over the grants for this role until all have
+        * Make repeated passes over the grants for this role until all have
         * been dumped.
         */
        while (remaining > 0)
index 47d0bfd110184f8c7c25f7a50dab4c6451911bfa..c1399c09b940d93d8d59c9984410833abaaa9c9d 100644 (file)
@@ -408,11 +408,10 @@ get_rel_infos(ClusterInfo *cluster, DbInfo *dbinfo)
    query[0] = '\0';            /* initialize query string to empty */
 
    /*
-    * Create a CTE that collects OIDs of regular user tables, including
-    * matviews and sequences, but excluding toast tables and indexes.  We
-    * assume that relations with OIDs >= FirstNormalObjectId belong to the
-    * user.  (That's probably redundant with the namespace-name exclusions,
-    * but let's be safe.)
+    * Create a CTE that collects OIDs of regular user tables and matviews,
+    * but excluding toast tables and indexes.  We assume that relations with
+    * OIDs >= FirstNormalObjectId belong to the user.  (That's probably
+    * redundant with the namespace-name exclusions, but let's be safe.)
     *
     * pg_largeobject contains user data that does not appear in pg_dump
     * output, so we have to copy that system table.  It's easiest to do that
index 9ff21bd41dc6e986a7ad5449d2354697e2161200..3a9209d1c107caff10c7ada55c9e13b470cd5812 100644 (file)
@@ -162,7 +162,7 @@ typedef struct dlist_head
 
 
 /*
- * Doubly linked list iterator type for dlist_head and and dclist_head types.
+ * Doubly linked list iterator type for dlist_head and dclist_head types.
  *
  * Used as state in dlist_foreach() and dlist_reverse_foreach() (and the
  * dclist variant thereof).
index 3fad1c52d1fe53c516b2e52d251737510503fda6..8a8bf43fdea0fb67ce3cb4827779361929552b9f 100644 (file)
@@ -182,7 +182,7 @@ create table header_copytest (
    b int,
    c text
 );
--- Make sure it works with with dropped columns
+-- Make sure it works with dropped columns
 alter table header_copytest drop column c;
 alter table header_copytest add column c text;
 copy header_copytest to stdout with (header match);
index df432d4c92713ac39c74d630f03163288d89f3f4..2d5342ab2818db0c76cfd1c4149c33c81942602a 100644 (file)
@@ -86,7 +86,7 @@ WARNING:  TIMESTAMP(7) precision reduced to maximum allowed, 6
  t
 (1 row)
 
--- current_role/user/user is tested in rolnames.sql
+-- current_role/user/user is tested in rolenames.sql
 -- current database / catalog
 SELECT current_catalog = current_database();
  ?column? 
index 285022e07c6232f4d0fcbabe85b66393bd75a1f9..f9da7b1508f38425d42a5bf595944c4b2068da06 100644 (file)
@@ -204,7 +204,7 @@ create table header_copytest (
    b int,
    c text
 );
--- Make sure it works with with dropped columns
+-- Make sure it works with dropped columns
 alter table header_copytest drop column c;
 alter table header_copytest add column c text;
 copy header_copytest to stdout with (header match);
index fea5358d2526ae2aa44b374365320d5ec5ce5d06..d315ef5af50a7e734a4c5891b54d0dfeff70f480 100644 (file)
@@ -29,7 +29,7 @@ SELECT current_timestamp = current_timestamp(7);
 SELECT localtime = localtime(7);
 SELECT localtimestamp = localtimestamp(7);
 
--- current_role/user/user is tested in rolnames.sql
+-- current_role/user/user is tested in rolenames.sql
 
 -- current database / catalog
 SELECT current_catalog = current_database();
index 0bc209bbd4d048d03a1a91731f0a8fdec3a61ed4..b6344b936a5b203e704a163cb2c4e81439504492 100644 (file)
@@ -257,7 +257,7 @@ The certificate file to use. Implementation is SSL backend specific.
 
 =item keyfile => B<value>
 
-The private key to use. Implementation is SSL backend specific.
+The private key file to use. Implementation is SSL backend specific.
 
 =item crlfile => B<value>