Fix spelling and grammar errors #1632

pull luke-jr wants to merge 3 commits into bitcoin:master from luke-jr:spelling changing 39 files +97 −97

luke-jr commented at 3:27 AM on July 26, 2012: member

Just tired of seeing a bunch of tiny commits that fix one or two spelling/grammar errors, but leave many more of the same. I went over the entire git repository with a spell checker by hand and fixed a bunch (hopefully all) of spelling errors, then checkout out each instance of "'s" to correct the grammar violations. Hopefully this will save time overall :)
laanwj commented at 4:48 AM on July 26, 2012: member

Much better indeed
Diapolo commented at 8:07 AM on July 26, 2012: none

Of course ACK to such a thing.

in src/qt/forms/rpcconsole.ui:None in 1360a6e420 outdated

 321 | @@ -322,7 +322,7 @@
 322 |         <item row="15" column="0">
 323 |          <widget class="QPushButton" name="openDebugLogfileButton">
 324 |           <property name="toolTip">
 325 | -          <string>Open the Bitcoin debug log file from the current data directory. This can take a few seconds for large log files.</string>
 326 | +          <string>Open the Bitcoin debug logfile from the current data directory. This can take a few seconds for large log files.</string>

Diapolo commented at 8:11 AM on July 26, 2012:

I don't understand this change, why logfile first and in the end log file ... I thought log file is the correct spelling?

luke-jr commented at 3:35 PM on July 26, 2012:

A mistake rebasing. Fixed.

Diapolo commented at 6:44 AM on July 27, 2012:

Thanks :).

laanwj commented at 8:43 AM on July 28, 2012: member

Oops, seems this needs rebase now, it conflicts with #1571.
jgarzik commented at 4:48 PM on August 1, 2012: contributor
sigh, weak ACK, needs rebase.

a few quibbles, nothing NAK worthy:
- I prefer "nId's" construction, as it separates the variable's precise spelling. addrinfo's etc.
- "ciphertext" is valid and common
- ditto "filename"
- "cleanup" and "whitespace" are common even if invalid
Bugfix: Fix a variety of misspellings 814efd6f1f
Bugfix: Use standard BTC unit in comments 3c726dd3c0
Bugfix: Correct English grammar regarding "'s" b49f1398a1
luke-jr commented at 5:53 PM on August 1, 2012: member
Rebased and fixed @jgarzik 's spelling qubbles. However, I left nIds and addrinfos, because:
- "nId's" construction is invalid grammar
- nIds is in fact used as a variable name already
- addrinfo is not a variable name at all (CAddrInfo is a class)
jgarzik referenced this in commit f81e6f779b on Aug 1, 2012
jgarzik merged this on Aug 1, 2012
jgarzik closed this on Aug 1, 2012
suprnurd referenced this in commit 8f850c60fa on Dec 5, 2017
DrahtBot locked this on Sep 8, 2021

Contributors