Fix collection of typos and grammar mistakes in the tree

This fixes some comments and documentation new as of Postgres 13.

Author: Justin Pryzby
Discussion: https://postgr.es/m/20200408165653[email protected]

diff --git a/doc/src/sgml/btree.sgml b/doc/src/sgml/btree.sgml
index f02e02b0acc270272acdc843a74b60b08a99a2b3..e9cab4a55dc6fcbb31b2b3d6249ace1a48d08f68 100644 (file)
--- a/doc/src/sgml/btree.sgml
+++ b/doc/src/sgml/btree.sgml
@@ -685,7 +685,7 @@ equalimage(opcintype oid) returns bool
    Deduplication cannot be used in all cases due to
    implementation-level restrictions.  Deduplication safety is
    determined when CREATE INDEX or
-   REINDEX run.
+   REINDEX is run.
   
   
    Note that deduplication is deemed unsafe and cannot be used in the

diff --git a/doc/src/sgml/catalogs.sgml b/doc/src/sgml/catalogs.sgml
index 386c6d7bd1b65b287d6c5d5036422c3ae9604cac..ce33df9e581963abcd47c546071699b4de954655 100644 (file)
--- a/doc/src/sgml/catalogs.sgml
+++ b/doc/src/sgml/catalogs.sgml
@@ -6472,7 +6472,7 @@ SCRAM-SHA-256$<iteration count>:&l
        .
        A zero value indicates that no statistics should be collected.
        A negative value says to use the system default statistics target.
-       Positive values stxstattarget
+       Positive values of stxstattarget
        determine the target number of most common values
        to collect.
       

diff --git a/doc/src/sgml/config.sgml b/doc/src/sgml/config.sgml
index a0da4aabac059b8f953201f9bc75bdb271b67ac5..a71ca624639422b04282dbf3428669be5a20900c 100644 (file)
--- a/doc/src/sgml/config.sgml
+++ b/doc/src/sgml/config.sgml
@@ -112,7 +112,7 @@
        For example, 30.1 GB will be converted
        to 30822 MB not 32319628902 B.
        If the parameter is of integer type, a final rounding to integer
-       occurs after any units conversion.
+       occurs after any unit conversion.
       
      
 
@@ -6110,8 +6110,8 @@ local0.*    /var/log/postgresql
          unoptimized queries in your applications.
          If this value is specified without units, it is taken as milliseconds.
          Setting this to zero prints all statement durations.
-         Minus-one (the default) disables logging statement durations.
-         Only superusers can change this setting.
+         -1 (the default) disables logging statement
+         durations. Only superusers can change this setting.
         
 
         
@@ -6162,8 +6162,8 @@ local0.*    /var/log/postgresql
          traffic is too high to log all queries.
          If this value is specified without units, it is taken as milliseconds.
          Setting this to zero samples all statement durations.
-         Minus-one (the default) disables sampling statement durations.
-         Only superusers can change this setting.
+         -1 (the default) disables sampling statement
+         durations. Only superusers can change this setting.
         
 
         

diff --git a/doc/src/sgml/func.sgml b/doc/src/sgml/func.sgml
index a38387b8c60c3158014ac2dab38fb84da4474718..12d75b476f3ea31f6051f3f188bd98950f7fa066 100644 (file)
--- a/doc/src/sgml/func.sgml
+++ b/doc/src/sgml/func.sgml
@@ -10006,7 +10006,7 @@ gen_random_uuid() returns uuid
    Producing XML Content
 
    
-    A set of functions and function-like expressions are available for
+    A set of functions and function-like expressions is available for
     producing XML content from SQL data.  As such, they are
     particularly suitable for formatting query results into XML
     documents for processing in client applications.
@@ -12360,7 +12360,7 @@ table2-mapping
 
   
    
-    Some of the jsonb_path_* functions have the
+    Some of the jsonb_path_* functions have a
     _tz suffix. These functions have been implemented to
     support comparison of date/time values that involves implicit
     timezone-aware casts. Since operations with time zones are not immutable,
