Add "volatile" keyword to "struct gdb_exception" declaration
authorSergio Durigan Junior <sergiodj@redhat.com>
Fri, 17 Jan 2014 21:39:57 +0000 (19:39 -0200)
committerSergio Durigan Junior <sergiodj@redhat.com>
Fri, 17 Jan 2014 21:39:57 +0000 (19:39 -0200)
While doing something else, I found that those 2 places were incorrectly
declaring a "struct gdb_exception" without using the "volatile" keyword.
This commit fixes that.

2014-01-17  Sergio Durigan Junior  <sergiodj@redhat.com>

* breakpoint.c (insert_bp_location): Add "volatile" keyword to "struct
gdb_exception" declaration.
* remote.c (getpkt_or_notif_sane): Likewise.

gdb/ChangeLog
gdb/breakpoint.c
gdb/remote.c

index a19de9f..28e8f6a 100644 (file)
@@ -1,3 +1,9 @@
+2014-01-17  Sergio Durigan Junior  <sergiodj@redhat.com>
+
+       * breakpoint.c (insert_bp_location): Add "volatile" keyword to "struct
+       gdb_exception" declaration.
+       * remote.c (getpkt_or_notif_sane): Likewise.
+
 2014-01-17  Doug Evans  <dje@google.com>
 
        * common/gdb_vecs.c (delim_string_to_char_ptr_vec_append): New
index 642ffdf..c8e7e88 100644 (file)
@@ -2396,7 +2396,7 @@ insert_bp_location (struct bp_location *bl,
 {
   enum errors bp_err = GDB_NO_ERROR;
   const char *bp_err_message = NULL;
-  struct gdb_exception e;
+  volatile struct gdb_exception e;
 
   if (!should_be_inserted (bl) || (bl->inserted && !bl->needs_update))
     return 0;
index e8469e3..d886929 100644 (file)
@@ -7873,7 +7873,7 @@ getpkt_or_notif_sane (char **buf, long *sizeof_buf, int forever,
 static void
 remote_kill (struct target_ops *ops)
 {
-  struct gdb_exception ex;
+  volatile struct gdb_exception ex;
 
   /* Catch errors so the user can quit from gdb even when we
      aren't on speaking terms with the remote system.  */