Re: [PATCH v4] devtools: spell check

[Thomas Monjalon <thomas@monjalon.net>]

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@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.