01/12/2021 17:59, Henry Nadeau: > A spell check script to check for errors in patches. An > example of usage being cat PATCH_FILE | spell_check.sh.
Does it mean all words are checked? Code and comments? I think there is no need to check the code. Note we are already using another tool, codespell inside checkpatch, based on a dictionary built with this script: devtools/build-dict.sh Do you know how aspell compares with codespell? > Errors will be > printed to console, following the file they are located in. Word exclusions > can be made by making additions to the dictionary, and problematic patterns > can be added to the regex filter. > > Signed-off-by: Henry Nadeau <hnad...@iol.unh.edu> > --- > devtools/spell_check.sh | 121 ++ > devtools/spell_check_dictionary.txt | 2160 +++++++++++++++++++++++++++ > devtools/spell_check_regex.txt | 2 + > 3 files changed, 2283 insertions(+) I think it is too much effort to maintain the dictionary and exclusions. I understand it avoids false positive, but I don't imagine contributors updating the dictionary as part of their patch.