@@ -18795,8 +18795,8 @@ SELECT collation for ('foo' COLLATE "de_DE");
    The functions to_regclass, to_regcollation,
    to_regnamespace, to_regoper,
    to_regoperator, to_regrole,
-   to_regproc, to_regprocedure, and
-   to_regtype, functions translate relation, collation, schema,
+   to_regproc, to_regprocedure and
+   to_regtype translate relation, collation, schema,
    operator, role, function, and type names (given as text) to
    objects of the corresponding reg* type (see 
    linkend="datatype-oid"/> about the types).  These functions differ from a


diff --git a/doc/src/sgml/libpq.sgml b/doc/src/sgml/libpq.sgml

index c2b3cdabad121ad1c057e89fb592d6baae482676..75d2224a61bfd45f054acfd57eae929e25270483 100644 (file)

--- a/doc/src/sgml/libpq.sgml
+++ b/doc/src/sgml/libpq.sgml
@@ -797,7 +797,7 @@ void PQsetSSLKeyPassHook(PQsslKeyPassHook_type hook);
        its default PQdefaultSSLKeyPassHook handler. The callback
        should determine the password for the key and copy it to result-buffer
        buf of size size. The string in 
-       buf must be null-terminated. The calback must return the length of
+       buf must be null-terminated. The callback must return the length of
        the password stored in buf excluding the null terminator.
        On failure, the callback should set buf[0] = '\0' and return 0.
        See PQdefaultSSLKeyPassHook in libpq's


diff --git a/doc/src/sgml/ref/pg_validatebackup.sgml b/doc/src/sgml/ref/pg_validatebackup.sgml

index 20d445efb8d8e4a52fb7ab0e6cb0679379943ace..5c626200f683e4afac3d9454449c7788bca649d6 100644 (file)

--- a/doc/src/sgml/ref/pg_validatebackup.sgml
+++ b/doc/src/sgml/ref/pg_validatebackup.sgml
@@ -41,7 +41,7 @@ PostgreSQL documentation
   
 
   
-   It is important to note that that the validation which is performed by
+   It is important to note that the validation which is performed by
    pg_validatebackup does not and can not include
    every check which will be performed by a running server when attempting
    to make use of the backup. Even if you use this tool, you should still
@@ -73,7 +73,7 @@ PostgreSQL documentation
    a backup_manifest file in the target directory or
    about anything inside pg_wal, even though these
    files won't be listed in the backup manifest. Only files are checked;
-   the presence or absence or directories is not verified, except
+   the presence or absence of directories is not verified, except
    indirectly: if a directory is missing, any files it should have contained
    will necessarily also be missing. 
   


diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c

index 6162fb018c72f212bd8495115f2e7778afffc837..037d457c3d469b873b45ae305080059fb31b8148 100644 (file)

--- a/src/backend/commands/tablecmds.c
+++ b/src/backend/commands/tablecmds.c
@@ -16846,7 +16846,7 @@ ATExecDetachPartition(Relation rel, RangeVar *name)
    list_free_deep(fks);
 
    /*
-    * Any sub-constrains that are in the referenced-side of a larger
+    * Any sub-constraints that are in the referenced-side of a larger
     * constraint have to be removed.  This partition is no longer part of the
     * key space of the constraint.
     */


diff --git a/src/backend/executor/execExpr.c b/src/backend/executor/execExpr.c

index 1370ffec5093536b6230cd8ea9505de04e610e0b..c6a77bd66fa529d5f873f0e458a90de8efe29b73 100644 (file)

--- a/src/backend/executor/execExpr.c
+++ b/src/backend/executor/execExpr.c
@@ -2366,7 +2366,7 @@ get_last_attnums_walker(Node *node, LastAttnumInfo *info)
  * evaluation of the expression will have the same type of slot, with an
  * equivalent descriptor.
  *
- * Returns true if the the deforming step is required, false otherwise.
+ * Returns true if the deforming step is required, false otherwise.
  */
 static bool
 ExecComputeSlotInfo(ExprState *state, ExprEvalStep *op)


