public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] Allow "unlimited" abbreviations
@ 2019-06-12 23:28 Pedro Alves
  0 siblings, 0 replies; only message in thread
From: Pedro Alves @ 2019-06-12 23:28 UTC (permalink / raw)
  To: gdb-cvs

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

commit 93bcb04349eb500781747a542ececc7a8ba3bd95
Author: Pedro Alves <palves@redhat.com>
Date:   Thu Jun 13 00:06:52 2019 +0100

    Allow "unlimited" abbreviations
    
    Currently we can abbreviate "on/off/enable/disable/yes/no" in boolean
    settings,
    
      (gdb) set non-stop of
      (gdb) set non-stop en
    
    we can abbreviate the items of enumeration commands,
    
      (gdb) set print frame-arguments scal
      (gdb) set scheduler-locking rep
    
    but we cannot abbreviate "unlimited" in integer commands.
    
     (gdb) set print elements u
     No symbol "u" in current context.
    
    This commit fixes that.
    
    Testcases will be in gdb.base/settings.exp and gdb.base/options.exp,
    in following patches.
    
    gdb/ChangeLog:
    2019-06-13  Pedro Alves  <palves@redhat.com>
    
    	* cli/cli-setshow.c (is_unlimited_literal): Allow abbreviations.

Diff:
---
 gdb/ChangeLog         |  4 ++++
 gdb/cli/cli-setshow.c | 12 ++++++++----
 2 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 538db3a..38e34b5 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,7 @@
+2019-06-13  Pedro Alves  <palves@redhat.com>
+
+	* cli/cli-setshow.c (is_unlimited_literal): Allow abbreviations.
+
 2019-06-13  Pedro Alves <palves@redhat.com>
 
 	* ax-gdb.c (agent_command_1): Remove skip_spaces call.
diff --git a/gdb/cli/cli-setshow.c b/gdb/cli/cli-setshow.c
index 5b87f90..96d7bf5 100644
--- a/gdb/cli/cli-setshow.c
+++ b/gdb/cli/cli-setshow.c
@@ -132,12 +132,16 @@ deprecated_show_value_hack (struct ui_file *ignore_file,
 static int
 is_unlimited_literal (const char *arg)
 {
-  size_t len = sizeof ("unlimited") - 1;
-
   arg = skip_spaces (arg);
 
-  return (strncmp (arg, "unlimited", len) == 0
-	  && (isspace (arg[len]) || arg[len] == '\0'));
+  const char *p = skip_to_space (arg);
+
+  size_t len = p - arg;
+
+  if (len > 0 && strncmp ("unlimited", arg, len) == 0)
+    return true;
+
+  return false;
 }


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2019-06-12 23:28 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-06-12 23:28 [binutils-gdb] Allow "unlimited" abbreviations Pedro Alves

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).