From patchwork Sun Dec 1 19:04:07 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Simon Marchi (Code Review)" X-Patchwork-Id: 36400 Received: (qmail 70320 invoked by alias); 1 Dec 2019 19:04:18 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 70253 invoked by uid 89); 1 Dec 2019 19:04:17 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-21.5 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, KAM_SHORT autolearn=ham version=3.3.1 spammy=closer, horizontal, shown, Show X-HELO: mx1.osci.io Received: from polly.osci.io (HELO mx1.osci.io) (8.43.85.229) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sun, 01 Dec 2019 19:04:14 +0000 Received: by mx1.osci.io (Postfix, from userid 994) id D65AD20A87; Sun, 1 Dec 2019 14:04:12 -0500 (EST) Received: from gnutoolchain-gerrit.osci.io (gnutoolchain-gerrit.osci.io [8.43.85.239]) by mx1.osci.io (Postfix) with ESMTP id 77DDB2039A; Sun, 1 Dec 2019 14:04:08 -0500 (EST) Received: from localhost (localhost [127.0.0.1]) by gnutoolchain-gerrit.osci.io (Postfix) with ESMTP id 36F592816F; Sun, 1 Dec 2019 14:04:08 -0500 (EST) X-Gerrit-PatchSet: 3 Date: Sun, 1 Dec 2019 14:04:07 -0500 From: "Sourceware to Gerrit sync (Code Review)" To: Tom Tromey , Andrew Burgess , gdb-patches@sourceware.org Auto-Submitted: auto-generated X-Gerrit-MessageType: newpatchset Subject: [pushed] Allow using less horizontal space in TUI source window X-Gerrit-Change-Id: I46ce9a68b12c9c79332d510f9c14b3c84b7efadd X-Gerrit-Change-Number: 605 X-Gerrit-ChangeURL: X-Gerrit-Commit: d1da6b01608841c846aa75209248e276f49e1587 In-Reply-To: References: Reply-To: noreply@gnutoolchain-gerrit.osci.io, tromey@sourceware.org, andrew.burgess@embecosm.com, gdb-patches@sourceware.org MIME-Version: 1.0 Content-Disposition: inline User-Agent: Gerrit/3.0.3-79-g83ff7f88f1 Message-Id: <20191201190408.36F592816F@gnutoolchain-gerrit.osci.io> The original change was created by Tom Tromey. Change URL: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/605 ...................................................................... Allow using less horizontal space in TUI source window The source window currently uses a field width of 6 for line numbers, and it further aligns to the next tab stop. This seemed a bit wasteful of horizontal space to me, so I changed that in an earlier patch. However, that change wasn't universally popular. This patch instead adds the option to use less horizontal space in the TUI source window. gdb/ChangeLog 2019-12-01 Tom Tromey * tui/tui-winsource.h (tui_copy_source_line): Add "ndigits" parameter. * tui/tui-winsource.c (tui_copy_source_line): Add "ndigits" parameter. * tui/tui-win.h (compact_source): Declare. * tui/tui-win.c (compact_source): New global. (tui_set_compact_source, tui_show_compact_source): New functions. (_initialize_tui_win): Add "compact-source" setting. * tui/tui-source.c (tui_source_window::set_contents): Handle compact_source setting. * tui/tui-disasm.c (tui_disasm_window::set_contents): Update. * NEWS: Document new setting. gdb/doc/ChangeLog 2019-12-01 Tom Tromey * gdb.texinfo (TUI Configuration): Document new setting. Change-Id: I46ce9a68b12c9c79332d510f9c14b3c84b7efadd --- M gdb/ChangeLog M gdb/NEWS M gdb/doc/ChangeLog M gdb/doc/gdb.texinfo M gdb/tui/tui-disasm.c M gdb/tui/tui-source.c M gdb/tui/tui-win.c M gdb/tui/tui-win.h M gdb/tui/tui-winsource.c M gdb/tui/tui-winsource.h 10 files changed, 102 insertions(+), 11 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 621cbbb..15f3e6c 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,18 @@ +2019-12-01 Tom Tromey + + * tui/tui-winsource.h (tui_copy_source_line): Add "ndigits" + parameter. + * tui/tui-winsource.c (tui_copy_source_line): Add "ndigits" + parameter. + * tui/tui-win.h (compact_source): Declare. + * tui/tui-win.c (compact_source): New global. + (tui_set_compact_source, tui_show_compact_source): New functions. + (_initialize_tui_win): Add "compact-source" setting. + * tui/tui-source.c (tui_source_window::set_contents): Handle + compact_source setting. + * tui/tui-disasm.c (tui_disasm_window::set_contents): Update. + * NEWS: Document new setting. + 2019-11-30 Tom Tromey * dwarf2read.c (dwarf2_add_field): Include field offset when diff --git a/gdb/NEWS b/gdb/NEWS index e477986..56d4a34 100644 --- a/gdb/NEWS +++ b/gdb/NEWS @@ -188,6 +188,14 @@ 'frame', 'stepi'. The python frame filtering also respect this setting. The 'backtrace' '-frame-info' option can override this global setting. +set tui compact-source +show tui compact-source + + Enable the "compact" display mode for the TUI source window. The + compact display uses only as much space as is needed for the line + numbers in the current file, and only a single space to separate the + line numbers from the source. + info modules [-q] [REGEXP] Return a list of Fortran modules matching REGEXP, or all modules if no REGEXP is given. diff --git a/gdb/doc/ChangeLog b/gdb/doc/ChangeLog index aaac75a..8028f78 100644 --- a/gdb/doc/ChangeLog +++ b/gdb/doc/ChangeLog @@ -1,3 +1,7 @@ +2019-12-01 Tom Tromey + + * gdb.texinfo (TUI Configuration): Document new setting. + 2019-11-30 Philippe Waroquiers * gdb.texinfo (Define): Indicate that user-defined prefix can diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo index 2c30ea6..9b5297e 100644 --- a/gdb/doc/gdb.texinfo +++ b/gdb/doc/gdb.texinfo @@ -28017,6 +28017,14 @@ Set the width of tab stops to be @var{nchars} characters. This setting affects the display of TAB characters in the source and assembly windows. + +@item set tui compact-source @r{[}on@r{|}off@r{]} +@kindex set tui compact-source +Set whether the TUI source window is displayed in ``compact'' form. +The default display uses more space for line numbers and starts the +source text at the next tab stop; the compact display uses only as +much space as is needed for the line numbers in the current file, and +only a single space to separate the line numbers from the source. @end table @node Emacs diff --git a/gdb/tui/tui-disasm.c b/gdb/tui/tui-disasm.c index 8d5512e..9819cb9 100644 --- a/gdb/tui/tui-disasm.c +++ b/gdb/tui/tui-disasm.c @@ -245,7 +245,7 @@ + asm_lines[i].insn); const char *ptr = line.c_str (); - src->line = tui_copy_source_line (&ptr, -1, offset, line_width); + src->line = tui_copy_source_line (&ptr, -1, offset, line_width, 0); src->line_or_addr.loa = LOA_ADDRESS; src->line_or_addr.u.addr = asm_lines[i].addr; diff --git a/gdb/tui/tui-source.c b/gdb/tui/tui-source.c index 915f9e3..32877d7 100644 --- a/gdb/tui/tui-source.c +++ b/gdb/tui/tui-source.c @@ -20,6 +20,7 @@ along with this program. If not, see . */ #include "defs.h" +#include #include #include "symtab.h" #include "frame.h" @@ -33,6 +34,7 @@ #include "tui/tui-data.h" #include "tui/tui-io.h" #include "tui/tui-stack.h" +#include "tui/tui-win.h" #include "tui/tui-winsource.h" #include "tui/tui-source.h" #include "gdb_curses.h" @@ -59,8 +61,10 @@ nlines = (line_no + (height - 2)) - line_no; std::string srclines; + const std::vector *offsets; if (!g_source_cache.get_source_lines (s, line_no, line_no + nlines, - &srclines)) + &srclines) + || !g_source_cache.get_line_charpos (s, &offsets)) ret = TUI_FAILURE; else { @@ -78,6 +82,13 @@ start_line_or_addr.loa = LOA_LINE; cur_line_no = start_line_or_addr.u.line_no = line_no; + int digits = 0; + if (compact_source) + { + double l = log10 (offsets->size ()); + digits = 1 + (int) l; + } + const char *iter = srclines.c_str (); content.resize (nlines); while (cur_line < nlines) @@ -89,7 +100,7 @@ if (*iter != '\0') text = tui_copy_source_line (&iter, cur_line_no, horizontal_offset, - line_width); + line_width, digits); /* Set whether element is the execution point and whether there is a break point on it. */ diff --git a/gdb/tui/tui-win.c b/gdb/tui/tui-win.c index f47dad8..576f9a5 100644 --- a/gdb/tui/tui-win.c +++ b/gdb/tui/tui-win.c @@ -927,6 +927,29 @@ } +/* See tui-win.h. */ + +bool compact_source = false; + +/* Callback for "set tui compact-source". */ + +static void +tui_set_compact_source (const char *ignore, int from_tty, + struct cmd_list_element *c) +{ + if (TUI_SRC_WIN != nullptr) + TUI_SRC_WIN->refill (); +} + +/* Callback for "show tui compact-source". */ + +static void +tui_show_compact_source (struct ui_file *file, int from_tty, + struct cmd_list_element *c, const char *value) +{ + printf_filtered (_("TUI source window compactness is %s.\n"), value); +} + /* Set the tab width of the specified window. */ static void tui_set_tab_width_command (const char *arg, int from_tty) @@ -1484,4 +1507,14 @@ show_tui_resize_message, &maintenance_set_cmdlist, &maintenance_show_cmdlist); + + add_setshow_boolean_cmd ("compact-source", class_tui, + &compact_source, _("\ +Set whether the TUI source window is compact."), _("\ +Show whether the TUI source window is compact."), _("\ +This variable controls whether the TUI source window is shown\n\ +in a compact form. The compact form puts the source closer to\n\ +the line numbers and uses less horizontal space."), + tui_set_compact_source, tui_show_compact_source, + &tui_setlist, &tui_showlist); } diff --git a/gdb/tui/tui-win.h b/gdb/tui/tui-win.h index 81b7dac..1ffe683 100644 --- a/gdb/tui/tui-win.h +++ b/gdb/tui/tui-win.h @@ -54,4 +54,7 @@ /* Create or get the TUI command list. */ struct cmd_list_element **tui_get_cmd_list (void); +/* Whether compact source display should be used. */ +extern bool compact_source; + #endif /* TUI_TUI_WIN_H */ diff --git a/gdb/tui/tui-winsource.c b/gdb/tui/tui-winsource.c index 3ca723c..81937c1 100644 --- a/gdb/tui/tui-winsource.c +++ b/gdb/tui/tui-winsource.c @@ -69,7 +69,7 @@ std::string tui_copy_source_line (const char **ptr, int line_no, int first_col, - int line_width) + int line_width, int ndigits) { const char *lineptr = *ptr; @@ -78,10 +78,15 @@ if (line_no > 0) { - result = string_printf ("%-6d", line_no); - int len = result.size (); - len = len - ((len / tui_tab_width) * tui_tab_width); - result.append (len, ' '); + if (ndigits > 0) + result = string_printf ("%*d ", ndigits, line_no); + else + { + result = string_printf ("%-6d", line_no); + int len = result.size (); + len = len - ((len / tui_tab_width) * tui_tab_width); + result.append (len, ' '); + } } int column = 0; diff --git a/gdb/tui/tui-winsource.h b/gdb/tui/tui-winsource.h index 7c3c626..8b96200 100644 --- a/gdb/tui/tui-winsource.h +++ b/gdb/tui/tui-winsource.h @@ -238,12 +238,16 @@ /* Extract some source text from PTR. LINE_NO is the line number. If it is positive, it is printed at the start of the line. FIRST_COL is the first column to extract, and LINE_WIDTH is the number of - characters to display. Returns a string holding the desired text. - PTR is updated to point to the start of the next line. */ + characters to display. NDIGITS is used to format the line number + (if it is positive). If NDIGITS is greater than 0, then that many + digits are used; otherwise the line number is formatted with 6 + digits and the text is aligned to the next tab stop. Returns a + string holding the desired text. PTR is updated to point to the + start of the next line. */ extern std::string tui_copy_source_line (const char **ptr, int line_no, int first_col, - int line_width); + int line_width, int ndigits); /* Constant definitions. */ #define SCROLL_THRESHOLD 2 /* Threshold for lazy scroll. */