From b0b13bb4cc57090510c28345de1c5ab678c68b13 Mon Sep 17 00:00:00 2001
From: Daniel Jacobowitz <drow@false.org>
Date: Tue, 10 Apr 2007 14:53:46 +0000
Subject: [PATCH] 	* breakpoint.c (gdb_breakpoint_query): Really return
 an 	enum gdb_rc. 	(gdb_breakpoint): Likewise. 	* thread.c
 (do_captured_list_thread_ids): Likewise. 	(do_captured_thread_select):
 Likewise. 	* mi/mi-main.c (mi_cmd_thread_select): Expect an enum gdb_rc. 
 (mi_cmd_thread_list_ids): Remove bogus initialization.

---
 gdb/ChangeLog    | 10 ++++++++++
 gdb/breakpoint.c | 14 ++++++++++----
 gdb/mi/mi-main.c |  8 ++------
 gdb/thread.c     | 12 ++++++++----
 4 files changed, 30 insertions(+), 14 deletions(-)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 477de9ef9b..b123a9582d 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,13 @@
+2007-04-10  Daniel Jacobowitz  <dan@codesourcery.com>
+
+	* breakpoint.c (gdb_breakpoint_query): Really return an
+	enum gdb_rc.
+	(gdb_breakpoint): Likewise.
+	* thread.c (do_captured_list_thread_ids): Likewise.
+	(do_captured_thread_select): Likewise.
+	* mi/mi-main.c (mi_cmd_thread_select): Expect an enum gdb_rc.
+	(mi_cmd_thread_list_ids): Remove bogus initialization.
+
 2007-04-10  Daniel Jacobowitz  <dan@codesourcery.com>
 
 	* Makefile.in (SFILES): Remove hpacc-abi.c.
diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index fc8e389d3a..4ca386cfab 100644
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -3757,8 +3757,11 @@ gdb_breakpoint_query (struct ui_out *uiout, int bnum, char **error_message)
   args.bnum = bnum;
   /* For the moment we don't trust print_one_breakpoint() to not throw
      an error. */
-  return catch_exceptions_with_msg (uiout, do_captured_breakpoint_query, &args,
-				    error_message, RETURN_MASK_ALL);
+  if (catch_exceptions_with_msg (uiout, do_captured_breakpoint_query, &args,
+				 error_message, RETURN_MASK_ALL) < 0)
+    return GDB_RC_FAIL;
+  else
+    return GDB_RC_OK;
 }
 
 /* Return non-zero if B is user settable (breakpoints, watchpoints,
@@ -5600,8 +5603,11 @@ gdb_breakpoint (char *address, char *condition,
   args.tempflag = tempflag;
   args.thread = thread;
   args.ignore_count = ignore_count;
-  return catch_exceptions_with_msg (uiout, do_captured_breakpoint, &args,
-				    error_message, RETURN_MASK_ALL);
+  if (catch_exceptions_with_msg (uiout, do_captured_breakpoint, &args,
+				 error_message, RETURN_MASK_ALL) < 0)
+    return GDB_RC_FAIL;
+  else
+    return GDB_RC_OK;
 }
 
 
diff --git a/gdb/mi/mi-main.c b/gdb/mi/mi-main.c
index 449e7d0c3d..bdbb309e54 100644
--- a/gdb/mi/mi-main.c
+++ b/gdb/mi/mi-main.c
@@ -253,11 +253,7 @@ mi_cmd_thread_select (char *command, char **argv, int argc)
   else
     rc = gdb_thread_select (uiout, argv[0], &mi_error_message);
 
-  /* RC is enum gdb_rc if it is successful (>=0)
-     enum return_reason if not (<0).  */
-  if ((int) rc < 0 && (enum return_reason) rc == RETURN_ERROR)
-    return MI_CMD_ERROR;
-  else if ((int) rc >= 0 && rc == GDB_RC_FAIL)
+  if (rc == GDB_RC_FAIL)
     return MI_CMD_ERROR;
   else
     return MI_CMD_DONE;
@@ -266,7 +262,7 @@ mi_cmd_thread_select (char *command, char **argv, int argc)
 enum mi_cmd_result
 mi_cmd_thread_list_ids (char *command, char **argv, int argc)
 {
-  enum gdb_rc rc = MI_CMD_DONE;
+  enum gdb_rc rc;
 
   if (argc != 0)
     {
diff --git a/gdb/thread.c b/gdb/thread.c
index dbfd2dd82d..d2a6e6fc2b 100644
--- a/gdb/thread.c
+++ b/gdb/thread.c
@@ -284,8 +284,10 @@ do_captured_list_thread_ids (struct ui_out *uiout, void *arg)
 enum gdb_rc
 gdb_list_thread_ids (struct ui_out *uiout, char **error_message)
 {
-  return catch_exceptions_with_msg (uiout, do_captured_list_thread_ids, NULL,
-				    error_message, RETURN_MASK_ALL);
+  if (catch_exceptions_with_msg (uiout, do_captured_list_thread_ids, NULL,
+				 error_message, RETURN_MASK_ALL) < 0)
+    return GDB_RC_FAIL;
+  return GDB_RC_OK;
 }
 
 /* Load infrun state for the thread PID.  */
@@ -705,8 +707,10 @@ do_captured_thread_select (struct ui_out *uiout, void *tidstr)
 enum gdb_rc
 gdb_thread_select (struct ui_out *uiout, char *tidstr, char **error_message)
 {
-  return catch_exceptions_with_msg (uiout, do_captured_thread_select, tidstr,
-				    error_message, RETURN_MASK_ALL);
+  if (catch_exceptions_with_msg (uiout, do_captured_thread_select, tidstr,
+				 error_message, RETURN_MASK_ALL) < 0)
+    return GDB_RC_FAIL;
+  return GDB_RC_OK;
 }
 
 /* Commands with a prefix of `thread'.  */