[PATCH 0/3] Typo fixes

[PATCH 0/3] Typo fixes

[Mike Ralphson]

Three patches as a result of running the documentation and
source through aspell. If there is interest in carrying the
script and wordlists used in-tree, then I can submit patches
to add them to contrib/ maybe?

Mike

[PATCH 1/3] Documentation: fix typos / spelling mistakes

[Mike Ralphson]

Signed-off-by: Mike Ralphson <mike@abacus.co.uk>
---
 Documentation/git-cvsimport.txt         |    2 +-
 Documentation/git-format-patch.txt      |    2 +-
 contrib/thunderbird-patch-inline/README |    4 ++--
 gitweb/README                           |    2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/Documentation/git-cvsimport.txt b/Documentation/git-cvsimport.txt
index d7bab13..614e769 100644
--- a/Documentation/git-cvsimport.txt
+++ b/Documentation/git-cvsimport.txt
@@ -196,7 +196,7 @@ Problems related to tags:
 
 If you suspect that any of these issues may apply to the repository you
 want to import consider using these alternative tools which proved to be
-more stable in practise:
+more stable in practice:
 
 * cvs2git (part of cvs2svn), `http://cvs2svn.tigris.org`
 * parsecvs, `http://cgit.freedesktop.org/~keithp/parsecvs`
diff --git a/Documentation/git-format-patch.txt b/Documentation/git-format-patch.txt
index eb2fbcf..5eddca9 100644
--- a/Documentation/git-format-patch.txt
+++ b/Documentation/git-format-patch.txt
@@ -194,7 +194,7 @@ CONFIGURATION
 -------------
 You can specify extra mail header lines to be added to each message
 in the repository configuration, new defaults for the subject prefix
-and file suffix, control attachements, and number patches when outputting
+and file suffix, control attachments, and number patches when outputting
 more than one.
 
 ------------
diff --git a/contrib/thunderbird-patch-inline/README b/contrib/thunderbird-patch-inline/README
index 39f96aa..000147b 100644
--- a/contrib/thunderbird-patch-inline/README
+++ b/contrib/thunderbird-patch-inline/README
@@ -1,12 +1,12 @@
 appp.sh is a script that is supposed to be used together with ExternalEditor
-for Mozilla Thundebird. It will let you include patches inline in e-mails
+for Mozilla Thunderbird. It will let you include patches inline in e-mails
 in an easy way.
 
 Usage:
 - Generate the patch with git format-patch.
 - Start writing a new e-mail in Thunderbird.
 - Press the external editor button (or Ctrl-E) to run appp.sh
-- Select the previosly generated patch file.
+- Select the previously generated patch file.
 - Finish editing the e-mail.
 
 Any text that is entered into the message editor before appp.sh is called
diff --git a/gitweb/README b/gitweb/README
index 8433dd1..ad73c6a 100644
--- a/gitweb/README
+++ b/gitweb/README
@@ -206,7 +206,7 @@ not include variables usually directly set during build):
  * $fallback_encoding
    Gitweb assumes this charset if line contains non-UTF-8 characters.
    Fallback decoding is used without error checking, so it can be even
-   'utf-8'. Value mist be valid encodig; see Encoding::Supported(3pm) man
+   'utf-8'. Value mist be valid encoding; see Encoding::Supported(3pm) man
    page for a list.   By default 'latin1', aka. 'iso-8859-1'.
  * @diff_opts
    Rename detection options for git-diff and git-diff-tree. By default
-- 
1.6.0.2.229.g1293c.dirty

[PATCH 2/3] builtin-remote: fix typo in option description

[Mike Ralphson]

Signed-off-by: Mike Ralphson <mike@abacus.co.uk>
---
 builtin-remote.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/builtin-remote.c b/builtin-remote.c
index ca7c639..2ed752c 100644
--- a/builtin-remote.c
+++ b/builtin-remote.c
@@ -1216,7 +1216,7 @@ static int update(int argc, const char **argv)
 	struct option options[] = {
 		OPT_GROUP("update specific options"),
 		OPT_BOOLEAN('p', "prune", &prune,
-			    "prune remotes after fecthing"),
+			    "prune remotes after fetching"),
 		OPT_END()
 	};
 
-- 
1.6.0.2.229.g1293c.dirty

[PATCH 3/3] Fix typos / spelling in comments

[Mike Ralphson]

Signed-off-by: Mike Ralphson <mike@abacus.co.uk>
---
 builtin-checkout.c       |    2 +-
 builtin-ls-files.c       |    2 +-
 builtin-pack-objects.c   |    2 +-
 builtin-reflog.c         |    2 +-
 builtin-reset.c          |    2 +-
 compat/cygwin.c          |    2 +-
 compat/fnmatch/fnmatch.c |    4 ++--
 compat/mingw.c           |    2 +-
 compat/regex/regex.c     |    4 ++--
 diffcore-rename.c        |    2 +-
 fast-import.c            |    6 +++---
 fsck.h                   |    2 +-
 git.c                    |    2 +-
 graph.c                  |    2 +-
 levenshtein.c            |    2 +-
 parse-options.h          |    8 ++++----
 revision.c               |    2 +-
 strbuf.h                 |    2 +-
 tree-diff.c              |    2 +-
 xdiff/xdiffi.c           |    2 +-
 20 files changed, 27 insertions(+), 27 deletions(-)

