These coding issues are not just things that shouldn't be done. They are
styles which should never be submitted.
Signed-off-by: Aaron Conole <acon...@bytheb.org>
---
utilities/checkpatch.py | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/utilities/checkpatch.py b/utilities/checkpatch.py
index 5fbac9a..721e4d6 100755
--- a/utilities/checkpatch.py
+++ b/utilities/checkpatch.py
@@ -219,14 +219,13 @@ def ovs_checkpatch_parse(text):
lineno)
if not if_and_for_whitespace_checks(line[1:]):
print_line = True
- print_warning("Improper whitespace around control block",
- lineno)
+ print_error("Improper whitespace around control block",
+ lineno)
if not if_and_for_end_with_bracket_check(line[1:]):
print_line = True
- print_warning("Inappropriate bracing around statement",
- lineno)
+ print_error("Inappropriate bracing around statement", lineno)
if print_line:
- print(line)
+ print("\n%s\n" % line)
if __errors or __warnings:
return -1
return 0
--
2.7.4
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
