[Bug record/31224] New: Hard to understand warning when reaching end of reverse-execution history

[Bug record/31224] New: Hard to understand warning when reaching end of reverse-execution history

[blarsen at redhat dot com]

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

            Bug ID: 31224
           Summary: Hard to understand warning when reaching end of
                    reverse-execution history
           Product: gdb
           Version: HEAD
            Status: NEW
          Severity: normal
          Priority: P2
         Component: record
          Assignee: unassigned at sourceware dot org
          Reporter: blarsen at redhat dot com
  Target Milestone: ---

Currently, if a user is running a record session, moves backwards a bit and
moves forward again, they may hit the following warning:

> No more reverse-execution history.

My very scientific testing of asking one end user highlighted that this message
sounds like GDB will be unable to move the inferior forward, when all it means
is that we will make the CPU actually execute the inferior, instead of
simulating it.

This is made worse because when the inferior executes backwards to the end of
the history, GDB really can't make the inferior continue backwards and the
message is exactly the same.

It would be nice if the warning was different when moving forward, and was
reworded to make clear that the inferior can continue executing. I would
suggest changing it to:

> No more reverse-execution history. Switching to normal execution

However, I'm open to other ideas.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug record/31224] Hard to understand warning when reaching end of reverse-execution history

[blarsen at redhat dot com]

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

Guinevere Larsen <blarsen at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |good-first-bug

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug record/31224] Hard to understand warning when reaching end of reverse-execution history

[achronop at gmail dot com]

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

Alex Chronopoulos <achronop at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |achronop at gmail dot com

--- Comment #1 from Alex Chronopoulos <achronop at gmail dot com> ---
Hi, I could work on it, if that's ok. My understanding is that this bug is
about changing the string in `print_no_history_reason()` method. Is there more
than that?

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug record/31224] Hard to understand warning when reaching end of reverse-execution history

[blarsen at redhat dot com]

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

--- Comment #2 from Guinevere Larsen <blarsen at redhat dot com> ---
that is all there is to it, yes. instead of always printing the same thing, it
should check if the inferior is moving forwards or backwards and change the
message accordingly.

I'll be happy to review your changes, once you send them to the list, or give
you guidance in private if you need it :) (though fair warning, I usually only
look at that stuff during work hours for Europe)

-- 
You are receiving this mail because:
You are on the CC list for the bug.