diff options
author | Eisuke Kawashima <e.kawaschima+github@gmail.com> | 2024-06-26 23:59:07 +0900 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-06-26 15:59:07 +0100 |
commit | 586114510c5fa71d1377c7f53e68a3b12c472aa2 (patch) | |
tree | 088488d9fd8516621a755de83a45cf4d6c39c5e4 /lldb/packages/Python/lldbsuite/test | |
parent | fd35a92300a00edaf56ae94176317390677569a4 (diff) | |
download | llvm-586114510c5fa71d1377c7f53e68a3b12c472aa2.zip llvm-586114510c5fa71d1377c7f53e68a3b12c472aa2.tar.gz llvm-586114510c5fa71d1377c7f53e68a3b12c472aa2.tar.bz2 |
[lldb] fix(lldb/**.py): fix comparison to None (#94017)
from PEP8
(https://peps.python.org/pep-0008/#programming-recommendations):
> Comparisons to singletons like None should always be done with is or
is not, never the equality operators.
Co-authored-by: Eisuke Kawashima <e-kwsm@users.noreply.github.com>
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test')
4 files changed, 8 insertions, 8 deletions
diff --git a/lldb/packages/Python/lldbsuite/test/dotest.py b/lldb/packages/Python/lldbsuite/test/dotest.py index 06acfb2..f14a00a 100644 --- a/lldb/packages/Python/lldbsuite/test/dotest.py +++ b/lldb/packages/Python/lldbsuite/test/dotest.py @@ -47,7 +47,7 @@ from ..support import seven def is_exe(fpath): """Returns true if fpath is an executable.""" - if fpath == None: + if fpath is None: return False if sys.platform == "win32": if not fpath.endswith(".exe"): diff --git a/lldb/packages/Python/lldbsuite/test/lldbtest.py b/lldb/packages/Python/lldbsuite/test/lldbtest.py index 1854f6c..5e50b0c 100644 --- a/lldb/packages/Python/lldbsuite/test/lldbtest.py +++ b/lldb/packages/Python/lldbsuite/test/lldbtest.py @@ -229,7 +229,7 @@ def pointer_size(): def is_exe(fpath): """Returns true if fpath is an executable.""" - if fpath == None: + if fpath is None: return False if sys.platform == "win32": if not fpath.endswith(".exe"): @@ -2191,7 +2191,7 @@ class TestBase(Base, metaclass=LLDBTestCaseFactory): if num_matches == 0: compare_string = str_input else: - if common_match != None and len(common_match) > 0: + if common_match is not None and len(common_match) > 0: compare_string = str_input + common_match else: compare_string = "" @@ -2552,7 +2552,7 @@ FileCheck output: if obj.Success(): self.fail(self._formatMessage(msg, "Error not in a fail state")) - if error_str == None: + if error_str is None: return error = obj.GetCString() diff --git a/lldb/packages/Python/lldbsuite/test/lldbutil.py b/lldb/packages/Python/lldbsuite/test/lldbutil.py index 02ec651..629565b 100644 --- a/lldb/packages/Python/lldbsuite/test/lldbutil.py +++ b/lldb/packages/Python/lldbsuite/test/lldbutil.py @@ -1594,11 +1594,11 @@ def set_actions_for_signal( ): return_obj = lldb.SBCommandReturnObject() command = "process handle {0}".format(signal_name) - if pass_action != None: + if pass_action is not None: command += " -p {0}".format(pass_action) - if stop_action != None: + if stop_action is not None: command += " -s {0}".format(stop_action) - if notify_action != None: + if notify_action is not None: command += " -n {0}".format(notify_action) testcase.dbg.GetCommandInterpreter().HandleCommand(command, return_obj) diff --git a/lldb/packages/Python/lldbsuite/test/tools/intelpt/intelpt_testcase.py b/lldb/packages/Python/lldbsuite/test/tools/intelpt/intelpt_testcase.py index fd1eb64..f1b7d7c 100644 --- a/lldb/packages/Python/lldbsuite/test/tools/intelpt/intelpt_testcase.py +++ b/lldb/packages/Python/lldbsuite/test/tools/intelpt/intelpt_testcase.py @@ -134,7 +134,7 @@ class TraceIntelPTTestCaseBase(TestBase): self.assertSBError(trace.Start(configuration), error=error) else: command = "process trace start" - if processBufferSizeLimit != None: + if processBufferSizeLimit is not None: command += " -l " + str(processBufferSizeLimit) if enableTsc: command += " --tsc" |