Fix various common mispellings.

gsstark · gsstark · commit e1623c3959aa · 2016-06-03T16:08:45.000+01:00
Mostly these are just comments but there are a few in documentation and a handful in code and tests. Hopefully this doesn't cause too much unnecessary pain for backpatching. I relented from some of the most common like "thru" for that reason. The rest don't seem numerous enough to cause problems. Thanks to Kevin Lyda's tool https://pypi.python.org/pypi/misspellings
diff --git a/doc/src/sgml/config.sgml b/doc/src/sgml/config.sgml
@@ -2518,7 +2518,7 @@ include_dir 'conf.d'
         less than <varname>wal_writer_flush_after</> bytes of WAL have been
         produced since, WAL is only written to the OS, not flushed to disk.
         If <varname>wal_writer_flush_after</> is set to <literal>0</> WAL is
-        flushed everytime the WAL writer has written WAL.  The default is
+        flushed every time the WAL writer has written WAL.  The default is
         <literal>1MB</literal>. This parameter can only be set in the
         <filename>postgresql.conf</> file or on the server command line.
        </para>
diff --git a/doc/src/sgml/pgcrypto.sgml b/doc/src/sgml/pgcrypto.sgml
@@ -866,7 +866,7 @@ Applies to: pgp_sym_encrypt
     be a value between 1024 and 65011712, inclusive.
    </para>
 <literallayout>
-Default: A random value bewteen 65536 and 253952
+Default: A random value between 65536 and 253952
 Applies to: pgp_sym_encrypt, only with s2k-mode=3
 </literallayout>
   </sect4>
diff --git a/doc/src/sgml/ref/psql-ref.sgml b/doc/src/sgml/ref/psql-ref.sgml
@@ -4168,7 +4168,7 @@ testdb=&gt; <userinput>\crosstabview first second</userinput>
 </programlisting>
 
 This second example shows a multiplication table with rows sorted in reverse
