diff options
author | Philippe Waroquiers <philippe.waroquiers@skynet.be> | 2019-03-09 23:55:44 +0100 |
---|---|---|
committer | Philippe Waroquiers <philippe.waroquiers@skynet.be> | 2019-04-27 14:25:28 +0200 |
commit | 8a522c6cab56bd55f1454638786f999f6f636354 (patch) | |
tree | a73bb0f05e55c318a7026e0aee00c74853f410d9 /gdb/gdbcmd.h | |
parent | 136afab8c7d8a8a91dbf38e79be4f9dc4125d552 (diff) | |
download | fsf-binutils-gdb-8a522c6cab56bd55f1454638786f999f6f636354.zip fsf-binutils-gdb-8a522c6cab56bd55f1454638786f999f6f636354.tar.gz fsf-binutils-gdb-8a522c6cab56bd55f1454638786f999f6f636354.tar.bz2 |
Have 'thread|frame apply' style their output.
'thread|frame apply CMD' launches CMD so that CMD output goes to a string_file.
This patch ensures that string_file for such CMD output contains
style escape sequences that 'thread|frame apply' will later on
output on the real terminal, so as to have CMD output properly styled.
The idea is to have the class ui_file having overridable methods
to indicate that the output to this ui_file should be done using
'terminal' behaviour such as styling.
Then these methods are overriden in string_file so that a specially
constructed string_file will get output with style escape sequences.
After this patch, the output of CMD by thread|frame apply CMD is styled
similarly as when CMD is launched directly.
Note that string_file (term_out true) could also support wrapping,
but this is not done (yet?).
Tested on debian/amd64.
gdb/ChangeLog
2019-04-27 Philippe Waroquiers <philippe.waroquiers@skynet.be>
Support style in 'frame|thread apply'
* gdbcmd.h (execute_command_to_string): New term_out parameter.
* record.c (record_start, record_stop): Update callers of
execute_command_to_string with false.
* ui-file.h (class ui_file): New term_out and can_emit_style_escape
methods.
(class string_file): New constructor with term_out parameter.
Override methods term_out and can_emit_style_escape. New member
term_out.
(class stdio_file): Override can_emit_style_escape.
(class tee_file): Override term_out and can_emit_style_escape.
* utils.h (can_emit_style_escape): Remove.
* utils.c (can_emit_style_escape): Likewise.
Update all callers of can_emit_style_escape (SOMESTREAM) to
SOMESTREAM->can_emit_style_escape.
* source-cache.c (source_cache::get_source_lines): Likewise.
* stack.c (frame_apply_command_count): Call execute_command_to_string
passing the term_out characteristic of the current gdb_stdout.
* thread.c (thr_try_catch_cmd): Likewise.
* top.c (execute_command_to_string): pass term_out parameter
to construct the string_file for the command output.
* ui-file.c (term_cli_styling): New function (most code moved
from utils.c can_emit_style_escape).
(string_file::string_file, string_file::can_emit_style_escape,
stdio_file::can_emit_style_escape, tee_file::term_out,
tee_file::can_emit_style_escape): New functions.
Diffstat (limited to 'gdb/gdbcmd.h')
-rw-r--r-- | gdb/gdbcmd.h | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/gdb/gdbcmd.h b/gdb/gdbcmd.h index 4614ec7..5d0e697 100644 --- a/gdb/gdbcmd.h +++ b/gdb/gdbcmd.h @@ -133,7 +133,12 @@ extern struct cmd_list_element *showchecklist; extern struct cmd_list_element *save_cmdlist; extern void execute_command (const char *, int); -extern std::string execute_command_to_string (const char *p, int from_tty); + +/* Execute command P and returns its output. If TERM_OUT, + the output is built using terminal output behaviour such + as cli_styling. */ +extern std::string execute_command_to_string (const char *p, int from_tty, + bool term_out); extern void print_command_line (struct command_line *, unsigned int, struct ui_file *); |