diff --git a/src/backend/postmaster/bgworker.c b/src/backend/postmaster/bgworker.c

index 6c684b5e12b913bd488ae96540d4f6aec4a98054..beb5e85434cef6d876ab6b4c01a36a3a6e121a1a 100644 (file)

--- a/src/backend/postmaster/bgworker.c
+++ b/src/backend/postmaster/bgworker.c
@@ -1149,7 +1149,7 @@ WaitForBackgroundWorkerShutdown(BackgroundWorkerHandle *handle)
  * Instruct the postmaster to terminate a background worker.
  *
  * Note that it's safe to do this without regard to whether the worker is
- * still running, or even if the worker may already have existed and been
+ * still running, or even if the worker may already have exited and been
  * unregistered.
  */
 void


diff --git a/src/backend/replication/basebackup.c b/src/backend/replication/basebackup.c

index 50a9e9c28040fe76935b4a9525cd1b6656f15cf0..f5b2411d5495a6a3ce7fe6ee29cc770f5d2fe820 100644 (file)

--- a/src/backend/replication/basebackup.c
+++ b/src/backend/replication/basebackup.c
@@ -1157,7 +1157,7 @@ AddFileToManifest(manifest_info *manifest, const char *spcoid,
    }
 
    /*
-    * Each file's entry need to be separated from any entry that follows by a
+    * Each file's entry needs to be separated from any entry that follows by a
     * comma, but there's no comma before the first one or after the last one.
     * To make that work, adding a file to the manifest starts by terminating
     * the most recently added line, with a comma if appropriate, but does not


diff --git a/src/include/lib/simplehash.h b/src/include/lib/simplehash.h

index 8cb03cda6cc6ba52bfc14fa187bcb07c4934778b..f7af921f5aca8c8dbdfca1ccf28073d61c088d56 100644 (file)

--- a/src/include/lib/simplehash.h
+++ b/src/include/lib/simplehash.h
@@ -633,7 +633,7 @@ restart:
            /* shift forward, starting at last occupied element */
 
            /*
-            * TODO: This could be optimized to be one memcpy in may cases,
+            * TODO: This could be optimized to be one memcpy in many cases,
             * excepting wrapping around at the end of ->data. Hasn't shown up
             * in profiles so far though.
             */

diff --git a/doc/src/sgml/libpq.sgml b/doc/src/sgml/libpq.sgml
index c2b3cdabad121ad1c057e89fb592d6baae482676..75d2224a61bfd45f054acfd57eae929e25270483 100644 (file)
--- a/doc/src/sgml/libpq.sgml
+++ b/doc/src/sgml/libpq.sgml
@@ -797,7 +797,7 @@ void PQsetSSLKeyPassHook(PQsslKeyPassHook_type hook);
        its default PQdefaultSSLKeyPassHook handler. The callback
        should determine the password for the key and copy it to result-buffer
        buf of size size. The string in 
-       buf must be null-terminated. The calback must return the length of
+       buf must be null-terminated. The callback must return the length of
        the password stored in buf excluding the null terminator.
        On failure, the callback should set buf[0] = '\0' and return 0.
        See PQdefaultSSLKeyPassHook in libpq's

diff --git a/doc/src/sgml/ref/pg_validatebackup.sgml b/doc/src/sgml/ref/pg_validatebackup.sgml
index 20d445efb8d8e4a52fb7ab0e6cb0679379943ace..5c626200f683e4afac3d9454449c7788bca649d6 100644 (file)
--- a/doc/src/sgml/ref/pg_validatebackup.sgml
+++ b/doc/src/sgml/ref/pg_validatebackup.sgml
@@ -41,7 +41,7 @@ PostgreSQL documentation
   
 
   
-   It is important to note that that the validation which is performed by
+   It is important to note that the validation which is performed by
    pg_validatebackup does not and can not include
    every check which will be performed by a running server when attempting
    to make use of the backup. Even if you use this tool, you should still
