aboutsummaryrefslogtreecommitdiff
path: root/gdb/arch-utils.c
diff options
context:
space:
mode:
authorTom Tromey <tom@tromey.com>2018-02-21 16:53:56 -0700
committerTom Tromey <tom@tromey.com>2018-02-24 10:01:11 -0700
commit281d762b1a56317171e462666b98d50bfa31a08a (patch)
tree67e6831f0f8964dcd8ff2ae2bc123ad641b0ae87 /gdb/arch-utils.c
parentd4333bab0a097f4bfa36d2e49f4dfb12c73dce24 (diff)
downloadgdb-281d762b1a56317171e462666b98d50bfa31a08a.zip
gdb-281d762b1a56317171e462666b98d50bfa31a08a.tar.gz
gdb-281d762b1a56317171e462666b98d50bfa31a08a.tar.bz2
Remove cleanups from check_fast_tracepoint_sals
This changes the gdbarch fast_tracepoint_valid_at method to use a std::string as its out parameter, and then updates all the uses. This allows removing a cleanup from breakpoint.c. Regression tested by the buildbot. ChangeLog 2018-02-24 Tom Tromey <tom@tromey.com> * i386-tdep.c (i386_fast_tracepoint_valid_at): "msg" now a std::string. * gdbarch.sh (fast_tracepoint_valid_at): Change "msg" to a std::string*. * gdbarch.c: Rebuild. * gdbarch.h: Rebuild. * breakpoint.c (check_fast_tracepoint_sals): Use std::string. * arch-utils.h (default_fast_tracepoint_valid_at): Update. * arch-utils.c (default_fast_tracepoint_valid_at): "msg" now a std::string*.
Diffstat (limited to 'gdb/arch-utils.c')
-rw-r--r--gdb/arch-utils.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/gdb/arch-utils.c b/gdb/arch-utils.c
index 693d7e3..2ff0f4d 100644
--- a/gdb/arch-utils.c
+++ b/gdb/arch-utils.c
@@ -813,12 +813,12 @@ default_has_shared_address_space (struct gdbarch *gdbarch)
int
default_fast_tracepoint_valid_at (struct gdbarch *gdbarch, CORE_ADDR addr,
- char **msg)
+ std::string *msg)
{
/* We don't know if maybe the target has some way to do fast
tracepoints that doesn't need gdbarch, so always say yes. */
if (msg)
- *msg = NULL;
+ msg->clear ();
return 1;
}