[PATCH v4] devtools: spell check
Thomas Monjalon
thomas at monjalon.net
Thu Jan 20 16:10:31 CET 2022
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 <hnadeau at 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.
More information about the dev
mailing list