This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [reverse RFA] Remove process record warning code
Add it in current time maybe make code not very clear.
Maybe I can add it back in the future.
On Sat, Oct 18, 2008 at 09:54, Michael Snyder <msnyder@vmware.com> wrote:
> I'm worried that the naive user might think that he can
> extend a recording session across several runs.
>
> Might seem counter-intuitive, but I think the SID reverse
> execution implementation will allow you to do that. If you
> run the inferior multiple times, you can actually "back up"
> into a previous execution.
>
> I know Process Record won't let you do that, I'm just
> saying, a user's intuition is not necessarily the same
> as yours or mine...
>
> teawater wrote:
>>
>> I think user already know P record will stop when inferior exit. So...
>>
>> How do you think about it?
>>
>>
>> On Sat, Oct 18, 2008 at 01:23, Michael Snyder <msnyder@vmware.com> wrote:
>>>
>>> You don't think you need this special warning?
>>>
>>> teawater wrote:
>>>>
>>>> Hi Michael,
>>>>
>>>> 2008-10-17 Hui Zhu <teawater@gmail.com>
>>>>
>>>> * infcmd.c (kill_if_already_running): Remove process record
>>>> warning code.
>>>>
>>>> Thanks,
>>>> Hui
>>>>
>>>>
>>>> ------------------------------------------------------------------------
>>>>
>>>> --- a/ChangeLog
>>>> +++ b/ChangeLog
>>>> @@ -1,3 +1,8 @@
>>>> +2008-10-17 Hui Zhu <teawater@gmail.com>
>>>> +
>>>> + * infcmd.c (kill_if_already_running): Remove process record
>>>> + warning code.
>>>> +
>>>> 2008-10-15 Hui Zhu <teawater@gmail.com>
>>>> Change "record-auto-delete" to "record-stop-at-limit".
>>>> --- a/infcmd.c
>>>> +++ b/infcmd.c
>>>> @@ -51,7 +51,6 @@
>>>> #include "exceptions.h"
>>>> #include "cli/cli-decode.h"
>>>> #include "gdbthread.h"
>>>> -#include "record.h"
>>>> /* Functions exported for general use, in inferior.h: */
>>>> @@ -427,12 +426,8 @@ kill_if_already_running (int from_tty)
>>>> restart it. */
>>>> target_require_runnable ();
>>>> - if (from_tty)
>>>> - if (!query (RECORD_IS_USED ?
>>>> - "The program being debugged has been started
>>>> already,\n\
>>>> -and is running in record/replay mode. Do you want to abandon\n\
>>>> -the recording and start the program from the beginning? " :
>>>> - "The program being debugged has been started
>>>> already.\n\
>>>> + if (from_tty
>>>> + && !query ("The program being debugged has been started
>>>> already.\n\
>>>> Start it from the beginning? "))
>>>> error (_("Program not restarted."));
>>>> target_kill ();
>>>
>
>