* breakpoint.c (get_number_trailer): No longer accept a NULL PP.

	* breakpoint.h (get_number_or_range): Declare.
	* printcmd.c (ALL_DISPLAYS): Declare.
	(delete_display): Reimplement taking a display pointer.
	(undisplay_command): Accept a range of displays to delete, using
	get_number_or_range.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index c55cc8f34608ddf553e678961cde355a5b68e720..bd987d61e7d06d13321e3d169f50e4d00bd29b0c 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,12 @@
+2011-02-18  Pedro Alves  <pedro@codesourcery.com>
+
+       * breakpoint.c (get_number_trailer): No longer accept a NULL PP.
+       * breakpoint.h (get_number_or_range): Declare.
+       * printcmd.c (ALL_DISPLAYS): Declare.
+       (delete_display): Reimplement taking a display pointer.
+       (undisplay_command): Accept a range of displays to delete, using
+       get_number_or_range.
+
 2011-02-18  Pierre Muller  <muller@ics.u-strasbg.fr>
 
        * c-valprint.c (c_val_print): Add embedded_offset to address

diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index ca56c435943b80c389fa87eb4da1828e6c328637..c9e149b656b0ba7dcf0d62db500d73910445b855 100644 (file)
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -561,8 +561,6 @@ struct program_space *default_breakpoint_pspace;
    name of a convenience variable.  Making it an expression wouldn't
    work well for map_breakpoint_numbers (e.g. "4 + 5 + 6").
 
-   If the string is a NULL pointer, that denotes the last breakpoint.
-   
    TRAILER is a character which can be found after the number; most
    commonly this is `-'.  If you don't want a trailer, use \0.  */
 
@@ -572,10 +570,7 @@ get_number_trailer (char **pp, int trailer)
   int retval = 0;      /* default */
   char *p = *pp;
 
-  if (p == NULL)
-    /* Empty line means refer to the last breakpoint.  */
-    return breakpoint_count;
-  else if (*p == '$')
+  if (*p == '$')
     {
       /* Make a copy of the name, so we can null-terminate it
          to pass to lookup_internalvar().  */
@@ -651,7 +646,7 @@ get_number (char **pp)
    is completed.  The call that completes the range will advance
    pointer PP past <number2>.  */
 
-int 
+int
 get_number_or_range (char **pp)
 {
   static int last_retval, end_value;

diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h
index 86fae669fc39a77719bdc81670d96f89c073a3d8..a3927cfbc188626ede87089bd46c6f8b20eddbb6 100644 (file)
--- a/gdb/breakpoint.h
+++ b/gdb/breakpoint.h
@@ -1191,4 +1191,6 @@ extern struct breakpoint *iterate_over_breakpoints (int (*) (struct breakpoint *
 
 extern int user_breakpoint_p (struct breakpoint *);
 
+extern int get_number_or_range (char **pp);
+
 #endif /* !defined (BREAKPOINT_H) */

diff --git a/gdb/printcmd.c b/gdb/printcmd.c
index 29ffbf5253f6f4daa66099a08c235f87f6c4241c..6576dce30216ab7f9f3efeadb0dbdfe350c5daf7 100644 (file)
--- a/gdb/printcmd.c
+++ b/gdb/printcmd.c
@@ -167,6 +167,11 @@ static struct display *display_chain;
 
 static int display_number;
 
+/* Walk the following statement or block through all displays.  */
+
+#define ALL_DISPLAYS(B)                                \
+  for (B = display_chain; B; B = B->next)
+
 /* Prototypes for exported functions.  */
 
 void output_command (char *, int);
@@ -1555,35 +1560,26 @@ clear_displays (void)
     }
 }
 
-/* Delete the auto-display number NUM.  */
+/* Delete the auto-display DISPLAY.  */
 
 static void
-delete_display (int num)
+delete_display (struct display *display)
 {
-  struct display *d1, *d;
+  struct display *d;
 
-  if (!display_chain)
-    error (_("No display number %d."), num);
+  gdb_assert (display != NULL);
 
-  if (display_chain->number == num)
-    {
-      d1 = display_chain;
-      display_chain = d1->next;
-      free_display (d1);
-    }
-  else
-    for (d = display_chain;; d = d->next)
+  if (display_chain == display)
+    display_chain = display->next;
+
+  ALL_DISPLAYS (d)
+    if (d->next == display)
       {
-       if (d->next == 0)
-         error (_("No display number %d."), num);
-       if (d->next->number == num)
-         {
-           d1 = d->next;
-           d->next = d1->next;
-           free_display (d1);
-           break;
-         }
+       d->next = display->next;
+       break;
       }
+
+  free_display (display);
 }
 
 /* Delete some values from the auto-display chain.
@@ -1607,18 +1603,24 @@ undisplay_command (char *args, int from_tty)
   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 = get_number_or_range (&p1);
+      if (num == 0)
+       warning (_("bad display number at or near '%s'"), p);
+      else
+       {
+         struct display *d;
 
-      delete_display (num);
+         ALL_DISPLAYS (d)
+           if (d->number == num)
+             break;
+         if (d == NULL)
+           printf_unfiltered (_("No display number %d.\n"), num);
+         else
+           delete_display (d);
+       }
 
       p = p1;
-      while (*p == ' ' || *p == '\t')
-       p++;
     }
   dont_repeat ();
 }