From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2126) id 79889394881C; Fri, 6 May 2022 18:11:27 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 79889394881C Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Tom Tromey To: gdb-cvs@sourceware.org Subject: [binutils-gdb] Introduce catchpoint class X-Act-Checkin: binutils-gdb X-Git-Author: Tom Tromey X-Git-Refname: refs/heads/master X-Git-Oldrev: b68f26dea78f1a390084a0e2bfa95f1bcf1942dc X-Git-Newrev: fed1c982dec3baba167b3fd4df3538f09fa3d339 Message-Id: <20220506181127.79889394881C@sourceware.org> Date: Fri, 6 May 2022 18:11:27 +0000 (GMT) X-BeenThere: gdb-cvs@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 06 May 2022 18:11:27 -0000 https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git;h=3Dfed1c982dec3= baba167b3fd4df3538f09fa3d339 commit fed1c982dec3baba167b3fd4df3538f09fa3d339 Author: Tom Tromey Date: Sun May 1 16:11:26 2022 -0600 Introduce catchpoint class =20 This introduces a catchpoint class that is used as the base class for all catchpoints. init_catchpoint is rewritten to be a constructor instead. =20 This changes the hierarchy a little -- some catchpoints now inherit from base_breakpoint whereas previously they did not. This isn't a problem, as long as re_set is redefined in catchpoint. Diff: --- gdb/break-catch-exec.c | 10 +++++----- gdb/break-catch-fork.c | 13 ++++++------- gdb/break-catch-load.c | 13 +++++++------ gdb/break-catch-sig.c | 10 +++++----- gdb/break-catch-syscall.c | 1 - gdb/break-catch-throw.c | 10 +++++----- gdb/breakpoint.c | 19 +++++++------------ gdb/breakpoint.h | 22 ++++++++++++++-------- 8 files changed, 49 insertions(+), 49 deletions(-) diff --git a/gdb/break-catch-exec.c b/gdb/break-catch-exec.c index c828b50cc62..07417ee499c 100644 --- a/gdb/break-catch-exec.c +++ b/gdb/break-catch-exec.c @@ -34,10 +34,10 @@ A breakpoint is really of this type iff its ops pointer points to CATCH_EXEC_BREAKPOINT_OPS. */ =20 -struct exec_catchpoint : public breakpoint +struct exec_catchpoint : public catchpoint { - explicit exec_catchpoint (struct gdbarch *gdbarch) - : breakpoint (gdbarch, bp_catchpoint) + exec_catchpoint (struct gdbarch *gdbarch, bool temp, const char *cond_st= ring) + : catchpoint (gdbarch, temp, cond_string) { } =20 @@ -208,8 +208,8 @@ catch_exec_command_1 (const char *arg, int from_tty, if ((*arg !=3D '\0') && !isspace (*arg)) error (_("Junk at end of arguments.")); =20 - std::unique_ptr c (new exec_catchpoint (gdbarch)); - init_catchpoint (c.get (), gdbarch, temp, cond_string); + std::unique_ptr c + (new exec_catchpoint (gdbarch, temp, cond_string)); =20 install_breakpoint (0, std::move (c), 1); } diff --git a/gdb/break-catch-fork.c b/gdb/break-catch-fork.c index 913ccecba35..1f8deec6a62 100644 --- a/gdb/break-catch-fork.c +++ b/gdb/break-catch-fork.c @@ -32,10 +32,11 @@ catchpoint. A breakpoint is really of this type iff its ops pointer po= ints to CATCH_FORK_BREAKPOINT_OPS. */ =20 -struct fork_catchpoint : public breakpoint +struct fork_catchpoint : public catchpoint { - fork_catchpoint (struct gdbarch *gdbarch, bool is_vfork_) - : breakpoint (gdbarch, bp_catchpoint), + fork_catchpoint (struct gdbarch *gdbarch, bool temp, + const char *cond_string, bool is_vfork_) + : catchpoint (gdbarch, temp, cond_string), is_vfork (is_vfork_) { } @@ -186,10 +187,8 @@ create_fork_vfork_event_catchpoint (struct gdbarch *gd= barch, bool temp, const char *cond_string, bool is_vfork) { - std::unique_ptr c (new fork_catchpoint (gdbarch, - is_vfork)); - - init_catchpoint (c.get (), gdbarch, temp, cond_string); + std::unique_ptr c + (new fork_catchpoint (gdbarch, temp, cond_string, is_vfork)); =20 install_breakpoint (0, std::move (c), 1); } diff --git a/gdb/break-catch-load.c b/gdb/break-catch-load.c index 8579f4e3b23..617ee2b694d 100644 --- a/gdb/break-catch-load.c +++ b/gdb/break-catch-load.c @@ -33,10 +33,12 @@ A breakpoint is really of this type iff its ops pointer points to CATCH_SOLIB_BREAKPOINT_OPS. */ =20 -struct solib_catchpoint : public breakpoint +struct solib_catchpoint : public catchpoint { - solib_catchpoint (struct gdbarch *gdbarch, bool is_load_, const char *ar= g) - : breakpoint (gdbarch, bp_catchpoint), + solib_catchpoint (struct gdbarch *gdbarch, bool temp, + const char *cond_string, + bool is_load_, const char *arg) + : catchpoint (gdbarch, temp, cond_string), is_load (is_load_), regex (arg =3D=3D nullptr ? nullptr : make_unique_xstrdup (arg)), compiled (arg =3D=3D nullptr @@ -229,11 +231,10 @@ add_solib_catchpoint (const char *arg, bool is_load, = bool is_temp, bool enabled) if (*arg =3D=3D '\0') arg =3D nullptr; =20 - std::unique_ptr c (new solib_catchpoint (gdbarch, + std::unique_ptr c (new solib_catchpoint (gdbarch, is_t= emp, + nullptr, is_load, arg)); =20 - init_catchpoint (c.get (), gdbarch, is_temp, NULL); - c->enable_state =3D enabled ? bp_enabled : bp_disabled; =20 install_breakpoint (0, std::move (c), 1); diff --git a/gdb/break-catch-sig.c b/gdb/break-catch-sig.c index 4430dd0b2b8..57a6255dc33 100644 --- a/gdb/break-catch-sig.c +++ b/gdb/break-catch-sig.c @@ -38,11 +38,12 @@ /* An instance of this type is used to represent a signal catchpoint. */ =20 -struct signal_catchpoint : public breakpoint +struct signal_catchpoint : public catchpoint { - signal_catchpoint (struct gdbarch *gdbarch, std::vector &&si= gs, + signal_catchpoint (struct gdbarch *gdbarch, bool temp, + std::vector &&sigs, bool catch_all_) - : breakpoint (gdbarch, bp_catchpoint), + : catchpoint (gdbarch, temp, nullptr), signals_to_be_caught (std::move (sigs)), catch_all (catch_all_) { @@ -325,8 +326,7 @@ create_signal_catchpoint (int tempflag, std::vector &&filter, struct gdbarch *gdbarch =3D get_current_arch (); =20 std::unique_ptr c - (new signal_catchpoint (gdbarch, std::move (filter), catch_all)); - init_catchpoint (c.get (), gdbarch, tempflag, nullptr); + (new signal_catchpoint (gdbarch, tempflag, std::move (filter), catch_a= ll)); =20 install_breakpoint (0, std::move (c), 1); } diff --git a/gdb/break-catch-syscall.c b/gdb/break-catch-syscall.c index 5ed15ac8bf9..d79ced47a12 100644 --- a/gdb/break-catch-syscall.c +++ b/gdb/break-catch-syscall.c @@ -355,7 +355,6 @@ create_syscall_event_catchpoint (int tempflag, std::vec= tor &&filter) =20 std::unique_ptr c (new syscall_catchpoint (gdbarch, std::move (filter))); - init_catchpoint (c.get (), gdbarch, tempflag, nullptr); =20 install_breakpoint (0, std::move (c), 1); } diff --git a/gdb/break-catch-throw.c b/gdb/break-catch-throw.c index 0237af7b2a7..f15fa531519 100644 --- a/gdb/break-catch-throw.c +++ b/gdb/break-catch-throw.c @@ -65,12 +65,13 @@ static const struct exception_names exception_functions= [] =3D =20 /* The type of an exception catchpoint. */ =20 -struct exception_catchpoint : public base_breakpoint +struct exception_catchpoint : public catchpoint { exception_catchpoint (struct gdbarch *gdbarch, + bool temp, const char *cond_string, enum exception_event_kind kind_, std::string &&except_rx) - : base_breakpoint (gdbarch, bp_catchpoint), + : catchpoint (gdbarch, temp, cond_string), kind (kind_), exception_rx (std::move (except_rx)), pattern (exception_rx.empty () @@ -371,9 +372,8 @@ handle_gnu_v3_exceptions (int tempflag, std::string &&e= xcept_rx, struct gdbarch *gdbarch =3D get_current_arch (); =20 std::unique_ptr cp - (new exception_catchpoint (gdbarch, ex_event, std::move (except_rx))); - - init_catchpoint (cp.get (), gdbarch, tempflag, cond_string); + (new exception_catchpoint (gdbarch, tempflag, cond_string, + ex_event, std::move (except_rx))); =20 cp->re_set (); =20 diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c index 4c7542a52c8..9abc1443d96 100644 --- a/gdb/breakpoint.c +++ b/gdb/breakpoint.c @@ -7804,23 +7804,18 @@ disable_breakpoints_in_freed_objfile (struct objfil= e *objfile) =20 /* See breakpoint.h. */ =20 -void -init_catchpoint (struct breakpoint *b, - struct gdbarch *gdbarch, bool temp, - const char *cond_string) +catchpoint::catchpoint (struct gdbarch *gdbarch, bool temp, + const char *cond_string_) + : base_breakpoint (gdbarch, bp_catchpoint) { symtab_and_line sal; sal.pspace =3D current_program_space; =20 - /* This should already have been set in the constructor. */ - gdb_assert (b->type =3D=3D bp_catchpoint); - init_raw_breakpoint (b, sal, bp_catchpoint); + init_raw_breakpoint (this, sal, bp_catchpoint); =20 - if (cond_string =3D=3D nullptr) - b->cond_string.reset (); - else - b->cond_string =3D make_unique_xstrdup (cond_string); - b->disposition =3D temp ? disp_del : disp_donttouch; + if (cond_string_ !=3D nullptr) + cond_string =3D make_unique_xstrdup (cond_string_); + disposition =3D temp ? disp_del : disp_donttouch; } =20 void diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h index d244e8daec0..ac738fd7c2d 100644 --- a/gdb/breakpoint.h +++ b/gdb/breakpoint.h @@ -1012,6 +1012,20 @@ struct tracepoint : public breakpoint int static_trace_marker_id_idx =3D 0; }; =20 +/* The base class for catchpoints. */ + +struct catchpoint : public base_breakpoint +{ + /* If TEMP is true, then make the breakpoint temporary. If + COND_STRING is not NULL, then store it in the breakpoint. */ + catchpoint (struct gdbarch *gdbarch, bool temp, const char *cond_string); + + void re_set () override + { + /* For catchpoints, the default is to do nothing. */ + } +}; + =0C /* The following stuff is an abstract data type "bpstat" ("breakpoint status"). This provides the ability to determine whether we have @@ -1440,14 +1454,6 @@ extern void int enabled, int from_tty); =20 -/* Initialize a new breakpoint of the bp_catchpoint kind. If TEMP - is true, then make the breakpoint temporary. If COND_STRING is - not NULL, then store it in the breakpoint. */ - -extern void init_catchpoint (struct breakpoint *b, - struct gdbarch *gdbarch, bool temp, - const char *cond_string); - /* Add breakpoint B on the breakpoint list, and notify the user, the target and breakpoint_created observers of its existence. If INTERNAL is non-zero, the breakpoint number will be allocated from