diff options
| author | Simon Marchi <simon.marchi@polymtl.ca> | 2018-08-07 21:37:40 -0400 |
|---|---|---|
| committer | Simon Marchi <simon.marchi@polymtl.ca> | 2018-08-07 21:38:27 -0400 |
| commit | 09ce46f230fee766c386384cd1f5672d12fde276 (patch) | |
| tree | 70c4f15a8d7f68622137032f6250cc971119b1dd /gdb/target-debug.h | |
| parent | b825f3a90e33ab6f333d12d676d51419f8fed77b (diff) | |
| download | gdb-09ce46f230fee766c386384cd1f5672d12fde276.zip gdb-09ce46f230fee766c386384cd1f5672d12fde276.tar.gz gdb-09ce46f230fee766c386384cd1f5672d12fde276.tar.bz2 | |
Make target_options_to_string return an std::string
Return an std::string instead of a char *, saving some manual freeing.
I only manually tested with "set debug target 1" and "set debug lin-lwp
1", since this only deals with debug output.
gdb/ChangeLog:
* target.h (target_options_to_string): Return an std::string.
* target.c (str_comma_list_concat_elem): Return void, use
std::string.
(do_option): Likewise.
(target_options_to_string): Return an std::string.
* linux-nat.c (linux_nat_target::wait): Adjust.
* target-debug.h (target_debug_print_options): Adjust.
Diffstat (limited to 'gdb/target-debug.h')
| -rw-r--r-- | gdb/target-debug.h | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/gdb/target-debug.h b/gdb/target-debug.h index 331baf5..1e904b9 100644 --- a/gdb/target-debug.h +++ b/gdb/target-debug.h @@ -203,10 +203,9 @@ target_debug_print_struct_target_waitstatus_p (struct target_waitstatus *status) static void target_debug_print_options (int options) { - char *str = target_options_to_string (options); + std::string str = target_options_to_string (options); - fputs_unfiltered (str, gdb_stdlog); - xfree (str); + fputs_unfiltered (str.c_str (), gdb_stdlog); } static void |