Switch gdb's TRY/CATCH to C++ try/catch
authorPedro Alves <palves@redhat.com>
Fri, 22 Apr 2016 15:18:33 +0000 (16:18 +0100)
committerPedro Alves <palves@redhat.com>
Fri, 22 Apr 2016 15:20:49 +0000 (16:20 +0100)
The exceptions-across-readline issue was fixed by the previous commit.
Let's try this again.

gdb/ChangeLog:
2016-04-22  Pedro Alves  <palves@redhat.com>

* common/common-exceptions.h (GDB_XCPT_TRY): Remove mention of
the foreign frames issue.
[__cplusplus] (GDB_XCPT): Define as GDB_XCPT_TRY.

gdb/ChangeLog
gdb/common/common-exceptions.h

index 408a179..828d390 100644 (file)
@@ -1,5 +1,11 @@
 2016-04-22  Pedro Alves  <palves@redhat.com>
 
+       * common/common-exceptions.h (GDB_XCPT_TRY): Remove mention of
+       the foreign frames issue.
+       [__cplusplus] (GDB_XCPT): Define as GDB_XCPT_TRY.
+
+2016-04-22  Pedro Alves  <palves@redhat.com>
+
        * common/common-exceptions.c (enum catcher_state, struct catcher)
        (current_catcher): Define in C++ mode too.
        (exceptions_state_mc_catch): Call throw_exception_sjlj instead of
index 84abcae..c494de2 100644 (file)
@@ -122,9 +122,7 @@ struct gdb_exception
    the only mode supported when GDB is built as a C program.  */
 #define GDB_XCPT_SJMP 1
 
-/* Make GDB exceptions use try/catch behind the scenes.  Can't be made
-   the default until we handle exceptions crossing foreign frames
-   (gdb -> readline callback -> gdb -> error).  */
+/* Make GDB exceptions use try/catch behind the scenes.  */
 #define GDB_XCPT_TRY 2
 
 /* Specify this mode to build with TRY/CATCH mapped directly to raw
@@ -133,8 +131,11 @@ struct gdb_exception
    spurious code between the TRY and the CATCH block.  */
 #define GDB_XCPT_RAW_TRY 3
 
-/* Always use setjmp/longmp, even in C++ mode.  */
-#define GDB_XCPT GDB_XCPT_SJMP
+#ifdef __cplusplus
+# define GDB_XCPT GDB_XCPT_TRY
+#else
+# define GDB_XCPT GDB_XCPT_SJMP
+#endif
 
 /* Functions to drive the sjlj-based exceptions state machine.  Though
    declared here by necessity, these functions should be considered