From b6484282f85bf7f11451b2441599c241d302ad9d Mon Sep 17 00:00:00 2001 From: Raul Tambre Date: Sat, 4 May 2019 15:48:17 -0400 Subject: [PATCH] Fix incorrect use of 'is' operator for comparison in python/lib/gdb/command/prompt.py The 'is' operator is not meant to be used for comparisons. It currently working is an implementation detail of CPython. CPython 3.8 has added a SyntaxWarning for this. --- gdb/ChangeLog | 6 ++++++ gdb/python/lib/gdb/command/prompt.py | 4 ++-- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 383b3533390..7eb5d9aef31 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2019-05-04 Raul Tambre + + * python/lib/gdb/prompt.py (_ExtendedPrompt) + : Fix incorrect use of 'is' + operator for comparison. + 2019-05-04 Tom Tromey * psymtab.c (psymbol_name_matches, match_partial_symbol) diff --git a/gdb/python/lib/gdb/command/prompt.py b/gdb/python/lib/gdb/command/prompt.py index 3d662a7d3f1..04b9e49c229 100644 --- a/gdb/python/lib/gdb/command/prompt.py +++ b/gdb/python/lib/gdb/command/prompt.py @@ -45,7 +45,7 @@ The currently defined substitutions are: self.hook_set = False def get_show_string (self, pvalue): - if self.value is not '': + if self.value: return "The extended prompt is: " + self.value else: return "The extended prompt is not set." @@ -57,7 +57,7 @@ The currently defined substitutions are: return "" def before_prompt_hook(self, current): - if self.value is not '': + if self.value: return gdb.prompt.substitute_prompt(self.value) else: return None -- 2.30.2