public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] Remove exception_catchpoint::resources_needed
@ 2023-03-06 15:13 Tom Tromey
  0 siblings, 0 replies; only message in thread
From: Tom Tromey @ 2023-03-06 15:13 UTC (permalink / raw)
  To: gdb-cvs

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=c52e4861c61926a1e22e56837487b819e2dc3cc8

commit c52e4861c61926a1e22e56837487b819e2dc3cc8
Author: Tom Tromey <tromey@adacore.com>
Date:   Thu Feb 16 10:25:23 2023 -0700

    Remove exception_catchpoint::resources_needed
    
    exception_catchpoint::resources_needed has a FIXME comment that I
    think makes this method obsolete.  Also, I note that similar
    catchpoints, for example Ada catchpoints, don't have this method.
    This patch removes the method.  Regression tested on x86-64 Fedora 36.

Diff:
---
 gdb/break-catch-throw.c | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/gdb/break-catch-throw.c b/gdb/break-catch-throw.c
index ab7899e2b9c..457446efbc6 100644
--- a/gdb/break-catch-throw.c
+++ b/gdb/break-catch-throw.c
@@ -94,13 +94,6 @@ struct exception_catchpoint : public code_breakpoint
   void check_status (struct bpstat *bs) override;
   struct bp_location *allocate_location () override;
 
-  /* FIXME this is temporary - until ordinary breakpoints have been
-     converted.  */
-  int resources_needed (const struct bp_location *) override
-  {
-    return 1;
-  }
-
   /* The kind of exception catchpoint.  */
 
   enum exception_event_kind kind;

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

only message in thread, other threads:[~2023-03-06 15:13 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-06 15:13 [binutils-gdb] Remove exception_catchpoint::resources_needed 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).