diff --git a/builtin-checkout.c b/builtin-checkout.c
index ee1edd4..383598c 100644
--- a/builtin-checkout.c
+++ b/builtin-checkout.c
@@ -178,7 +178,7 @@ static int checkout_merged(int pos, struct checkout *state)
 	/*
 	 * NEEDSWORK:
 	 * There is absolutely no reason to write this as a blob object
-	 * and create a phoney cache entry just to leak.  This hack is
+	 * and create a phony cache entry just to leak.  This hack is
 	 * primarily to get to the write_entry() machinery that massages
 	 * the contents to work-tree format and writes out which only
 	 * allows it for a cache entry.  The code in write_entry() needs
diff --git a/builtin-ls-files.c b/builtin-ls-files.c
index 88e2697..da2daf4 100644
--- a/builtin-ls-files.c
+++ b/builtin-ls-files.c
@@ -512,7 +512,7 @@ int cmd_ls_files(int argc, const char **argv, const char *prefix)
 
 	pathspec = get_pathspec(prefix, argv);
 
-	/* be nice with submodule patsh ending in a slash */
+	/* be nice with submodule paths ending in a slash */
 	read_cache();
 	if (pathspec)
 		strip_trailing_slash_from_submodules();
diff --git a/builtin-pack-objects.c b/builtin-pack-objects.c
index d3360ac..1af6c62 100644
--- a/builtin-pack-objects.c
+++ b/builtin-pack-objects.c
@@ -293,7 +293,7 @@ static unsigned long write_object(struct sha1file *f,
 				die("unable to read %s", sha1_to_hex(entry->idx.sha1));
 			/*
 			 * make sure no cached delta data remains from a
-			 * previous attempt before a pack split occured.
+			 * previous attempt before a pack split occurred.
 			 */
 			free(entry->delta_data);
 			entry->delta_data = NULL;
diff --git a/builtin-reflog.c b/builtin-reflog.c
index 249ad2a..ff8b4f6 100644
--- a/builtin-reflog.c
+++ b/builtin-reflog.c
@@ -243,7 +243,7 @@ static void mark_reachable(struct commit *commit, unsigned long expire_limit)
 	 * We need to compute if commit on either side of an reflog
 	 * entry is reachable from the tip of the ref for all entries.
 	 * Mark commits that are reachable from the tip down to the
-	 * time threashold first; we know a commit marked thusly is
+	 * time threshold first; we know a commit marked thusly is
 	 * reachable from the tip without running in_merge_bases()
 	 * at all.
 	 */
diff --git a/builtin-reset.c b/builtin-reset.c
index c0cb915..7e7ebab 100644
--- a/builtin-reset.c
+++ b/builtin-reset.c
@@ -228,7 +228,7 @@ int cmd_reset(int argc, const char **argv, const char *prefix)
 		}
 		/*
 		 * Otherwise, argv[i] could be either <rev> or <paths> and
-		 * has to be unambigous.
+		 * has to be unambiguous.
 		 */
 		else if (!get_sha1(argv[i], sha1)) {
 			/*
diff --git a/compat/cygwin.c b/compat/cygwin.c
index ebac148..1192874 100644
--- a/compat/cygwin.c
+++ b/compat/cygwin.c
@@ -92,7 +92,7 @@ static int cygwin_stat(const char *path, struct stat *buf)
  * Reading this option is not always possible immediately as git_dir may be
  * not be set yet. So until it is set, use cygwin lstat/stat functions.
  * However, if core.filemode is set, we must use the Cygwin posix
- * stat/lstat as the Windows stat fuctions do not determine posix filemode.
+ * stat/lstat as the Windows stat functions do not determine posix filemode.
  *
  * Note that git_cygwin_config() does NOT call git_default_config() and this
  * is deliberate.  Many commands read from config to establish initial
diff --git a/compat/fnmatch/fnmatch.c b/compat/fnmatch/fnmatch.c
index 1f4ead5..03157a4 100644
--- a/compat/fnmatch/fnmatch.c
+++ b/compat/fnmatch/fnmatch.c
@@ -39,7 +39,7 @@
 # include <stdlib.h>
 #endif
 
-/* For platform which support the ISO C amendement 1 functionality we
+/* For platform which support the ISO C amendment 1 functionality we
    support user defined character classes.  */
 #if defined _LIBC || (defined HAVE_WCTYPE_H && defined HAVE_WCHAR_H)
 /* Solaris 2.5 has a bug: <wchar.h> must be included before <wctype.h>.  */
@@ -90,7 +90,7 @@
 
 # if defined _LIBC || (defined HAVE_WCTYPE_H && defined HAVE_WCHAR_H)
 /* The GNU C library provides support for user-defined character classes
-   and the functions from ISO C amendement 1.  */
+   and the functions from ISO C amendment 1.  */
 #  ifdef CHARCLASS_NAME_MAX
 #   define CHAR_CLASS_MAX_LENGTH CHARCLASS_NAME_MAX
 #  else
diff --git a/compat/mingw.c b/compat/mingw.c
index 2839d9d..32d4c0c 100644
--- a/compat/mingw.c
+++ b/compat/mingw.c
@@ -396,7 +396,7 @@ repeat:
 		 * its own input data to become available. But since
 		 * the process (pack-objects) is itself CPU intensive,
 		 * it will happily pick up the time slice that we are
-		 * relinguishing here.
+		 * relinquishing here.
 		 */
 		Sleep(0);
 		goto repeat;
diff --git a/compat/regex/regex.c b/compat/regex/regex.c
index 87b33e4..5ea0075 100644
--- a/compat/regex/regex.c
+++ b/compat/regex/regex.c
@@ -1043,7 +1043,7 @@ regex_compile (pattern, size, syntax, bufp)
      they can be reliably used as array indices.  */
   register unsigned char c, c1;
 
-  /* A random tempory spot in PATTERN.  */
+  /* A random temporary spot in PATTERN.  */
   const char *p1;
 
   /* Points to the end of the buffer, where we should append.  */
@@ -1796,7 +1796,7 @@ regex_compile (pattern, size, syntax, bufp)
 		    we're all done, the pattern will look like:
 		      set_number_at <jump count> <upper bound>
 		      set_number_at <succeed_n count> <lower bound>
-		      succeed_n <after jump addr> <succed_n count>
+		      succeed_n <after jump addr> <succeed_n count>
 		      <body of loop>
 		      jump_n <succeed_n addr> <jump count>
 		    (The upper bound and `jump_n' are omitted if
diff --git a/diffcore-rename.c b/diffcore-rename.c
index 0b0d6b8..63ac998 100644
--- a/diffcore-rename.c
+++ b/diffcore-rename.c
@@ -267,7 +267,7 @@ static int find_identical_files(struct file_similarity *src,
 			int score;
 			struct diff_filespec *source = p->filespec;
 
-			/* False hash collission? */
+			/* False hash collision? */
 			if (hashcmp(source->sha1, target->sha1))
 				continue;
 			/* Non-regular files? If so, the modes must match! */
diff --git a/fast-import.c b/fast-import.c
index 23c496d..53617a1 100644
--- a/fast-import.c
+++ b/fast-import.c
@@ -76,7 +76,7 @@ Format of STDIN stream:
     delim lf;
 
      # note: declen indicates the length of binary_data in bytes.
-     # declen does not include the lf preceeding the binary data.
+     # declen does not include the lf preceding the binary data.
      #
   exact_data ::= 'data' sp declen lf
     binary_data;
@@ -134,7 +134,7 @@ Format of STDIN stream:
      #
      # In case it is not clear, the '#' that starts the comment
      # must be the first character on that the line (an lf have
-     # preceeded it).
+     # preceded it).
      #
   comment ::= '#' not_lf* lf;
   not_lf  ::= # Any byte that is not ASCII newline (LF);
@@ -953,7 +953,7 @@ static void end_packfile(void)
 		close(pack_data->pack_fd);
 		idx_name = keep_pack(create_index());
 
-		/* Register the packfile with core git's machinary. */
+		/* Register the packfile with core git's machinery. */
 		new_p = add_packed_git(idx_name, strlen(idx_name), 1);
 		if (!new_p)
 			die("core git rejected index %s", idx_name);
diff --git a/fsck.h b/fsck.h
index 990ee02..008456b 100644
--- a/fsck.h
+++ b/fsck.h
@@ -23,7 +23,7 @@ int fsck_error_function(struct object *obj, int type, const char *fmt, ...);
  * the return value is:
  *    -1	error in processing the object
  *    <0	return value of the callback, which lead to an abort
- *    >0	return value of the first sigaled error >0 (in the case of no other errors)
+ *    >0	return value of the first signaled error >0 (in the case of no other errors)
  *    0		everything OK
  */
 int fsck_walk(struct object *obj, fsck_walk_func walk, void *data);
diff --git a/git.c b/git.c
index bfb6508..cc5aaa7 100644
--- a/git.c
+++ b/git.c
@@ -497,7 +497,7 @@ int main(int argc, const char **argv)
 
 	/*
 	 * We use PATH to find git commands, but we prepend some higher
-	 * precidence paths: the "--exec-path" option, the GIT_EXEC_PATH
+	 * precedence paths: the "--exec-path" option, the GIT_EXEC_PATH
 	 * environment, and the $(gitexecdir) from the Makefile at build
 	 * time.
 	 */
diff --git a/graph.c b/graph.c
index 162a516..0daf5de 100644
--- a/graph.c
+++ b/graph.c
@@ -34,7 +34,7 @@ static void graph_padding_line(struct git_graph *graph, struct strbuf *sb);
  * newline.  A new graph line will not be printed after the final newline.
  * If the strbuf is empty, no output will be printed.
  *
- * Since the first line will not include the graph ouput, the caller is
+ * Since the first line will not include the graph output, the caller is
  * responsible for printing this line's graph (perhaps via
  * graph_show_commit() or graph_show_oneline()) before calling
  * graph_show_strbuf().
diff --git a/levenshtein.c b/levenshtein.c
index a32f4cd..fc28159 100644
--- a/levenshtein.c
+++ b/levenshtein.c
@@ -27,7 +27,7 @@
  *
  * It does so by calculating the costs of the path ending in characters
  * i (in string1) and j (in string2), respectively, given that the last
- * operation is a substition, a swap, a deletion, or an insertion.
+ * operation is a substitution, a swap, a deletion, or an insertion.
  *
  * This implementation allows the costs to be weighted:
  *
diff --git a/parse-options.h b/parse-options.h
index f8ef1db..b54eec1 100644
--- a/parse-options.h
+++ b/parse-options.h
@@ -52,7 +52,7 @@ typedef int parse_opt_cb(const struct option *, const char *arg, int unset);
  *
  * `argh`::
  *   token to explain the kind of argument this option wants. Keep it
- *   homogenous across the repository.
+ *   homogeneous across the repository.
  *
  * `help`::
  *   the short help associated to what the option does.
@@ -61,7 +61,7 @@ typedef int parse_opt_cb(const struct option *, const char *arg, int unset);
  *
  * `flags`::
  *   mask of parse_opt_option_flags.
- *   PARSE_OPT_OPTARG: says that the argument is optionnal (not for BOOLEANs)
+ *   PARSE_OPT_OPTARG: says that the argument is optional (not for BOOLEANs)
  *   PARSE_OPT_NOARG: says that this option takes no argument, for CALLBACKs
  *   PARSE_OPT_NONEG: says that this option cannot be negated
  *   PARSE_OPT_HIDDEN this option is skipped in the default usage, showed in
@@ -105,7 +105,7 @@ struct option {
 	{ OPTION_CALLBACK, (s), (l), (v), (a), (h), 0, (f) }
 
 /* parse_options() will filter out the processed options and leave the
- * non-option argments in argv[].
+ * non-option arguments in argv[].
  * Returns the number of arguments left in argv[].
  */
 extern int parse_options(int argc, const char **argv,
@@ -115,7 +115,7 @@ extern int parse_options(int argc, const char **argv,
 extern NORETURN void usage_with_options(const char * const *usagestr,
                                         const struct option *options);
 
-/*----- incremantal advanced APIs -----*/
+/*----- incremental advanced APIs -----*/
 
 enum {
 	PARSE_OPT_HELP = -1,
diff --git a/revision.c b/revision.c
index b6215cc..6007a2c 100644
--- a/revision.c
+++ b/revision.c
@@ -209,7 +209,7 @@ static struct commit *handle_commit(struct rev_info *revs, struct object *object
 	}
 
 	/*
-	 * Tree object? Either mark it uniniteresting, or add it
+	 * Tree object? Either mark it uninteresting, or add it
 	 * to the list of objects to look at later..
 	 */
 	if (object->type == OBJ_TREE) {
diff --git a/strbuf.h b/strbuf.h
index 9ee908a..eaa8704 100644
--- a/strbuf.h
+++ b/strbuf.h
@@ -11,7 +11,7 @@
  *    build complex strings/buffers whose final size isn't easily known.
  *
  *    It is NOT legal to copy the ->buf pointer away.
- *    `strbuf_detach' is the operation that detachs a buffer from its shell
+ *    `strbuf_detach' is the operation that detaches a buffer from its shell
  *    while keeping the shell valid wrt its invariants.
  *
  * 2. the ->buf member is a byte array that has at least ->len + 1 bytes
diff --git a/tree-diff.c b/tree-diff.c
index b05d0f4..edd8394 100644
--- a/tree-diff.c
+++ b/tree-diff.c
@@ -374,7 +374,7 @@ static void try_to_follow_renames(struct tree_desc *t1, struct tree_desc *t2, co
 	}
 
 	/*
-	 * Then, discard all the non-relevane file pairs...
+	 * Then, discard all the non-relevant file pairs...
 	 */
 	for (i = 0; i < q->nr; i++) {
 		struct diff_filepair *p = q->queue[i];
diff --git a/xdiff/xdiffi.c b/xdiff/xdiffi.c
index 02184d9..1ebab68 100644
--- a/xdiff/xdiffi.c
+++ b/xdiff/xdiffi.c
@@ -456,7 +456,7 @@ int xdl_change_compact(xdfile_t *xdf, xdfile_t *xdfo, long flags) {
 			/*
 			 * Record the end-of-group position in case we are matched
 			 * with a group of changes in the other file (that is, the
-			 * change record before the enf-of-group index in the other
+			 * change record before the end-of-group index in the other
 			 * file is set).
 			 */
 			ixref = rchgo[ixo - 1] ? ix: nrec;
-- 
1.6.0.2.229.g1293c.dirty

Re: [PATCH 3/3] Fix typos / spelling in comments

[Michael J Gruber]

Word based spell checkers can only do what they can...
I think we should use this occasion and have humans at least look at the
context.

Mike Ralphson venit, vidit, dixit 17.04.2009 20:13:
> Signed-off-by: Mike Ralphson <mike@abacus.co.uk>
> ---
>  builtin-checkout.c       |    2 +-
>  builtin-ls-files.c       |    2 +-
>  builtin-pack-objects.c   |    2 +-
>  builtin-reflog.c         |    2 +-
>  builtin-reset.c          |    2 +-
>  compat/cygwin.c          |    2 +-
>  compat/fnmatch/fnmatch.c |    4 ++--
>  compat/mingw.c           |    2 +-
>  compat/regex/regex.c     |    4 ++--
>  diffcore-rename.c        |    2 +-
>  fast-import.c            |    6 +++---
>  fsck.h                   |    2 +-
>  git.c                    |    2 +-
>  graph.c                  |    2 +-
>  levenshtein.c            |    2 +-
>  parse-options.h          |    8 ++++----
>  revision.c               |    2 +-
>  strbuf.h                 |    2 +-
>  tree-diff.c              |    2 +-
>  xdiff/xdiffi.c           |    2 +-
>  20 files changed, 27 insertions(+), 27 deletions(-)
> 
> diff --git a/builtin-checkout.c b/builtin-checkout.c
> index ee1edd4..383598c 100644
> --- a/builtin-checkout.c
> +++ b/builtin-checkout.c
> @@ -178,7 +178,7 @@ static int checkout_merged(int pos, struct checkout *state)
>  	/*
>  	 * NEEDSWORK:
>  	 * There is absolutely no reason to write this as a blob object
> -	 * and create a phoney cache entry just to leak.  This hack is
> +	 * and create a phony cache entry just to leak.  This hack is
>  	 * primarily to get to the write_entry() machinery that massages
>  	 * the contents to work-tree format and writes out which only
>  	 * allows it for a cache entry.  The code in write_entry() needs
> diff --git a/builtin-ls-files.c b/builtin-ls-files.c
> index 88e2697..da2daf4 100644
> --- a/builtin-ls-files.c
> +++ b/builtin-ls-files.c
> @@ -512,7 +512,7 @@ int cmd_ls_files(int argc, const char **argv, const char *prefix)
>  
>  	pathspec = get_pathspec(prefix, argv);
>  
> -	/* be nice with submodule patsh ending in a slash */
> +	/* be nice with submodule paths ending in a slash */
>  	read_cache();
>  	if (pathspec)
>  		strip_trailing_slash_from_submodules();
> diff --git a/builtin-pack-objects.c b/builtin-pack-objects.c
> index d3360ac..1af6c62 100644
> --- a/builtin-pack-objects.c
> +++ b/builtin-pack-objects.c
> @@ -293,7 +293,7 @@ static unsigned long write_object(struct sha1file *f,
>  				die("unable to read %s", sha1_to_hex(entry->idx.sha1));
>  			/*
>  			 * make sure no cached delta data remains from a
> -			 * previous attempt before a pack split occured.
> +			 * previous attempt before a pack split occurred.
>  			 */
>  			free(entry->delta_data);
>  			entry->delta_data = NULL;
> diff --git a/builtin-reflog.c b/builtin-reflog.c
> index 249ad2a..ff8b4f6 100644
> --- a/builtin-reflog.c
> +++ b/builtin-reflog.c
> @@ -243,7 +243,7 @@ static void mark_reachable(struct commit *commit, unsigned long expire_limit)
>  	 * We need to compute if commit on either side of an reflog

...compute whether the commit on either side of a reflog

>  	 * entry is reachable from the tip of the ref for all entries.
>  	 * Mark commits that are reachable from the tip down to the
> -	 * time threashold first; we know a commit marked thusly is
> +	 * time threshold first; we know a commit marked thusly is
>  	 * reachable from the tip without running in_merge_bases()
>  	 * at all.
>  	 */
> diff --git a/builtin-reset.c b/builtin-reset.c
> index c0cb915..7e7ebab 100644
> --- a/builtin-reset.c
> +++ b/builtin-reset.c
> @@ -228,7 +228,7 @@ int cmd_reset(int argc, const char **argv, const char *prefix)
>  		}
>  		/*
>  		 * Otherwise, argv[i] could be either <rev> or <paths> and
> -		 * has to be unambigous.
> +		 * has to be unambiguous.
>  		 */
>  		else if (!get_sha1(argv[i], sha1)) {
>  			/*
> diff --git a/compat/cygwin.c b/compat/cygwin.c
> index ebac148..1192874 100644
> --- a/compat/cygwin.c
> +++ b/compat/cygwin.c
> @@ -92,7 +92,7 @@ static int cygwin_stat(const char *path, struct stat *buf)
>   * Reading this option is not always possible immediately as git_dir may be
>   * not be set yet. So until it is set, use cygwin lstat/stat functions.

...may not be set yet.

>   * However, if core.filemode is set, we must use the Cygwin posix
> - * stat/lstat as the Windows stat fuctions do not determine posix filemode.
> + * stat/lstat as the Windows stat functions do not determine posix filemode.
>   *
>   * Note that git_cygwin_config() does NOT call git_default_config() and this
>   * is deliberate.  Many commands read from config to establish initial
> diff --git a/compat/fnmatch/fnmatch.c b/compat/fnmatch/fnmatch.c
> index 1f4ead5..03157a4 100644
> --- a/compat/fnmatch/fnmatch.c
> +++ b/compat/fnmatch/fnmatch.c
> @@ -39,7 +39,7 @@
>  # include <stdlib.h>
>  #endif
>  
> -/* For platform which support the ISO C amendement 1 functionality we
> +/* For platform which support the ISO C amendment 1 functionality we

platforms

>     support user defined character classes.  */
>  #if defined _LIBC || (defined HAVE_WCTYPE_H && defined HAVE_WCHAR_H)
>  /* Solaris 2.5 has a bug: <wchar.h> must be included before <wctype.h>.  */
> @@ -90,7 +90,7 @@
>  
>  # if defined _LIBC || (defined HAVE_WCTYPE_H && defined HAVE_WCHAR_H)
>  /* The GNU C library provides support for user-defined character classes
> -   and the functions from ISO C amendement 1.  */
> +   and the functions from ISO C amendment 1.  */
>  #  ifdef CHARCLASS_NAME_MAX
>  #   define CHAR_CLASS_MAX_LENGTH CHARCLASS_NAME_MAX
>  #  else
> diff --git a/compat/mingw.c b/compat/mingw.c
> index 2839d9d..32d4c0c 100644
> --- a/compat/mingw.c
> +++ b/compat/mingw.c
> @@ -396,7 +396,7 @@ repeat:
>  		 * its own input data to become available. But since
>  		 * the process (pack-objects) is itself CPU intensive,
>  		 * it will happily pick up the time slice that we are
> -		 * relinguishing here.
> +		 * relinquishing here.
>  		 */
>  		Sleep(0);
>  		goto repeat;
> diff --git a/compat/regex/regex.c b/compat/regex/regex.c
> index 87b33e4..5ea0075 100644
> --- a/compat/regex/regex.c
> +++ b/compat/regex/regex.c
> @@ -1043,7 +1043,7 @@ regex_compile (pattern, size, syntax, bufp)
>       they can be reliably used as array indices.  */
>    register unsigned char c, c1;
>  
> -  /* A random tempory spot in PATTERN.  */
> +  /* A random temporary spot in PATTERN.  */
>    const char *p1;
>  
>    /* Points to the end of the buffer, where we should append.  */
> @@ -1796,7 +1796,7 @@ regex_compile (pattern, size, syntax, bufp)
>  		    we're all done, the pattern will look like:
>  		      set_number_at <jump count> <upper bound>
>  		      set_number_at <succeed_n count> <lower bound>
> -		      succeed_n <after jump addr> <succed_n count>
> +		      succeed_n <after jump addr> <succeed_n count>
>  		      <body of loop>
>  		      jump_n <succeed_n addr> <jump count>
>  		    (The upper bound and `jump_n' are omitted if
> diff --git a/diffcore-rename.c b/diffcore-rename.c
> index 0b0d6b8..63ac998 100644
> --- a/diffcore-rename.c
> +++ b/diffcore-rename.c
> @@ -267,7 +267,7 @@ static int find_identical_files(struct file_similarity *src,
>  			int score;
>  			struct diff_filespec *source = p->filespec;
>  
> -			/* False hash collission? */
> +			/* False hash collision? */
>  			if (hashcmp(source->sha1, target->sha1))
>  				continue;
>  			/* Non-regular files? If so, the modes must match! */
> diff --git a/fast-import.c b/fast-import.c
> index 23c496d..53617a1 100644
> --- a/fast-import.c
> +++ b/fast-import.c
> @@ -76,7 +76,7 @@ Format of STDIN stream:
>      delim lf;
>  
>       # note: declen indicates the length of binary_data in bytes.
> -     # declen does not include the lf preceeding the binary data.
> +     # declen does not include the lf preceding the binary data.
>       #
>    exact_data ::= 'data' sp declen lf
>      binary_data;
> @@ -134,7 +134,7 @@ Format of STDIN stream:
>       #
>       # In case it is not clear, the '#' that starts the comment
>       # must be the first character on that the line (an lf have

on that line
(also, omit have)

> -     # preceeded it).
> +     # preceded it).
>       #
>    comment ::= '#' not_lf* lf;
>    not_lf  ::= # Any byte that is not ASCII newline (LF);
> @@ -953,7 +953,7 @@ static void end_packfile(void)
>  		close(pack_data->pack_fd);
>  		idx_name = keep_pack(create_index());
>  
> -		/* Register the packfile with core git's machinary. */
> +		/* Register the packfile with core git's machinery. */
>  		new_p = add_packed_git(idx_name, strlen(idx_name), 1);
>  		if (!new_p)
>  			die("core git rejected index %s", idx_name);
> diff --git a/fsck.h b/fsck.h
> index 990ee02..008456b 100644
> --- a/fsck.h
> +++ b/fsck.h
> @@ -23,7 +23,7 @@ int fsck_error_function(struct object *obj, int type, const char *fmt, ...);
>   * the return value is:
>   *    -1	error in processing the object
>   *    <0	return value of the callback, which lead to an abort
> - *    >0	return value of the first sigaled error >0 (in the case of no other errors)
> + *    >0	return value of the first signaled error >0 (in the case of no other errors)
>   *    0		everything OK
>   */
>  int fsck_walk(struct object *obj, fsck_walk_func walk, void *data);
> diff --git a/git.c b/git.c
> index bfb6508..cc5aaa7 100644
> --- a/git.c
> +++ b/git.c
> @@ -497,7 +497,7 @@ int main(int argc, const char **argv)
>  
>  	/*
>  	 * We use PATH to find git commands, but we prepend some higher
> -	 * precidence paths: the "--exec-path" option, the GIT_EXEC_PATH
> +	 * precedence paths: the "--exec-path" option, the GIT_EXEC_PATH
>  	 * environment, and the $(gitexecdir) from the Makefile at build
>  	 * time.
>  	 */
> diff --git a/graph.c b/graph.c
> index 162a516..0daf5de 100644
> --- a/graph.c
> +++ b/graph.c
> @@ -34,7 +34,7 @@ static void graph_padding_line(struct git_graph *graph, struct strbuf *sb);
>   * newline.  A new graph line will not be printed after the final newline.
>   * If the strbuf is empty, no output will be printed.
>   *
> - * Since the first line will not include the graph ouput, the caller is
> + * Since the first line will not include the graph output, the caller is
>   * responsible for printing this line's graph (perhaps via
>   * graph_show_commit() or graph_show_oneline()) before calling
>   * graph_show_strbuf().
> diff --git a/levenshtein.c b/levenshtein.c
> index a32f4cd..fc28159 100644
> --- a/levenshtein.c
> +++ b/levenshtein.c
> @@ -27,7 +27,7 @@
>   *
>   * It does so by calculating the costs of the path ending in characters
>   * i (in string1) and j (in string2), respectively, given that the last
> - * operation is a substition, a swap, a deletion, or an insertion.
> + * operation is a substitution, a swap, a deletion, or an insertion.
>   *
>   * This implementation allows the costs to be weighted:
>   *
> diff --git a/parse-options.h b/parse-options.h
> index f8ef1db..b54eec1 100644
> --- a/parse-options.h
> +++ b/parse-options.h
> @@ -52,7 +52,7 @@ typedef int parse_opt_cb(const struct option *, const char *arg, int unset);
>   *
>   * `argh`::
>   *   token to explain the kind of argument this option wants. Keep it
> - *   homogenous across the repository.
> + *   homogeneous across the repository.
>   *
>   * `help`::
>   *   the short help associated to what the option does.
> @@ -61,7 +61,7 @@ typedef int parse_opt_cb(const struct option *, const char *arg, int unset);
>   *
>   * `flags`::
>   *   mask of parse_opt_option_flags.
> - *   PARSE_OPT_OPTARG: says that the argument is optionnal (not for BOOLEANs)
> + *   PARSE_OPT_OPTARG: says that the argument is optional (not for BOOLEANs)
>   *   PARSE_OPT_NOARG: says that this option takes no argument, for CALLBACKs
>   *   PARSE_OPT_NONEG: says that this option cannot be negated
>   *   PARSE_OPT_HIDDEN this option is skipped in the default usage, showed in

shown in

> @@ -105,7 +105,7 @@ struct option {
>  	{ OPTION_CALLBACK, (s), (l), (v), (a), (h), 0, (f) }
>  
>  /* parse_options() will filter out the processed options and leave the
> - * non-option argments in argv[].
> + * non-option arguments in argv[].
>   * Returns the number of arguments left in argv[].
>   */
>  extern int parse_options(int argc, const char **argv,
> @@ -115,7 +115,7 @@ extern int parse_options(int argc, const char **argv,
>  extern NORETURN void usage_with_options(const char * const *usagestr,
>                                          const struct option *options);
>  
> -/*----- incremantal advanced APIs -----*/
> +/*----- incremental advanced APIs -----*/
>  
>  enum {
>  	PARSE_OPT_HELP = -1,
> diff --git a/revision.c b/revision.c
> index b6215cc..6007a2c 100644
> --- a/revision.c
> +++ b/revision.c
> @@ -209,7 +209,7 @@ static struct commit *handle_commit(struct rev_info *revs, struct object *object
>  	}
>  
>  	/*
> -	 * Tree object? Either mark it uniniteresting, or add it
> +	 * Tree object? Either mark it uninteresting, or add it
>  	 * to the list of objects to look at later..
>  	 */
>  	if (object->type == OBJ_TREE) {
> diff --git a/strbuf.h b/strbuf.h
> index 9ee908a..eaa8704 100644
> --- a/strbuf.h
> +++ b/strbuf.h
> @@ -11,7 +11,7 @@
>   *    build complex strings/buffers whose final size isn't easily known.
>   *
>   *    It is NOT legal to copy the ->buf pointer away.
> - *    `strbuf_detach' is the operation that detachs a buffer from its shell
> + *    `strbuf_detach' is the operation that detaches a buffer from its shell
>   *    while keeping the shell valid wrt its invariants.
>   *
>   * 2. the ->buf member is a byte array that has at least ->len + 1 bytes
> diff --git a/tree-diff.c b/tree-diff.c
> index b05d0f4..edd8394 100644
> --- a/tree-diff.c
> +++ b/tree-diff.c
> @@ -374,7 +374,7 @@ static void try_to_follow_renames(struct tree_desc *t1, struct tree_desc *t2, co
>  	}
>  
>  	/*
> -	 * Then, discard all the non-relevane file pairs...
> +	 * Then, discard all the non-relevant file pairs...
>  	 */
>  	for (i = 0; i < q->nr; i++) {
>  		struct diff_filepair *p = q->queue[i];
> diff --git a/xdiff/xdiffi.c b/xdiff/xdiffi.c
> index 02184d9..1ebab68 100644
> --- a/xdiff/xdiffi.c
> +++ b/xdiff/xdiffi.c
> @@ -456,7 +456,7 @@ int xdl_change_compact(xdfile_t *xdf, xdfile_t *xdfo, long flags) {
>  			/*
>  			 * Record the end-of-group position in case we are matched
>  			 * with a group of changes in the other file (that is, the
> -			 * change record before the enf-of-group index in the other
> +			 * change record before the end-of-group index in the other
>  			 * file is set).
>  			 */
>  			ixref = rchgo[ixo - 1] ? ix: nrec;

Re: [PATCH 3/3] Fix typos / spelling in comments

[Junio C Hamano]

Michael J Gruber <git@drmicha.warpmail.net> writes:

> Word based spell checkers can only do what they can...
> I think we should use this occasion and have humans at least look at the
> context.

Absolutely.  Care to make that into a follow-up incremental patch so that
I can squash it in when I apply this 3/3 patch?

[PATCH] Fix more typos/spelling in comments

[Michael J Gruber]

A few more fixes on top of the automatic spell checker generated ones.

Signed-off-by: Michael J Gruber <git@drmicha.warpmail.net>
---
Sorry it took me so long...

 builtin-reflog.c         |    2 +-
 compat/cygwin.c          |    2 +-
 compat/fnmatch/fnmatch.c |    2 +-
 fast-import.c            |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/builtin-reflog.c b/builtin-reflog.c
index ff8b4f6..ddfdf5a 100644
--- a/builtin-reflog.c
+++ b/builtin-reflog.c
@@ -240,7 +240,7 @@ static int unreachable(struct expire_reflog_cb *cb, struct commit *commit, unsig
 static void mark_reachable(struct commit *commit, unsigned long expire_limit)
 {
 	/*
-	 * We need to compute if commit on either side of an reflog
+	 * We need to compute whether the commit on either side of a reflog
 	 * entry is reachable from the tip of the ref for all entries.
 	 * Mark commits that are reachable from the tip down to the
 	 * time threshold first; we know a commit marked thusly is
diff --git a/compat/cygwin.c b/compat/cygwin.c
index 1192874..b4a51b9 100644
--- a/compat/cygwin.c
+++ b/compat/cygwin.c
@@ -89,7 +89,7 @@ static int cygwin_stat(const char *path, struct stat *buf)
 /*
  * At start up, we are trying to determine whether Win32 API or cygwin stat
  * functions should be used. The choice is determined by core.ignorecygwinfstricks.
- * Reading this option is not always possible immediately as git_dir may be
+ * Reading this option is not always possible immediately as git_dir may
  * not be set yet. So until it is set, use cygwin lstat/stat functions.
  * However, if core.filemode is set, we must use the Cygwin posix
  * stat/lstat as the Windows stat functions do not determine posix filemode.
diff --git a/compat/fnmatch/fnmatch.c b/compat/fnmatch/fnmatch.c
index 03157a4..14feac7 100644
--- a/compat/fnmatch/fnmatch.c
+++ b/compat/fnmatch/fnmatch.c
@@ -39,7 +39,7 @@
 # include <stdlib.h>
 #endif
 
-/* For platform which support the ISO C amendment 1 functionality we
+/* For platforms which support the ISO C amendment 1 functionality we
    support user defined character classes.  */
 #if defined _LIBC || (defined HAVE_WCTYPE_H && defined HAVE_WCHAR_H)
 /* Solaris 2.5 has a bug: <wchar.h> must be included before <wctype.h>.  */
diff --git a/fast-import.c b/fast-import.c
index 53617a1..8d959af 100644
--- a/fast-import.c
+++ b/fast-import.c
@@ -133,7 +133,7 @@ Format of STDIN stream:
      # always escapes the related input from comment processing.
      #
      # In case it is not clear, the '#' that starts the comment
-     # must be the first character on that the line (an lf have
+     # must be the first character on that line (an lf
      # preceded it).
      #
   comment ::= '#' not_lf* lf;
-- 
1.6.3.rc1.34.g0be9b

Re: [PATCH] Fix more typos/spelling in comments

[Junio C Hamano]

Thanks.

Re: [PATCH 3/3] Fix typos / spelling in comments

[Junio C Hamano]

Thanks.

Re: [PATCH 2/3] builtin-remote: fix typo in option description

[Sverre Rabbelier]

Heya,

On Fri, Apr 17, 2009 at 20:13, Mike Ralphson <mike@abacus.co.uk> wrote:
> -                           "prune remotes after fecthing"),
> +                           "prune remotes after fetching"),

For those who had a hard time finding the change (like me), it is:
s/fecthing/fetching

-- 
Cheers,

Sverre Rabbelier

Re: [PATCH 1/3] Documentation: fix typos / spelling mistakes

[Markus Heidelberg]

Mike Ralphson, 17.04.2009:
> diff --git a/gitweb/README b/gitweb/README
> -   'utf-8'. Value mist be valid encodig; see Encoding::Supported(3pm) man
> +   'utf-8'. Value mist be valid encoding; see Encoding::Supported(3pm) man

I guess this should be "must" instead of "mist". Of course the spell
checker couldn't find it, but while we are in this line...

Markus

Re: [PATCH 1/3] Documentation: fix typos / spelling mistakes

[Mike Ralphson]

2009/4/17 Markus Heidelberg <markus.heidelberg@web.de>:
> Mike Ralphson, 17.04.2009:
>> diff --git a/gitweb/README b/gitweb/README
>> -   'utf-8'. Value mist be valid encodig; see Encoding::Supported(3pm) man
>> +   'utf-8'. Value mist be valid encoding; see Encoding::Supported(3pm) man
>
> I guess this should be "must" instead of "mist". Of course the spell
> checker couldn't find it, but while we are in this line...

Annoyingly, I *did* find it and fix that in the proofreading stage the
last time I ran my check, but I foolishly decided to just bin my work
in progress and reset to HEAD and merge in the latest master. Then I
forgot that change in my hurry to get the patches out before I was
thrown out of the building by the cleaners. And that's a very painful
way to be thrown out 8-)

Thank you both for spotting it.

Mike

Re: [PATCH 1/3] Documentation: fix typos / spelling mistakes

[Michael J Gruber]

Mike Ralphson venit, vidit, dixit 17.04.2009 20:13:
> Signed-off-by: Mike Ralphson <mike@abacus.co.uk>
> ---
>  Documentation/git-cvsimport.txt         |    2 +-
>  Documentation/git-format-patch.txt      |    2 +-
>  contrib/thunderbird-patch-inline/README |    4 ++--
>  gitweb/README                           |    2 +-
>  4 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/Documentation/git-cvsimport.txt b/Documentation/git-cvsimport.txt
> index d7bab13..614e769 100644
> --- a/Documentation/git-cvsimport.txt
> +++ b/Documentation/git-cvsimport.txt
> @@ -196,7 +196,7 @@ Problems related to tags:
>  
>  If you suspect that any of these issues may apply to the repository you
>  want to import consider using these alternative tools which proved to be
> -more stable in practise:
> +more stable in practice:
>  
>  * cvs2git (part of cvs2svn), `http://cvs2svn.tigris.org`
>  * parsecvs, `http://cgit.freedesktop.org/~keithp/parsecvs`
> diff --git a/Documentation/git-format-patch.txt b/Documentation/git-format-patch.txt
> index eb2fbcf..5eddca9 100644
> --- a/Documentation/git-format-patch.txt
> +++ b/Documentation/git-format-patch.txt
> @@ -194,7 +194,7 @@ CONFIGURATION
>  -------------
>  You can specify extra mail header lines to be added to each message
>  in the repository configuration, new defaults for the subject prefix
> -and file suffix, control attachements, and number patches when outputting
> +and file suffix, control attachments, and number patches when outputting
>  more than one.
>  
>  ------------
> diff --git a/contrib/thunderbird-patch-inline/README b/contrib/thunderbird-patch-inline/README
> index 39f96aa..000147b 100644
> --- a/contrib/thunderbird-patch-inline/README
> +++ b/contrib/thunderbird-patch-inline/README
> @@ -1,12 +1,12 @@
>  appp.sh is a script that is supposed to be used together with ExternalEditor
> -for Mozilla Thundebird. It will let you include patches inline in e-mails
> +for Mozilla Thunderbird. It will let you include patches inline in e-mails
>  in an easy way.
>  
>  Usage:
>  - Generate the patch with git format-patch.
>  - Start writing a new e-mail in Thunderbird.
>  - Press the external editor button (or Ctrl-E) to run appp.sh
> -- Select the previosly generated patch file.
> +- Select the previously generated patch file.
>  - Finish editing the e-mail.
>  
>  Any text that is entered into the message editor before appp.sh is called
> diff --git a/gitweb/README b/gitweb/README
> index 8433dd1..ad73c6a 100644
> --- a/gitweb/README
> +++ b/gitweb/README
> @@ -206,7 +206,7 @@ not include variables usually directly set during build):
>   * $fallback_encoding
>     Gitweb assumes this charset if line contains non-UTF-8 characters.
>     Fallback decoding is used without error checking, so it can be even
> -   'utf-8'. Value mist be valid encodig; see Encoding::Supported(3pm) man
> +   'utf-8'. Value mist be valid encoding; see Encoding::Supported(3pm) man

That line clearly demonstrates the limits of word based spell
checkers... s/mist/must/ ;)

>     page for a list.   By default 'latin1', aka. 'iso-8859-1'.
>   * @diff_opts
>     Rename detection options for git-diff and git-diff-tree. By default