diff options
| author | Tom Tromey <tom@tromey.com> | 2019-07-18 22:07:10 -0600 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2019-08-06 11:37:51 -0600 |
| commit | 046bebe1c0a367687bd693eb4594fcdcba0b8f7e (patch) | |
| tree | 38fdfa265e21d04c7bed98491f3bb7b4a8185ad8 /gdb/testsuite/gdb.base | |
| parent | ddbcedf5de138bc83d5d16554a3f44163dcaef7a (diff) | |
| download | gdb-046bebe1c0a367687bd693eb4594fcdcba0b8f7e.zip gdb-046bebe1c0a367687bd693eb4594fcdcba0b8f7e.tar.gz gdb-046bebe1c0a367687bd693eb4594fcdcba0b8f7e.tar.bz2 | |
Add more styling to "disassemble"
This adds more styling to the disassemble command. In particular,
addresses and function names in the disassembly are now styled.
This required fixing a small latent bug in set_output_style. This
function always passed NULL to emit_style_escape; but when writing to
a file other than gdb_stdout, it should emit the style escape
directly. (FWIW this is another argument for better integrating the
pager with ui_file and getting rid of this entire layer.)
gdb/ChangeLog
2019-08-06 Tom Tromey <tom@tromey.com>
* utils.c (set_output_style): Sometimes pass stream to
emit_style_escape.
* ui-out.h (class ui_out) <can_emit_style_escape>: Declare.
* record-btrace.c (btrace_insn_history): Update.
* mi/mi-out.h (class mi_ui_out) <can_emit_style_escape>: New
method.
* disasm.h (gdb_pretty_print_disassembler): Add uiout parameter.
Update initializers.
<m_uiout>: New field.
<m_di>: Move lower.
* disasm.c (gdb_pretty_print_disassembler::pretty_print_insn):
Remove "uiout" parameter.
(dump_insns): Update.
* cli-out.h (class cli_ui_out) <can_emit_style_escape>: Declare.
* cli-out.c (cli_ui_out::can_emit_style_escape): New method.
gdb/testsuite/ChangeLog
2019-08-06 Tom Tromey <tom@tromey.com>
* gdb.base/style.exp: Add disassemble test.
* gdb.base/style.c (some_called_function): New function.
(main): Use it.
Diffstat (limited to 'gdb/testsuite/gdb.base')
| -rw-r--r-- | gdb/testsuite/gdb.base/style.c | 7 | ||||
| -rw-r--r-- | gdb/testsuite/gdb.base/style.exp | 4 |
2 files changed, 10 insertions, 1 deletions
diff --git a/gdb/testsuite/gdb.base/style.c b/gdb/testsuite/gdb.base/style.c index a44936e..82dca7d 100644 --- a/gdb/testsuite/gdb.base/style.c +++ b/gdb/testsuite/gdb.base/style.c @@ -15,8 +15,13 @@ #define SOME_MACRO 23 +int some_called_function (void) +{ + return 0; +} + int main (int argc, char **argv) { - return 0; /* break here */ + return some_called_function (); /* break here */ } diff --git a/gdb/testsuite/gdb.base/style.exp b/gdb/testsuite/gdb.base/style.exp index a17f201..842c52e 100644 --- a/gdb/testsuite/gdb.base/style.exp +++ b/gdb/testsuite/gdb.base/style.exp @@ -83,6 +83,10 @@ save_vars { env(TERM) } { "Defined at $base_file_expr:16\r\n#define SOME_MACRO 23" } + set func [style some_called_function function] + # Somewhere should see the call to the function. + gdb_test "disassemble main" "[style $hex address].*$func.*" + gdb_exit gdb_spawn |