While in general correct spelling in comments is a laudable goal, fixing spelling has to be weighted against other factors that may speak against touching the code in question at all: The code in question may be imported from a third party and eventually be rebased to a newer upstream version; or, the code may be actively worked on by another developer. In both cases, the spelling fixes create more work (rebasing) for very little benefit, which can be quite annoying. (The same can be said of whitespace fixes or code cleanups.)
Of the files touched by this patch, Doom, GNU chess, and some of the codec libraries belong to the first category; and I think at least elftosb belongs to the second.
Thus, I suggest to not blindly accept this patch, but to use pieces of it at an opportune time.
PS: Also, I don’t regard “thru” as a spelling error, especially not in informal code comments.
0001-trivial-fix-various-typo...
While in general correct spelling in comments is a laudable goal, fixing spelling has to be weighted against other factors that may speak against touching the code in question at all: The code in question may be imported from a third party and eventually be rebased to a newer upstream version; or, the code may be actively worked on by another developer. In both cases, the spelling fixes create more work (rebasing) for very little benefit, which can be quite annoying. (The same can be said of whitespace fixes or code cleanups.)
Of the files touched by this patch, Doom, GNU chess, and some of the codec libraries belong to the first category; and I think at least elftosb belongs to the second.
Thus, I suggest to not blindly accept this patch, but to use pieces of it at an opportune time.
PS: Also, I don’t regard “thru” as a spelling error, especially not in informal code comments.