Single out flagged words and allow for more useful details, like spelling suggestions. Fixed syntax error from v1
Signed-off-by: Chandan Somani <csom...@redhat.com> --- utilities/checkpatch.py | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py index 64f0efeb4..acf9a0102 100755 --- a/utilities/checkpatch.py +++ b/utilities/checkpatch.py @@ -411,6 +411,8 @@ def check_spelling(line, comment): words = filter_comments(line, True) if comment else line words = words.replace(':', ' ').split(' ') + flagged_words = [] + for word in words: skip = False strword = re.subn(r'\W+', '', word)[0].replace(',', '') @@ -435,9 +437,13 @@ def check_spelling(line, comment): skip = True if not skip: - print_warning("Check for spelling mistakes (e.g. \"%s\")" - % strword) - return True + flagged_words.append(strword) + + if len(flagged_words) > 0: + for mistake in flagged_words: + print_warning("Possible misspelled word: \"%s\"" % mistake) + + return True return False -- 2.26.3 _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev