[binutils-gdb] Implement breakpoint_find_if

[Sergio Durigan Junior <sergiodj at sourceware dot org>]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=badd37cec86e705a190e501ab26975d352718619

commit badd37cec86e705a190e501ab26975d352718619
Author: Sergio Durigan Junior <sergiodj@redhat.com>
Date:   Wed Mar 11 14:09:51 2015 -0400

    Implement breakpoint_find_if
    
    This commit implements the 'breakpoint_find_if' function, which allows
    code external to gdb/breakpoint.c to iterate through the list of
    'struct breakpoint *'.  This is needed in order to create the
    'gdb/break-catch-syscall.c' file, because one of its functions
    (catching_syscall_number) needs to do this iteration.
    
    My first thought was to share the ALL_BREAKPOINTS* macros on
    gdb/breakpoint.h, but they use a global variable local to
    gdb/breakpoint.c, and I did not want to share that variable.  So, in
    order to keep the minimal separation, I decided to implement this
    way of iterating through the existing 'struct breakpoint *'.
    
    This function was based on BFD's bfd_sections_find_if.  If the
    user-provided function returns 0, the iteration proceeds.  Otherwise,
    the iteration stops and the function returns the 'struct breakpoint *'
    that is being processed.  This means that the return value of this
    function can be either NULL or a pointer to a 'struct breakpoint'.
    
    gdb/ChangeLog:
    2015-03-11  Sergio Durigan Junior  <sergiodj@redhat.com>
    
    	* breakpoint.c (breakpoint_find_if): New function.
    	* breakpoint.h (breakpoint_find_if): New prototype.

Diff:
---
 gdb/ChangeLog    |  5 +++++
 gdb/breakpoint.c | 17 +++++++++++++++++
 gdb/breakpoint.h | 14 ++++++++++++++
 3 files changed, 36 insertions(+)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 8aa3862..ca9fc15 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2015-03-11  Sergio Durigan Junior  <sergiodj@redhat.com>
+
+	* breakpoint.c (breakpoint_find_if): New function.
+	* breakpoint.h (breakpoint_find_if): New prototype.
+
 2015-03-11  Gary Benson <gbenson@redhat.com>
 
 	* remote-fileio.h (remote_fileio_to_host_stat): New declaration.
diff --git a/gdb/breakpoint.c b/gdb/breakpoint.c
index 923523e..c0ab1d7 100644
--- a/gdb/breakpoint.c
+++ b/gdb/breakpoint.c
@@ -644,6 +644,23 @@ static struct cmd_list_element *breakpoint_set_cmdlist;
 static struct cmd_list_element *breakpoint_show_cmdlist;
 struct cmd_list_element *save_cmdlist;
 
+/* See declaration at breakpoint.h.  */
+
+struct breakpoint *
+breakpoint_find_if (int (*func) (struct breakpoint *b, void *d),
+		    void *user_data)
+{
+  struct breakpoint *b = NULL;
+
+  ALL_BREAKPOINTS (b)
+    {
+      if (func (b, user_data) != 0)
+	break;
+    }
+
+  return b;
+}
+
 /* Return whether a breakpoint is an active enabled breakpoint.  */
 static int
 breakpoint_enabled (struct breakpoint *b)
diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h
index 85c2240..b85939a 100644
--- a/gdb/breakpoint.h
+++ b/gdb/breakpoint.h
@@ -825,6 +825,20 @@ struct watchpoint
   CORE_ADDR hw_wp_mask;
 };
 
+/* Given a function FUNC (struct breakpoint *B, void *DATA) and
+   USER_DATA, call FUNC for every known breakpoint passing USER_DATA
+   as argument.
+
+   If FUNC returns 1, the loop stops and the current
+   'struct breakpoint' being processed is returned.  If FUNC returns
+   zero, the loop continues.
+
+   This function returns either a 'struct breakpoint' pointer or NULL.
+   It was based on BFD's bfd_sections_find_if function.  */
+
+extern struct breakpoint *breakpoint_find_if
+  (int (*func) (struct breakpoint *b, void *d), void *user_data);
+
 /* Return true if BPT is either a software breakpoint or a hardware
    breakpoint.  */