From 828a3fda1f77a44910c6940b6cab9c34c2807766 Mon Sep 17 00:00:00 2001 From: Sebastian Date: Wed, 6 Nov 2019 17:49:37 +0100 Subject: [PATCH] pylintrc_travis: Enable more checks, fix anomalous-backslash-in-string issues (#2334) --- addons/misra.py | 4 ++-- pylintrc_travis | 7 +++++++ tools/extracttests.py | 2 +- 3 files changed, 10 insertions(+), 3 deletions(-) diff --git a/addons/misra.py b/addons/misra.py index 535c82e70..cd3a350aa 100755 --- a/addons/misra.py +++ b/addons/misra.py @@ -484,7 +484,7 @@ def isHexEscapeSequence(symbols): def isOctalEscapeSequence(symbols): - """Checks that given symbols are valid octal escape sequence: + r"""Checks that given symbols are valid octal escape sequence: octal-escape-sequence: \ octal-digit @@ -2699,7 +2699,7 @@ def main(): rules_violated[misra_id] = rules_violated.get(misra_id, 0) + 1 print("MISRA rules violated:") convert = lambda text: int(text) if text.isdigit() else text - misra_sort = lambda key: [ convert(c) for c in re.split('[\.-]([0-9]*)', key) ] + misra_sort = lambda key: [ convert(c) for c in re.split('[.-]([0-9]*)', key) ] for misra_id in sorted(rules_violated.keys(), key=misra_sort): res = re.match(r'misra-c2012-([0-9]+)\\.([0-9]+)', misra_id) if res is None: diff --git a/pylintrc_travis b/pylintrc_travis index c6a82f52f..6348293a4 100644 --- a/pylintrc_travis +++ b/pylintrc_travis @@ -5,6 +5,13 @@ enable=mixed-indentation, print-statement, literal-comparison, unnecessary-semicolon, + mixed-line-endings, + bad-open-mode, + redundant-unittest-assert, + boolean-datetime, + deprecated-method, + anomalous-unicode-escape-in-string, + anomalous-backslash-in-string, #bad-indentation [REPORTS] reports=no diff --git a/tools/extracttests.py b/tools/extracttests.py index b6152e7a2..401ca59eb 100755 --- a/tools/extracttests.py +++ b/tools/extracttests.py @@ -72,7 +72,7 @@ class Extract: continue # check - res = re.match('\s+check.*\(' + string, line) + res = re.match('\\s+check.*\\(' + string, line) if res is not None: code = res.group(1)