* mi/mi-cmd-stack.c (list_args_or_locals): Use internal_error.

	Put "break" statements on their own line.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index c18ed80..cc69b57 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2009-10-10  Pedro Alves  <pedro@codesourcery.com>
+
+       * mi/mi-cmd-stack.c (list_args_or_locals): Use internal_error.
+       Put "break" statements on their own line.
+
 2009-10-09  Pedro Alves  <pedro@codesourcery.com>
 
        * linux-nat.c (linux_nat_wait_1): Bail out, if TARGET_WNOHANG and

diff --git a/gdb/mi/mi-cmd-stack.c b/gdb/mi/mi-cmd-stack.c
index 3dad54f..537af77 100644 (file)
--- a/gdb/mi/mi-cmd-stack.c
+++ b/gdb/mi/mi-cmd-stack.c
@@ -256,13 +256,17 @@ list_args_or_locals (enum what_to_list what, int values, struct frame_info *fi)
   switch (what)
     {
     case locals:
-      name_of_result = "locals"; break;
+      name_of_result = "locals";
+      break;
     case arguments:
-      name_of_result = "args"; break;
+      name_of_result = "args";
+      break;
     case all:
-      name_of_result = "variables"; break;
+      name_of_result = "variables";
+      break;
     default:
-      gdb_assert (("unexpected value", 0));
+      internal_error (__FILE__, __LINE__,
+                     "unexpected what_to_list: %d", (int) what);
     }
 
   cleanup_list = make_cleanup_ui_out_list_begin_end (uiout, name_of_result);