-numerical order and columns with an independant, ascending numerical order.
+numerical order and columns with an independent, ascending numerical order.
 <programlisting>
 testdb=&gt; <userinput>SELECT t1.first as "A", t2.first+100 AS "B", t1.first*(t2.first+100) as "AxB",</userinput>
 testdb(&gt; <userinput>row_number() over(order by t2.first) AS ord</userinput>
diff --git a/src/backend/access/transam/generic_xlog.c b/src/backend/access/transam/generic_xlog.c
@@ -64,7 +64,7 @@ struct GenericXLogState
 	/*
 	 * page's images. Should be first in this struct to have MAXALIGN'ed
 	 * images addresses, because some code working with pages directly aligns
-	 * addresses, not an offsets from begining of page
+	 * addresses, not offsets from beginning of page
 	 */
 	char		images[MAX_GENERIC_XLOG_PAGES * BLCKSZ];
 	PageData	pages[MAX_GENERIC_XLOG_PAGES];
diff --git a/src/backend/catalog/namespace.c b/src/backend/catalog/namespace.c
@@ -297,7 +297,7 @@ RangeVarGetRelidExtended(const RangeVar *relation, LOCKMODE lockmode,
 					namespaceId = LookupExplicitNamespace(relation->schemaname, missing_ok);
 
 					/*
-					 * For missing_ok, allow a non-existant schema name to
+					 * For missing_ok, allow a non-existent schema name to
 					 * return InvalidOid.
 					 */
 					if (namespaceId != myTempNamespace)
diff --git a/src/backend/catalog/sql_features.txt b/src/backend/catalog/sql_features.txt
@@ -316,7 +316,7 @@ F831	Full cursor update			NO
 F831	Full cursor update	01	Updatable scrollable cursors	NO	
 F831	Full cursor update	02	Updatable ordered cursors	NO	
 F841	LIKE_REGEX predicate			NO	
-F842	OCCURENCES_REGEX function			NO	
+F842	OCCURRENCES_REGEX function			NO	
 F843	POSITION_REGEX function			NO	
 F844	SUBSTRING_REGEX function			NO	
 F845	TRANSLATE_REGEX function			NO	
diff --git a/src/backend/commands/cluster.c b/src/backend/commands/cluster.c
@@ -248,7 +248,7 @@ cluster(ClusterStmt *stmt, bool isTopLevel)
  * swapping the relfilenodes of the new table and the old table, so
  * the OID of the original table is preserved.  Thus we do not lose
  * GRANT, inheritance nor references to this table (this was a bug
- * in releases thru 7.3).
+ * in releases through 7.3).
  *
  * Indexes are rebuilt too, via REINDEX. Since we are effectively bulk-loading
  * the new table, it's better to create the indexes afterwards than to fill
diff --git a/src/backend/main/main.c b/src/backend/main/main.c
@@ -179,7 +179,7 @@ main(int argc, char *argv[])
 		 * read-only activities.  The -C case is important because pg_ctl may
 		 * try to invoke it while still holding administrator privileges on
 		 * Windows.  Note that while -C can normally be in any argv position,
-		 * if you wanna bypass the root check you gotta put it first.  This
+		 * if you want to bypass the root check you must put it first.  This
 		 * reduces the risk that we might misinterpret some other mode's -C
 		 * switch as being the postmaster/postgres one.
 		 */
diff --git a/src/backend/nodes/README b/src/backend/nodes/README
@@ -46,7 +46,7 @@ FILES IN src/include/nodes/
 Steps to Add a Node
 -------------------
 
-Suppose you wanna define a node Foo:
+Suppose you want to define a node Foo:
 
 1. Add a tag (T_Foo) to the enum NodeTag in nodes.h.  (If you insert the
    tag in a way that moves the numbers associated with existing tags,
diff --git a/src/backend/optimizer/util/tlist.c b/src/backend/optimizer/util/tlist.c
@@ -770,7 +770,7 @@ apply_pathtarget_labeling_to_tlist(List *tlist, PathTarget *target)
  *	  the aggtranstype or aggserialtype. This allows exprType() to return the
  *	  actual type that will be produced.
  *
- * Note: We expect 'target' to be a flat target list and not have Aggrefs burried
+ * Note: We expect 'target' to be a flat target list and not have Aggrefs buried
  * within other expressions.
  */
 void
diff --git a/src/backend/replication/logical/reorderbuffer.c b/src/backend/replication/logical/reorderbuffer.c
@@ -563,7 +563,7 @@ ReorderBufferTXNByXid(ReorderBuffer *rb, TransactionId xid, bool create,
 		}
 
 		/*
-		 * cached as non-existant, and asked not to create? Then nothing else
+		 * cached as non-existent, and asked not to create? Then nothing else
 		 * to do.
 		 */
 		if (!create)
diff --git a/src/backend/replication/walsender.c b/src/backend/replication/walsender.c
@@ -747,7 +747,7 @@ StartReplication(StartReplicationCmd *cmd)
  *
  * Inside the walsender we can do better than logical_read_local_xlog_page,
  * which has to do a plain sleep/busy loop, because the walsender's latch gets
- * set everytime WAL is flushed.
+ * set every time WAL is flushed.
  */
 static int
 logical_read_xlog_page(XLogReaderState *state, XLogRecPtr targetPagePtr, int reqLen,
diff --git a/src/backend/storage/buffer/bufmgr.c b/src/backend/storage/buffer/bufmgr.c
@@ -1973,7 +1973,7 @@ BufferSync(int flags)
 		}
 
 		/*
-		 * Measure progress independent of actualy having to flush the buffer
+		 * Measure progress independent of actually having to flush the buffer
 		 * - otherwise writing become unbalanced.
 		 */
 		ts_stat->progress += ts_stat->progress_slice;
diff --git a/src/backend/storage/ipc/procarray.c b/src/backend/storage/ipc/procarray.c
@@ -642,7 +642,7 @@ ProcArrayInitRecovery(TransactionId initializedUptoXID)
 	Assert(TransactionIdIsNormal(initializedUptoXID));
 
 	/*
-	 * we set latestObservedXid to the xid SUBTRANS has been initialized upto,
+	 * we set latestObservedXid to the xid SUBTRANS has been initialized up to,
 	 * so we can extend it from that point onwards in
 	 * RecordKnownAssignedTransactionIds, and when we get consistent in
 	 * ProcArrayApplyRecoveryInfo().
diff --git a/src/backend/storage/smgr/md.c b/src/backend/storage/smgr/md.c
@@ -1788,7 +1788,7 @@ _mdfd_getseg(SMgrRelation reln, ForkNumber forknum, BlockNumber blkno,
 	BlockNumber targetseg;
 	BlockNumber nextsegno;
 
-	/* some way to handle non-existant segments needs to be specified */
+	/* some way to handle non-existent segments needs to be specified */
 	Assert(behavior &
 		   (EXTENSION_FAIL | EXTENSION_CREATE | EXTENSION_RETURN_NULL));
 
diff --git a/src/backend/tsearch/spell.c b/src/backend/tsearch/spell.c
@@ -1162,7 +1162,7 @@ getAffixFlagSet(IspellDict *Conf, char *s)
 
 		if (curaffix > 0 && curaffix <= Conf->nAffixData)
 			/*
-			 * Do not substract 1 from curaffix
+			 * Do not subtract 1 from curaffix
 			 * because empty string was added in NIImportOOAffixes
 			 */
 			return Conf->AffixData[curaffix];
diff --git a/src/backend/utils/adt/formatting.c b/src/backend/utils/adt/formatting.c
@@ -3146,15 +3146,15 @@ DCH_from_char(FormatNode *node, char *in, TmFromChar *out)
 				{
 					int			matched,
 								years,
-								millenia,
+								millennia,
 								nch;
 
-					matched = sscanf(s, "%d,%03d%n", &millenia, &years, &nch);
+					matched = sscanf(s, "%d,%03d%n", &millennia, &years, &nch);
 					if (matched < 2)
 						ereport(ERROR,
 								(errcode(ERRCODE_INVALID_DATETIME_FORMAT),
 							  errmsg("invalid input string for \"Y,YYY\"")));
-					years += (millenia * 1000);
+					years += (millennia * 1000);
 					from_char_set_int(&out->year, years, n);
 					out->yysz = 4;
 					s += nch;
diff --git a/src/backend/utils/adt/geo_spgist.c b/src/backend/utils/adt/geo_spgist.c
@@ -45,7 +45,7 @@
  *	(3) the prefix of the current node
  *
  * If we visualize them on our simplified drawing (see the drawing above);
- * transfered boundaries of (1) would be the outer axis, relevant part
+ * transferred boundaries of (1) would be the outer axis, relevant part
  * of (2) would be the up right part of the other axis, and (3) would be
  * the inner axis.
  *
diff --git a/src/backend/utils/resowner/resowner.c b/src/backend/utils/resowner/resowner.c
@@ -373,7 +373,7 @@ ResourceArrayGetAny(ResourceArray *resarr, Datum *value)
 	}
 	else
 	{
-		/* Hash: search forward from whereever we were last. */
+		/* Hash: search forward from wherever we were last. */
 		uint32		mask = resarr->capacity - 1;
 
 		for (;;)
diff --git a/src/bin/pg_dump/dumputils.c b/src/bin/pg_dump/dumputils.c
@@ -236,7 +236,7 @@ buildACLCommands(const char *name, const char *subname,
 			 * manner by first REVOKE'ing the rights and then GRANT'ing them
 			 * after.  With 9.6 and above, what we need to REVOKE and what we
 			 * need to GRANT is figured out when we dump and stashed into
-			 * "racls" and "acls", respectivly.  See above.
+			 * "racls" and "acls", respectively.  See above.
 			 */
 			if (remoteVersion < 90600 && owner
 				&& strcmp(grantee->data, owner) == 0
diff --git a/src/include/port/win32.h b/src/include/port/win32.h
@@ -9,7 +9,7 @@
  * Leave a higher value in place. When building with at least Visual
  * Studio 2015 the minimum requirement is Windows Vista (0x0600) to
  * get support for GetLocaleInfoEx() with locales. For everything else
- * the minumum version is Windows XP (0x0501).
+ * the minimum version is Windows XP (0x0501).
  * Also for VS2015, add a define that stops compiler complaints about
  * using the old Winsock API.
  */
diff --git a/src/include/storage/freespace.h b/src/include/storage/freespace.h
@@ -33,8 +33,8 @@ extern void XLogRecordPageWithFreeSpace(RelFileNode rnode, BlockNumber heapBlk,
 extern void FreeSpaceMapTruncateRel(Relation rel, BlockNumber nblocks);
 extern void FreeSpaceMapVacuum(Relation rel);
 extern void UpdateFreeSpaceMap(Relation rel,
-							BlockNumber firtsBlkNum,
-							BlockNumber lastBlkNum,
+							BlockNumber startBlkNum,
+							BlockNumber endBlkNum,
 							Size freespace);
 
 #endif   /* FREESPACE_H_ */
diff --git a/src/test/modules/test_ddl_deparse/README b/src/test/modules/test_ddl_deparse/README
@@ -4,5 +4,5 @@ demonstration of how to use the datatype, and to provide some unit tests for
 it.
 
 The functions in this extension are intended to be able to process some
-part of the struct and produce some readable output, preferrably handling
+part of the struct and produce some readable output, preferably handling
 all possible cases so that SQL test code can be written.
diff --git a/src/test/regress/expected/event_trigger.out b/src/test/regress/expected/event_trigger.out
@@ -30,14 +30,14 @@ create event trigger regress_event_trigger_end on ddl_command_end
    execute procedure test_event_trigger();
 -- should fail, food is not a valid filter variable
 create event trigger regress_event_trigger2 on ddl_command_start
-   when food in ('sandwhich')
+   when food in ('sandwich')
    execute procedure test_event_trigger();
 ERROR:  unrecognized filter variable "food"
--- should fail, sandwhich is not a valid command tag
+-- should fail, sandwich is not a valid command tag
 create event trigger regress_event_trigger2 on ddl_command_start
-   when tag in ('sandwhich')
+   when tag in ('sandwich')
    execute procedure test_event_trigger();
-ERROR:  filter value "sandwhich" not recognized for filter variable "tag"
+ERROR:  filter value "sandwich" not recognized for filter variable "tag"
 -- should fail, create skunkcabbage is not a valid command tag
 create event trigger regress_event_trigger2 on ddl_command_start
    when tag in ('create table', 'create skunkcabbage')
diff --git a/src/test/regress/sql/event_trigger.sql b/src/test/regress/sql/event_trigger.sql
@@ -32,12 +32,12 @@ create event trigger regress_event_trigger_end on ddl_command_end
 
 -- should fail, food is not a valid filter variable
 create event trigger regress_event_trigger2 on ddl_command_start
-   when food in ('sandwhich')
+   when food in ('sandwich')
    execute procedure test_event_trigger();
 
--- should fail, sandwhich is not a valid command tag
+-- should fail, sandwich is not a valid command tag
 create event trigger regress_event_trigger2 on ddl_command_start
-   when tag in ('sandwhich')
+   when tag in ('sandwich')
    execute procedure test_event_trigger();
 
 -- should fail, create skunkcabbage is not a valid command tag
diff --git a/src/tools/msvc/Project.pm b/src/tools/msvc/Project.pm
@@ -368,7 +368,7 @@ sub Save
 
 sub GetAdditionalLinkerDependencies
 {
-	my ($self, $cfgname, $seperator) = @_;
+	my ($self, $cfgname, $separator) = @_;
 	my $libcfg = (uc $cfgname eq "RELEASE") ? "MD" : "MDd";
 	my $libs = '';
 	foreach my $lib (@{ $self->{libraries} })
@@ -382,7 +382,7 @@ sub GetAdditionalLinkerDependencies
 				last;
 			}
 		}
-		$libs .= $xlib . $seperator;
+		$libs .= $xlib . $separator;
 	}
 	$libs =~ s/.$//;
 	$libs =~ s/__CFGNAME__/$cfgname/g;

Original file line number	Diff line number	Diff line change
`@@ -248,7 +248,7 @@ cluster(ClusterStmt *stmt, bool isTopLevel)`
`248`	`248`	`* swapping the relfilenodes of the new table and the old table, so`
`249`	`249`	`* the OID of the original table is preserved. Thus we do not lose`
`250`	`250`	`* GRANT, inheritance nor references to this table (this was a bug`
`251`		`- * in releases thru 7.3).`
	`251`	`+ * in releases through 7.3).`
`252`	`252`	`*`
`253`	`253`	`* Indexes are rebuilt too, via REINDEX. Since we are effectively bulk-loading`
`254`	`254`	`* the new table, it's better to create the indexes afterwards than to fill`
Original file line number	Diff line number	Diff line change
`@@ -770,7 +770,7 @@ apply_pathtarget_labeling_to_tlist(List tlist, PathTarget target)`
`770`	`770`	`* the aggtranstype or aggserialtype. This allows exprType() to return the`
`771`	`771`	`* actual type that will be produced.`
`772`	`772`	`*`
`773`		`- * Note: We expect 'target' to be a flat target list and not have Aggrefs burried`
	`773`	`+ * Note: We expect 'target' to be a flat target list and not have Aggrefs buried`
`774`	`774`	`* within other expressions.`
`775`	`775`	`*/`
`776`	`776`	`void`