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]

Re: [PATCH] Fix for 15072: stopped response missing for reverse-finish


>>>>> "Andreas" == Andreas Kaufmann <Andreas.Kaufmann@synopsys.com> writes:

Andreas> The above is not done (and nor needed) for "reverse-finish". So
Andreas> I added additional check "execution_direction != EXEC_REVERSE"
Andreas> to ensure that observer_notify_normal_stop() is called for
Andreas> "reverse-finish" from normal_stop() in gdb/infrun.c.

It seems reasonable to me (though this isn't really my area and so Pedro
should probably weigh in).

Andreas> 2013-04-18 Andreas Kaufmann <Andreas.Kaufmann@synopsys.com>

Andreas>     PR gdb/15072
Andreas>     * infrun.c (normal_stop): Ensure "observer_notify_normal_stop" is called
Andreas>       when "reverse-finish" command hits a breakpoint.

I think a test case is needed.

But that will probably push the patch over the limit where we need
copyright papers (unless maybe there is some way to see the bug with the
CLI and the test case can be added to an existing file).  Contact me
off-list to get started on that; or if you already have them in place
and I forgot, just mention it.  Thanks.

Tom


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