From: Tom Tromey <tromey@adacore.com>
To: gdb-patches@sourceware.org
Subject: [PATCH 0/5] Fix DAP pause bug
Date: Fri, 16 Feb 2024 11:37:48 -0700 [thread overview]
Message-ID: <20240216-py-simple-exc-v1-0-c275bcfb5e4a@adacore.com> (raw)
Tom de Vriest 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.
---
Tom Tromey (5):
Introduce new overload of read_text_file_to_string
Fix formatting buglet in python.c
kfail a FinishBreakpoint test
Rewrite "python" command exception handling
Use the .py file in gdb.dap/pause.exp
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 | 8 +-
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 +-
gdbsupport/filestuff.cc | 24 +++--
gdbsupport/filestuff.h | 4 +
35 files changed, 260 insertions(+), 285 deletions(-)
---
base-commit: d9511b64b85296f39ded68f82f7f25ee7d6b25fb
change-id: 20240216-py-simple-exc-91903ca871f5
Best regards,
--
Tom Tromey <tromey@adacore.com>
next reply other threads:[~2024-02-16 18:37 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-16 18:37 Tom Tromey [this message]
2024-02-16 18:37 ` [PATCH 1/5] Introduce new overload of read_text_file_to_string Tom Tromey
2024-02-19 15:36 ` Tom de Vries
2024-02-20 14:08 ` Tom Tromey
2024-02-20 14:48 ` Tom de Vries
2024-02-16 18:37 ` [PATCH 2/5] Fix formatting buglet in python.c Tom Tromey
2024-02-16 18:37 ` [PATCH 3/5] kfail a FinishBreakpoint test Tom Tromey
2024-02-16 18:37 ` [PATCH 4/5] Rewrite "python" command exception handling Tom Tromey
2024-02-16 18:37 ` [PATCH 5/5] Use the .py file in gdb.dap/pause.exp Tom Tromey
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240216-py-simple-exc-v1-0-c275bcfb5e4a@adacore.com \
--to=tromey@adacore.com \
--cc=gdb-patches@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).