public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc r14-4323] libstdc++: Use Python "not in" operator
@ 2023-09-28 20:56 Tom Tromey
  0 siblings, 0 replies; only message in thread
From: Tom Tromey @ 2023-09-28 20:56 UTC (permalink / raw)
  To: gcc-cvs, libstdc++-cvs

https://gcc.gnu.org/g:202810947ca793b17653658e584008fb151e0937

commit r14-4323-g202810947ca793b17653658e584008fb151e0937
Author: Tom Tromey <tromey@adacore.com>
Date:   Wed Sep 27 09:26:01 2023 -0600

    libstdc++: Use Python "not in" operator
    
    flake8 warns about code like
    
        not something in "whatever"
    
    Ordinarily in Python this should be written as:
    
        something not in "whatever"
    
    This patch makes this change.
    
    libstdc++-v3/ChangeLog:
    
            * python/libstdcxx/v6/printers.py (Printer.add_version)
            (add_one_template_type_printer)
            (FilteringTypePrinter.add_one_type_printer): Use Python
            "not in" operator.

Diff:
---
 libstdc++-v3/python/libstdcxx/v6/printers.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/libstdc++-v3/python/libstdcxx/v6/printers.py b/libstdc++-v3/python/libstdcxx/v6/printers.py
index e530cfc0e3a..23efbd171ec 100644
--- a/libstdc++-v3/python/libstdcxx/v6/printers.py
+++ b/libstdc++-v3/python/libstdcxx/v6/printers.py
@@ -2355,7 +2355,7 @@ class Printer(object):
     # Add a name using _GLIBCXX_BEGIN_NAMESPACE_VERSION.
     def add_version(self, base, name, function):
         self.add(base + name, function)
-        if _versioned_namespace and not '__cxx11' in base:
+        if _versioned_namespace and '__cxx11' not in base:
             vbase = re.sub('^(std|__gnu_cxx)::', r'\g<0>%s' %
                            _versioned_namespace, base)
             self.add(vbase + name, function)
@@ -2527,7 +2527,7 @@ def add_one_template_type_printer(obj, name, defargs):
     printer = TemplateTypePrinter('std::__debug::' + name, defargs)
     gdb.types.register_type_printer(obj, printer)
 
-    if _versioned_namespace and not '__cxx11' in name:
+    if _versioned_namespace and '__cxx11' not in name:
         # Add second type printer for same type in versioned namespace:
         ns = 'std::' + _versioned_namespace
         # PR 86112 Cannot use dict comprehension here:
@@ -2628,7 +2628,7 @@ class FilteringTypePrinter(object):
 def add_one_type_printer(obj, template, name, targ1=None):
     printer = FilteringTypePrinter('std::' + template, 'std::' + name, targ1)
     gdb.types.register_type_printer(obj, printer)
-    if _versioned_namespace and not '__cxx11' in template:
+    if _versioned_namespace and '__cxx11' not in template:
         ns = 'std::' + _versioned_namespace
         printer = FilteringTypePrinter(ns + template, ns + name, targ1)
         gdb.types.register_type_printer(obj, printer)

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2023-09-28 20:56 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-28 20:56 [gcc r14-4323] libstdc++: Use Python "not in" operator Tom Tromey

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).