@@ -73,7 +73,7 @@ PostgreSQL documentation
    a backup_manifest file in the target directory or
    about anything inside pg_wal, even though these
    files won't be listed in the backup manifest. Only files are checked;
-   the presence or absence or directories is not verified, except
+   the presence or absence of directories is not verified, except
    indirectly: if a directory is missing, any files it should have contained
    will necessarily also be missing. 
   

diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c
index 6162fb018c72f212bd8495115f2e7778afffc837..037d457c3d469b873b45ae305080059fb31b8148 100644 (file)
--- a/src/backend/commands/tablecmds.c
+++ b/src/backend/commands/tablecmds.c
@@ -16846,7 +16846,7 @@ ATExecDetachPartition(Relation rel, RangeVar *name)
    list_free_deep(fks);
 
    /*
-    * Any sub-constrains that are in the referenced-side of a larger
+    * Any sub-constraints that are in the referenced-side of a larger
     * constraint have to be removed.  This partition is no longer part of the
     * key space of the constraint.
     */

diff --git a/src/backend/executor/execExpr.c b/src/backend/executor/execExpr.c
index 1370ffec5093536b6230cd8ea9505de04e610e0b..c6a77bd66fa529d5f873f0e458a90de8efe29b73 100644 (file)
--- a/src/backend/executor/execExpr.c
+++ b/src/backend/executor/execExpr.c
@@ -2366,7 +2366,7 @@ get_last_attnums_walker(Node *node, LastAttnumInfo *info)
  * evaluation of the expression will have the same type of slot, with an
  * equivalent descriptor.
  *
- * Returns true if the the deforming step is required, false otherwise.
+ * Returns true if the deforming step is required, false otherwise.
  */
 static bool
 ExecComputeSlotInfo(ExprState *state, ExprEvalStep *op)

diff --git a/src/backend/postmaster/bgworker.c b/src/backend/postmaster/bgworker.c
index 6c684b5e12b913bd488ae96540d4f6aec4a98054..beb5e85434cef6d876ab6b4c01a36a3a6e121a1a 100644 (file)
--- a/src/backend/postmaster/bgworker.c
+++ b/src/backend/postmaster/bgworker.c
@@ -1149,7 +1149,7 @@ WaitForBackgroundWorkerShutdown(BackgroundWorkerHandle *handle)
  * Instruct the postmaster to terminate a background worker.
  *
  * Note that it's safe to do this without regard to whether the worker is
- * still running, or even if the worker may already have existed and been
+ * still running, or even if the worker may already have exited and been
  * unregistered.
  */
 void

diff --git a/src/backend/replication/basebackup.c b/src/backend/replication/basebackup.c
index 50a9e9c28040fe76935b4a9525cd1b6656f15cf0..f5b2411d5495a6a3ce7fe6ee29cc770f5d2fe820 100644 (file)
--- a/src/backend/replication/basebackup.c
+++ b/src/backend/replication/basebackup.c
@@ -1157,7 +1157,7 @@ AddFileToManifest(manifest_info *manifest, const char *spcoid,
    }
 
    /*
-    * Each file's entry need to be separated from any entry that follows by a
+    * Each file's entry needs to be separated from any entry that follows by a
     * comma, but there's no comma before the first one or after the last one.
     * To make that work, adding a file to the manifest starts by terminating
     * the most recently added line, with a comma if appropriate, but does not

diff --git a/src/include/lib/simplehash.h b/src/include/lib/simplehash.h
index 8cb03cda6cc6ba52bfc14fa187bcb07c4934778b..f7af921f5aca8c8dbdfca1ccf28073d61c088d56 100644 (file)
--- a/src/include/lib/simplehash.h
+++ b/src/include/lib/simplehash.h
@@ -633,7 +633,7 @@ restart:
            /* shift forward, starting at last occupied element */
 
            /*
-            * TODO: This could be optimized to be one memcpy in may cases,
+            * TODO: This could be optimized to be one memcpy in many cases,
             * excepting wrapping around at the end of ->data. Hasn't shown up
             * in profiles so far though.
             */