[PATCH] Fix overridable written with an extra 'e'

Subsystems: documentation, the rest

STALE3732d

3 messages, 3 authors, 2016-06-15 · open the first message on its own page

[PATCH] Fix overridable written with an extra 'e'

From: Nanako Shiraishi <hidden>
Date: 2016-06-15 22:47:19

Signed-off-by: Nanako Shiraishi <redacted>
---
Found during the lunch break by one of my students...
diff --git a/Documentation/git-gc.txt b/Documentation/git-gc.txt
index b292e98..dcac8c8 100644
--- a/Documentation/git-gc.txt
+++ b/Documentation/git-gc.txt
@@ -61,7 +61,7 @@ automatic consolidation of packs.
 
 --prune=<date>::
 	Prune loose objects older than date (default is 2 weeks ago,
-	overrideable by the config variable `gc.pruneExpire`).  This
+	overridable by the config variable `gc.pruneExpire`).  This
 	option is on by default.
 
 --no-prune::
diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index 33ef190..43fa791 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -376,7 +376,7 @@ sub gitweb_get_feature {
 		@{$feature{$name}{'default'}});
 	if (!$override) { return @defaults; }
 	if (!defined $sub) {
-		warn "feature $name is not overrideable";
+		warn "feature $name is not overridable";
 		return @defaults;
 	}
 	return $sub->(@defaults);
-- 
Nanako Shiraishi
http://ivory.ap.teacup.com/nanako3/

Re: [PATCH] Fix overridable written with an extra 'e'

From: Todd Zullinger <hidden>
Date: 2016-06-15 22:47:19

Nanako Shiraishi wrote:
Found during the lunch break by one of my students...
Is overridable a word itself?  While English is my native language, I
wouldn't call myself an expert on its proper usage. ;)

However, I can't find 'overridable' in several online dictionaries:

http://dictionary.reference.com/browse/overridable
http://www.merriam-webster.com/dictionary/overridable
http://www.google.com/dictionary?aq=f&langpair=en|en&q=overridable&hl=en
http://dictionary.cambridge.org/results.asp?searchword=overridable&x=0&y=0

Perhaps using overridden would be more accurate?
 	Prune loose objects older than date (default is 2 weeks ago,
-	overrideable by the config variable `gc.pruneExpire`).  This
+	overridable by the config variable `gc.pruneExpire`).  This
 	option is on by default.
 	Prune loose objects older than date (default is 2 weeks ago,
-	overrideable by the config variable `gc.pruneExpire`).  This
+	which can be overridden by the config variable `gc.pruneExpire`).  This
 	option is on by default.
-		warn "feature $name is not overrideable";
+		warn "feature $name is not overridable";
and

-		warn "feature $name is not overrideable";
+		warn "feature $name cannot be overridden";

?

-- 
Todd        OpenPGP -> KeyID: 0xBEAF0CE3 | URL: www.pobox.com/~tmz/pgp
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
I got stopped by a cop the other day.  He said, "Why'd you run that
stop sign?"  I said, "Because I don't believe everything I read."
    -- Stephen Wright

Re: [PATCH] Fix overridable written with an extra 'e'

From: Uri Okrent <hidden>
Date: 2016-06-15 22:47:20

Todd Zullinger <tmz <at> pobox.com> writes:
Nanako Shiraishi wrote:
quoted
Found during the lunch break by one of my students...
Is overridable a word itself?  While English is my native language, I
wouldn't call myself an expert on its proper usage. ;)
No, it's not a word =). But of the two non-words, overridable is probably
preferred.
However, I can't find 'overridable' in several online dictionaries:

http://dictionary.reference.com/browse/overridable
http://www.merriam-webster.com/dictionary/overridable
http://www.google.com/dictionary?aq=f&langpair=en|en&q=overridable&hl=en
http://dictionary.cambridge.org/results.asp?searchword=overridable&x=0&y=0

Perhaps using overridden would be more accurate?
It would be more correct, but also slightly more verbose. Still, as a grammar
nazi, my two cents says it raises git's esteem if the language in the docs is
proper English.

Uri
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help