This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[obv] breakpoint.c (bpstat_stop_status): Delete useless code.


Hi.

I checked this in as obvious.

The assignment to bs is unnecessary as it gets assigned again
immediately afterwards:

  /* If we aren't stopping, the value of some hardware watchpoint may
     not have changed, but the intermediate memory locations we are
     watching may have.  Don't bother if we're stopping; this will get
     done later.  */
  for (bs = root_bs->next; bs != NULL; bs = bs->next)
    if (bs->stop)
      break;


2010-01-26  Doug Evans  <dje@google.com>

	* breakpoint.c (bpstat_stop_status): Delete useless code.

Index: breakpoint.c
===================================================================
RCS file: /cvs/src/src/gdb/breakpoint.c,v
retrieving revision 1.451
diff -u -p -r1.451 breakpoint.c
--- breakpoint.c	19 Jan 2010 20:31:34 -0000	1.451
+++ breakpoint.c	26 Jan 2010 23:55:06 -0000
@@ -3668,7 +3668,6 @@ bpstat_stop_status (struct address_space
     }
 
   bs->next = NULL;		/* Terminate the chain */
-  bs = root_bs->next;		/* Re-grab the head of the chain */
 
   /* If we aren't stopping, the value of some hardware watchpoint may
      not have changed, but the intermediate memory locations we are


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]