diff options
| author | Tom Tromey <tom@tromey.com> | 2018-02-21 16:53:56 -0700 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2018-02-24 10:01:11 -0700 |
| commit | 281d762b1a56317171e462666b98d50bfa31a08a (patch) | |
| tree | 67e6831f0f8964dcd8ff2ae2bc123ad641b0ae87 /gdb/arch-utils.c | |
| parent | d4333bab0a097f4bfa36d2e49f4dfb12c73dce24 (diff) | |
| download | gdb-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.c | 4 |
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; } |