public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
From: Tom Tromey <tromey@sourceware.org>
To: gdb-cvs@sourceware.org
Subject: [binutils-gdb] Rename puts_filtered_tabular
Date: Tue, 29 Mar 2022 19:43:11 +0000 (GMT)	[thread overview]
Message-ID: <20220329194311.3D09D3865C2A@sourceware.org> (raw)

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=9fbf7f083e21263b0a0e1f76d615adaa4e13cfe5

commit 9fbf7f083e21263b0a0e1f76d615adaa4e13cfe5
Author: Tom Tromey <tom@tromey.com>
Date:   Sun Jan 2 11:54:06 2022 -0700

    Rename puts_filtered_tabular
    
    puts_filtered_tabular is now misnamed, because whether filtering
    happens is now up to the stream.  So, rename it.  (This function is
    pretty weird, and should probably be rewritten to avoid using the
    chars_printed global, and moved into objc-lang.c.  However, I haven't
    done so.)

Diff:
---
 gdb/objc-lang.c |  4 ++--
 gdb/utils.c     | 13 ++++++-------
 gdb/utils.h     |  2 +-
 3 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/gdb/objc-lang.c b/gdb/objc-lang.c
index e116fbe5611..ed13097f7a8 100644
--- a/gdb/objc-lang.c
+++ b/gdb/objc-lang.c
@@ -666,7 +666,7 @@ info_selectors_command (const char *regexp, int from_tty)
 	    *p++ = *name++;
 	  *p++ = '\0';
 	  /* Print in columns.  */
-	  puts_filtered_tabular(asel, maxlen + 1, 0);
+	  puts_tabular(asel, maxlen + 1, 0);
 	}
       begin_line();
     }
@@ -800,7 +800,7 @@ info_classes_command (const char *regexp, int from_tty)
 	    *p++ = *name++;
 	  *p++ = '\0';
 	  /* Print in columns.  */
-	  puts_filtered_tabular(aclass, maxlen + 1, 0);
+	  puts_tabular(aclass, maxlen + 1, 0);
 	}
       begin_line();
     }
diff --git a/gdb/utils.c b/gdb/utils.c
index e358a5a560d..bf6f4fb9b89 100644
--- a/gdb/utils.c
+++ b/gdb/utils.c
@@ -1504,15 +1504,14 @@ pager_file::wrap_here (int indent)
     }
 }
 
-/* Print input string to gdb_stdout, filtered, with wrap, 
-   arranging strings in columns of n chars.  String can be
-   right or left justified in the column.  Never prints 
-   trailing spaces.  String should never be longer than
-   width.  FIXME: this could be useful for the EXAMINE 
-   command, which currently doesn't tabulate very well.  */
+/* Print input string to gdb_stdout arranging strings in columns of n
+   chars.  String can be right or left justified in the column.  Never
+   prints trailing spaces.  String should never be longer than width.
+   FIXME: this could be useful for the EXAMINE command, which
+   currently doesn't tabulate very well.  */
 
 void
-puts_filtered_tabular (char *string, int width, int right)
+puts_tabular (char *string, int width, int right)
 {
   int spaces = 0;
   int stringlen;
diff --git a/gdb/utils.h b/gdb/utils.h
index 981090e3019..8b7eccb7bb3 100644
--- a/gdb/utils.h
+++ b/gdb/utils.h
@@ -241,7 +241,7 @@ extern int gdb_putc (int c);
 
 extern void gdb_puts (const char *);
 
-extern void puts_filtered_tabular (char *string, int width, int right);
+extern void puts_tabular (char *string, int width, int right);
 
 extern void gdb_vprintf (const char *, va_list) ATTRIBUTE_PRINTF (1, 0);


                 reply	other threads:[~2022-03-29 19:43 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220329194311.3D09D3865C2A@sourceware.org \
    --to=tromey@sourceware.org \
    --cc=gdb-cvs@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).