Re: [PATCH v2] Change message when reaching end of reverse history.

[Eli Zaretskii <eliz@gnu.org>]

> From: Alex Chronopoulos <achronop@gmail.com>
> Cc: Alex Chronopoulos <achronop@gmail.com>
> Date: Sun,  7 Apr 2024 21:55:01 +0200
> 
> In a record session, when we move backward, GDB switches from normal
> execution to simulation. Moving forward again, the emulation continues
> until the end of the reverse history. When the end is reached, the
> execution stops, and a warning message is shown. This message has been
> modified to indicate that the forward emulation has reached the end, but
> the execution can continue as normal, and the recording will also continue.
> 
> Before this patch, the warning message shown in that case was the same as
> in the reverse case. This meant that when the end of history was reached in
> either backward or forward emulation, the same message was displayed:
> 
> "No more reverse-execution history."
> 
> This message remains for backward emulation. However, in forward emulation,
> it has been modified to:
> 
> "End of recorded history; following steps will be added to history."
> 
> The reason for this change is that the initial message was deceiving, for
> the forward case, making the user believe that forward debugging could not
> continue.
> 
> Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=31224
> ---
>  gdb/NEWS                                      |  5 ++++
>  gdb/infrun.c                                  |  8 ++++-
>  gdb/testsuite/gdb.btrace/non-stop.exp         | 30 ++++++++++++-------
>  gdb/testsuite/gdb.reverse/break-precsave.exp  |  4 +--
>  gdb/testsuite/gdb.reverse/break-reverse.exp   |  2 +-
>  .../gdb.reverse/machinestate-precsave.exp     |  2 +-
>  6 files changed, 36 insertions(+), 15 deletions(-)

Thanks, the NEWS part is approved.

Reviewed-By: Eli Zaretskii <eliz@gnu.org>