mbox series

[0/9] Make gdb/python flake-clean

Message ID 20240319-more-flake8-v1-0-893549dbed75@adacore.com
Headers
Series Make gdb/python flake-clean |

Message

Tom Tromey March 19, 2024, 5:33 p.m. UTC 
  This series changes gdb/python to be flake8-clean.

Regression tested on x86-64 Fedora 38.

---
Tom Tromey (9):
      Remove .flake8
      Fix flake8 errors in dap/server.py
      Ignore unsed import in dap/__init__.py
      Remove unused import from gdb/__init__.py
      Remove bare "except" from disassembler.py
      Suppress star import errors
      Specify ImportError in styling.py
      Suppress some "undefined" warnings from flake8
      Do not use bare "except"

 gdb/.flake8                        |  2 --
 gdb/python/lib/gdb/__init__.py     | 21 ++++++++++-----------
 gdb/python/lib/gdb/dap/__init__.py | 32 +++++++++++++++++---------------
 gdb/python/lib/gdb/dap/server.py   |  3 ++-
 gdb/python/lib/gdb/disassembler.py | 26 ++++++++++----------------
 gdb/python/lib/gdb/styling.py      |  8 ++++----
 gdb/setup.cfg                      |  3 ++-
 7 files changed, 45 insertions(+), 50 deletions(-)
---
base-commit: 12d5d5bfd0201711ac3b14d8cd92589919a82b7a
change-id: 20240319-more-flake8-b714194c9a35

Best regards,

Comments

Simon Marchi March 20, 2024, 7:01 p.m. UTC | #1 
On 3/19/24 13:33, Tom Tromey wrote:
> This series changes gdb/python to be flake8-clean.
> 
> Regression tested on x86-64 Fedora 38.

I sent one request on patch 8, but other than that:

Approved-By: Simon Marchi <simon.marchi@efficios.com>

Simon
Tom Tromey April 2, 2024, 5:01 p.m. UTC | #2 
>>>>> "Tom" == Tom Tromey <tromey@adacore.com> writes:

Tom> This series changes gdb/python to be flake8-clean.
Tom> Regression tested on x86-64 Fedora 38.

I'm going to check in the updated version of this series.

Tom