From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-78383-listarch-gdb-patches=sources.redhat.com@sourceware.org>
Received: (qmail 6953 invoked by alias); 18 Feb 2011 09:47:32 -0000
Received: (qmail 6944 invoked by uid 22791); 18 Feb 2011 09:47:31 -0000
X-SWARE-Spam-Status: No, hits=-2.7 required=5.0	tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,T_TO_NO_BRKTS_FREEMAIL
X-Spam-Check-By: sourceware.org
Received: from mail-qw0-f41.google.com (HELO mail-qw0-f41.google.com) (209.85.216.41)    by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 18 Feb 2011 09:47:25 +0000
Received: by qwd7 with SMTP id 7so205946qwd.0        for <gdb-patches@sourceware.org>; Fri, 18 Feb 2011 01:47:24 -0800 (PST)
MIME-Version: 1.0
Received: by 10.229.229.83 with SMTP id jh19mr355433qcb.234.1298022444194; Fri, 18 Feb 2011 01:47:24 -0800 (PST)
Received: by 10.229.31.198 with HTTP; Fri, 18 Feb 2011 01:47:24 -0800 (PST)
In-Reply-To: <AANLkTikgj9Pxw15Ryx=E=tkGqALaK+WOzrGykfrOggTY@mail.gmail.com>
References: <AANLkTikgj9Pxw15Ryx=E=tkGqALaK+WOzrGykfrOggTY@mail.gmail.com>
Date: Fri, 18 Feb 2011 10:30:00 -0000
Message-ID: <AANLkTinN_Tes7oY=J4d3KgbmmW=r2A64tL1F-PSL+Z0p@mail.gmail.com>
Subject: Re: [patch] delete a range of display numbers
From: Guillaume Leconte <guillaume.leconte@gmail.com>
To: gdb-patches@sourceware.org
Content-Type: text/plain; charset=ISO-8859-1
X-IsSubscribed: yes
Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <gdb-patches.sourceware.org>
List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org>
List-Archive: <http://sourceware.org/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sourceware.org>
List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs>
Sender: gdb-patches-owner@sourceware.org
X-SW-Source: 2011-02/txt/msg00450.txt.bz2

This version fits more with the indentation style, and fixes a typo (I
omitted to change a call
to delete_display()). Sorry for the noise.

diff --git a/gdb/printcmd.c b/gdb/printcmd.c
index 29ffbf5..deb1b80 100644
--- a/gdb/printcmd.c
+++ b/gdb/printcmd.c
@@ -1558,12 +1558,16 @@ clear_displays (void)
 /* Delete the auto-display number NUM.  */

 static void
-delete_display (int num)
+delete_display (int num, int ignore)
 {
   struct display *d1, *d;

   if (!display_chain)
-    error (_("No display number %d."), num);
+    {
+      if (ignore)
+        return;
+      error (_("No display number %d."), num);
+    }

   if (display_chain->number == num)
     {
@@ -1575,7 +1579,11 @@ delete_display (int num)
     for (d = display_chain;; d = d->next)
       {
 	if (d->next == 0)
-	  error (_("No display number %d."), num);
+          {
+            if (ignore)
+              return;
+            error (_("No display number %d."), num);
+          }
 	if (d->next->number == num)
 	  {
 	    d1 = d->next;
@@ -1586,6 +1594,40 @@ delete_display (int num)
       }
 }

+static int
+set_int_from_str (char *str, int *result, char *end_delim)
+{
+  char *p;
+  int ret;
+  char *occurrence;
+
+  p = str;
+  ret = -1;
+
+  if (! str)
+    goto err;
+
+  while (p && *p && (' ' == *p || '\t' == *p))
+    p++;
+
+  while (p && *p && *p >= '0' && *p <= '9')
+    p++;
+
+  if (p && *p)
+    {
+      occurrence = strpbrk (p, end_delim);
+      if (! occurrence || occurrence != p)
+        goto err;
+    }
+
+  if (result)
+    *result = atoi (str);
+
+  ret = 0;
+ err:
+  return ret;
+}
+
 /* Delete some values from the auto-display chain.
    Specify the element numbers.  */

@@ -1595,7 +1637,8 @@ undisplay_command (char *args, int from_tty)
   char *p = args;
   char *p1;
   int num;
-
+  int lower, upper, i;
+  char *dash;
   if (args == 0)
     {
       if (query (_("Delete all auto-display expressions? ")))
@@ -1604,23 +1647,44 @@ undisplay_command (char *args, int from_tty)
       return;
     }

-  while (*p)
+  dash = strchr(p, '-');
+  if (dash) /* remove all the display IDs within a range */
+    {
+      if (-1 == set_int_from_str(p, &lower, "- \t"))
+        error (_("Arguments must be display numbers."));
+
+      p = dash+1;
+      while (p && *p && (' ' == *p || '\t' == *p))
+        p++;
+
+      if (-1 == set_int_from_str(p, &upper, " \t"))
+        error (_("Arguments must be display numbers."));
+
+      for (i = lower; i <= upper; i++)
+        delete_display(i, 1);
+
+      dont_repeat ();
+    }
+  else
     {
-      p1 = p;
-      while (*p1 >= '0' && *p1 <= '9')
-	p1++;
-      if (*p1 && *p1 != ' ' && *p1 != '\t')
-	error (_("Arguments must be display numbers."));
+      while (*p)
+        {
+          p1 = p;
+          while (*p1 >= '0' && *p1 <= '9')
+            p1++;
+          if (*p1 && *p1 != ' ' && *p1 != '\t')
+            error (_("Arguments must be display numbers."));

-      num = atoi (p);
+          num = atoi (p);

-      delete_display (num);
+          delete_display (num, 0);

-      p = p1;
-      while (*p == ' ' || *p == '\t')
-	p++;
+          p = p1;
+          while (*p == ' ' || *p == '\t')
+            p++;
+        }
+      dont_repeat ();
     }
-  dont_repeat ();
 }

 /* Display a single auto-display.


--
"A fellow of infinite jest, of most excellent fancy."