diff options
author | Laura M?dioni <laura.medioni@logilab.fr> | 2015-10-29 09:37:06 +0100 |
---|---|---|
committer | Laura M?dioni <laura.medioni@logilab.fr> | 2015-10-29 09:37:06 +0100 |
commit | cb68a81eee457947b9750013c90bcfeb4eb22bcf (patch) | |
tree | 8b8cf0396b17cdadb2a3a73f72425f6c9e498c6a /pylint/checkers/base.py | |
parent | f17f2d9546c42bbf53bb75a77734df33d0c2e182 (diff) | |
download | pylint-cb68a81eee457947b9750013c90bcfeb4eb22bcf.tar.gz |
Fix crash with 'in' operator on unneeded-not
Diffstat (limited to 'pylint/checkers/base.py')
-rw-r--r-- | pylint/checkers/base.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/pylint/checkers/base.py b/pylint/checkers/base.py index df76c60..aba648c 100644 --- a/pylint/checkers/base.py +++ b/pylint/checkers/base.py @@ -1856,7 +1856,7 @@ class NotChecker(_BasicChecker): } reverse_op = {'<': '>=', '<=': '>', '>': '<=', '>=': '<', '==': '!=', - '!=': '=='} + '!=': '==', 'in': 'not in'} @check_messages('unneeded-not') def visit_unaryop(self, node): @@ -1870,6 +1870,8 @@ class NotChecker(_BasicChecker): elif isinstance(operand, astroid.Compare): left = operand.left operator, right = operand.ops[0] + if operator not in self.reverse_op: + return suggestion = '%s %s %s' % (left.as_string(), self.reverse_op[operator], right.as_string()) |