[PATCH, master + 11][gdb/tui] Fix breakpoint display functionality

[Tom de Vries via Gdb-patches <gdb-patches@sourceware.org>]

[-- Attachment #1: Type: text/plain, Size: 1349 bytes --]

[ was: Re: [PATCH 8/9] gdb: remove iterate_over_breakpoints function ]

On 10/21/21 12:20 PM, Tom de Vries wrote:
> On 5/27/21 5:35 PM, Simon Marchi via Gdb-patches wrote:
>>  /* Create and register solib event breakpoints.  PROBES is an array
>> diff --git a/gdb/tui/tui-winsource.c b/gdb/tui/tui-winsource.c
>> index 738f69156485..afd51e95980c 100644
>> --- a/gdb/tui/tui-winsource.c
>> +++ b/gdb/tui/tui-winsource.c
>> @@ -457,7 +457,7 @@ tui_source_window_base::update_breakpoint_info
>>  	 do with it.  Identify enable/disabled breakpoints as well as
>>  	 those that we already hit.  */
>>        tui_bp_flags mode = 0;
>> -      iterate_over_breakpoints ([&] (breakpoint *bp) -> bool
>> +      for (breakpoint *bp : all_breakpoints ())
>>  	{
>>  	  if (bp == being_deleted)
>>  	    return false;
>> @@ -479,7 +479,8 @@ tui_source_window_base::update_breakpoint_info
>>  		}
>>  	    }
>>  	  return false;
>> -	});
>> +	}
>> +
>>        if (line->break_mode != mode)
>>  	{
>>  	  line->break_mode = mode;
>> -- 
> 
> This changes a lambda function body to a loop body, but fails to update
> the two returns.  Consequently, showing breakpoints in tui is broken
> (and unfortunately there's no test-case to detect that).
> 
> This works for me:

Now with commit log, referring to corresponding PR.

OK for trunk / 11 branch ?

Thanks,
- Tom

[-- Attachment #2: 0001-gdb-tui-Fix-breakpoint-display-functionality.patch --]
[-- Type: text/x-patch, Size: 1661 bytes --]

[gdb/tui] Fix breakpoint display functionality

In commit 81e6b8eb208 "Make tui-winsource not use breakpoint_chain", a loop
body was transformed into a lambda function body:
...
-      for (bp = breakpoint_chain;
-           bp != NULL;
-           bp = bp->next)
+      iterate_over_breakpoints ([&] (breakpoint *bp) -> bool
...
and consequently:
- a continue was replaced by a return, and
- a final return was added.

Then in commit 240edef62f0 "gdb: remove iterate_over_breakpoints function", we
transformed back to a loop body:
...
-      iterate_over_breakpoints ([&] (breakpoint *bp) -> bool
+      for (breakpoint *bp : all_breakpoints ())
...
but without reverting the changes that introduced the two returns.

Consequently, breakpoints no longer show up in the tui source window.

Fix this by reverting the changes that introduced the two returns.

Build on x86_64-linux, tested with all .exp test-cases that contain
tuiterm_env.

Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=28483

---
 gdb/tui/tui-winsource.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/gdb/tui/tui-winsource.c b/gdb/tui/tui-winsource.c
index afd51e95980..955b68901fe 100644
--- a/gdb/tui/tui-winsource.c
+++ b/gdb/tui/tui-winsource.c
@@ -460,7 +460,7 @@ tui_source_window_base::update_breakpoint_info
       for (breakpoint *bp : all_breakpoints ())
 	{
 	  if (bp == being_deleted)
-	    return false;
+	    continue;
 
 	  for (bp_location *loc : bp->locations ())
 	    {
@@ -478,7 +478,6 @@ tui_source_window_base::update_breakpoint_info
 		    mode |= TUI_BP_HARDWARE;
 		}
 	    }
-	  return false;
 	}
 
       if (line->break_mode != mode)