While browsing the core source code I noticed some spelling problems, so I ran codespell and rolled a patch to fix about 150 spelling mistakes in core. Although the patch is big, it's 90% context.
All but three of the changes are in the doxygen comments or inline comments, and don't affect the code at all. The exceptions are:
1) One change affects a user-facing help string (line 805 of the patch).
2) Two changes affect a string literal in a Test case (lines 927 and 1033 of the patch).
3) One change was to a variable name in a Test case (line 1176 of the patch).
No other changes were made, however in some places fixing the spelling caused the comment line to be too long so I had to re-wrap the comments.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | D7-spelling.patch | 7.9 KB | tr |
| spelling.patch | 61.81 KB | tr |
Comments
Comment #1
tr commentedplural.
Comment #2
tr commentedComment #3
longwaveRead through the patch, all looks good to me.
Comment #4
jhodgdonWow. Thanks! I checked it over too and it all looked good to me. Committed to 8.x.
Want to do the same operation for 7.x? In that case we should avoid changing any translated text, but comment spelling fixes would be nice.
Comment #5
tr commentedThere are surprisingly few problems in D7, because many were already fixed two years ago by #1054848: Fixing common typos through codespell.py.
The attached patch fixes the 11 spelling mistakes I found in D7. None of them are in translatable strings or in user-facing text. 2 are in schema descriptions for the field_test module. 3 of them are for assert messages in image.test
Comment #6
jhodgdonThanks! Very carefully done. Committed to 7.x.
Comment #7
nod_Patch changes JS files, tagging.