Skip to content

Commit

Permalink
checkpatch: Check for pointer whitespace.
Browse files Browse the repository at this point in the history
Signed-off-by: Joe Stringer <joe@ovn.org>
Acked-by: Russell Bryant <russell@ovn.org>
  • Loading branch information
joestringer committed Mar 9, 2017
1 parent 4868107 commit 6fc2799
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions utilities/checkpatch.py
Expand Up @@ -68,6 +68,7 @@ def print_warning(message, lineno=None):
re.compile(r'^ +(if|for|while) \(.*\)')
__regex_ends_with_bracket = \
re.compile(r'[^\s]\) {(\s+/\*[\s\Sa-zA-Z0-9\.,\?\*/+-]*)?$')
__regex_ptr_declaration_missing_whitespace = re.compile(r'[a-zA-Z0-9]\*')

skip_leading_whitespace_check = False
skip_trailing_whitespace_check = False
Expand Down Expand Up @@ -157,6 +158,12 @@ def balanced_parens(line):
return True


def pointer_whitespace_check(line):
"""Return TRUE if there is no space between a pointer name and the
asterisk that denotes this is a apionter type, ie: 'struct foo*'"""
return __regex_ptr_declaration_missing_whitespace.search(line) is not None


def ovs_checkpatch_parse(text):
global print_file_name
lineno = 0
Expand Down Expand Up @@ -257,6 +264,10 @@ def ovs_checkpatch_parse(text):
if not if_and_for_end_with_bracket_check(cmp_line):
print_line = True
print_error("Inappropriate bracing around statement", lineno)
if pointer_whitespace_check(cmp_line):
print_line = True
print_error("Inappropriate spacing in pointer declaration",
lineno)
if print_line:
print("\n%s\n" % line)
if __errors or __warnings:
Expand Down

0 comments on commit 6fc2799

Please sign in to comment.