public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [PATCH v4 0/4] Fix DAP pause bug
@ 2024-02-27 14:45 Tom Tromey
  2024-02-27 14:45 ` [PATCH v4 1/4] Introduce read_remainder_of_file Tom Tromey
                   ` (4 more replies)
  0 siblings, 5 replies; 11+ messages in thread
From: Tom Tromey @ 2024-02-27 14:45 UTC (permalink / raw)
  To: gdb-patches; +Cc: Tom de Vries

Tom de Vries pointed out that the DAP pause.exp test wasn't actually
testing interruption of Python.  Digging into this revealed that the
"python" and "source" commands don't handle Python exceptions in the
normal way (in gdb terms).

This series changes how these are implemented and fixes the original
problem that Tom pointed out.

Tested on x86-64 Fedora 38.

---
Changes in v4:
- Dropped the kfail patch
- Updated the documentation to reflect Python error behavior
- Link to v3: https://inbox.sourceware.org/gdb-patches/20240220-py-simple-exc-v3-0-a2825caa4b74@adacore.com

Changes in v3:
- Fix style.exp failure when Pygments is missing, pointed out by CI
- Link to v2: https://inbox.sourceware.org/gdb-patches/20240220-py-simple-exc-v2-0-c672f1984b4d@adacore.com

Changes in v2:
- Renamed new function in first patch
- Link to v1: https://inbox.sourceware.org/gdb-patches/20240216-py-simple-exc-v1-0-c275bcfb5e4a@adacore.com

---
Tom Tromey (4):
      Introduce read_remainder_of_file
      Fix formatting buglet in python.c
      Rewrite "python" command exception handling
      Use the .py file in gdb.dap/pause.exp

 gdb/doc/python.texi                               |   6 +-
 gdb/python/lib/gdb/__init__.py                    |  27 ------
 gdb/python/python.c                               | 111 +++++++++-------------
 gdb/testsuite/gdb.ada/tasks.exp                   |   8 +-
 gdb/testsuite/gdb.dap/pause.exp                   |   2 +-
 gdb/testsuite/gdb.python/py-arch.exp              |   2 +-
 gdb/testsuite/gdb.python/py-block.exp             |   4 +-
 gdb/testsuite/gdb.python/py-breakpoint.exp        |  18 ++--
 gdb/testsuite/gdb.python/py-connection.exp        |   8 +-
 gdb/testsuite/gdb.python/py-disasm.exp            |  32 +++----
 gdb/testsuite/gdb.python/py-error.exp             |   9 +-
 gdb/testsuite/gdb.python/py-finish-breakpoint.exp |   2 +-
 gdb/testsuite/gdb.python/py-format-address.exp    |  28 +++---
 gdb/testsuite/gdb.python/py-format-string.exp     |   8 +-
 gdb/testsuite/gdb.python/py-frame.exp             |  18 ++--
 gdb/testsuite/gdb.python/py-inferior.exp          |  24 ++---
 gdb/testsuite/gdb.python/py-lazy-string.exp       |   6 +-
 gdb/testsuite/gdb.python/py-mi-cmd.exp            |  40 ++++----
 gdb/testsuite/gdb.python/py-missing-debug.exp     |  16 ++--
 gdb/testsuite/gdb.python/py-objfile.exp           |   2 +-
 gdb/testsuite/gdb.python/py-parameter.exp         |  14 +--
 gdb/testsuite/gdb.python/py-pp-registration.exp   |   2 +-
 gdb/testsuite/gdb.python/py-progspace-events.exp  |   2 +-
 gdb/testsuite/gdb.python/py-progspace.exp         |   8 +-
 gdb/testsuite/gdb.python/py-record-btrace.exp     |   4 +-
 gdb/testsuite/gdb.python/py-record-full.exp       |   4 +-
 gdb/testsuite/gdb.python/py-template.exp          |   2 +-
 gdb/testsuite/gdb.python/py-thrhandle.exp         |   2 +-
 gdb/testsuite/gdb.python/py-type.exp              |  20 ++--
 gdb/testsuite/gdb.python/py-unwind.exp            |  24 ++---
 gdb/testsuite/gdb.python/py-value.exp             |  36 +++----
 gdb/testsuite/gdb.python/python.exp               |  10 +-
 gdb/testsuite/gdb.python/tui-window-names.exp     |  16 ++--
 gdb/testsuite/gdb.python/tui-window.exp           |   2 +-
 gdb/testsuite/lib/gdb-python.exp                  |   2 +-
 gdbsupport/filestuff.cc                           |  24 +++--
 gdbsupport/filestuff.h                            |   4 +
 37 files changed, 259 insertions(+), 288 deletions(-)
---
base-commit: 5998b4a287772756e33d311a570df080a4f6e051
change-id: 20240216-py-simple-exc-91903ca871f5

Best regards,
-- 
Tom Tromey <tromey@adacore.com>


^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2024-03-08 19:24 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-02-27 14:45 [PATCH v4 0/4] Fix DAP pause bug Tom Tromey
2024-02-27 14:45 ` [PATCH v4 1/4] Introduce read_remainder_of_file Tom Tromey
2024-02-27 14:45 ` [PATCH v4 2/4] Fix formatting buglet in python.c Tom Tromey
2024-02-27 14:45 ` [PATCH v4 3/4] Rewrite "python" command exception handling Tom Tromey
2024-02-27 16:03   ` Tom de Vries
2024-02-27 16:04   ` Eli Zaretskii
2024-03-07 10:00   ` Lancelot SIX
2024-03-08 16:19     ` Tom Tromey
2024-03-08 19:24       ` Tom Tromey
2024-02-27 14:45 ` [PATCH v4 4/4] Use the .py file in gdb.dap/pause.exp Tom Tromey
2024-02-27 16:56 ` [PATCH v4 0/4] Fix DAP pause bug 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).