From 4d7031437c7a81b52c776d4ae1e32741bdb851ca Mon Sep 17 00:00:00 2001 From: Dylan Baker Date: Tue, 31 Aug 2021 09:55:01 -0700 Subject: pylint: turn on superflous-parens We have a lot of these. Some of them are harmless, if unidiomatic, such as `if (condition)`, others are potentially dangerous `assert(...)`, as `assert(condtion)` works as expected, but `assert(condition, message)` will result in an assertion that never triggers, as what you're actually asserting is `bool(tuple[2])`, which will always be true. --- test cases/frameworks/1 boost/test_python_module.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'test cases/frameworks/1 boost/test_python_module.py') diff --git a/test cases/frameworks/1 boost/test_python_module.py b/test cases/frameworks/1 boost/test_python_module.py index acf6e42d6..8ef96d2ff 100644 --- a/test cases/frameworks/1 boost/test_python_module.py +++ b/test cases/frameworks/1 boost/test_python_module.py @@ -19,9 +19,9 @@ def run(): w.set(msg) - assert(msg == w.greet()) + assert msg == w.greet() version_string = str(sys.version_info[0]) + "." + str(sys.version_info[1]) - assert(version_string == w.version()) + assert version_string == w.version() if __name__ == '__main__': run() -- cgit v1.2.3