This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA] Process record and replay, 10/10
- From: Eli Zaretskii <eliz at gnu dot org>
- To: teawater <teawater at gmail dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Fri, 07 Nov 2008 16:48:27 +0200
- Subject: Re: [RFA] Process record and replay, 10/10
- References: <daef60380811052352r48807fc1sbbeeb0f34c035217@mail.gmail.com>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> Date: Thu, 6 Nov 2008 15:52:30 +0800
> From: teawater <teawater@gmail.com>
>
> This patch add document for process record and replay.
>
> 2008-11-06 Hui Zhu <teawater@gmail.com>
>
> * gdb.texinfo: Add documentation for process record and replay.
Thanks. I have a couple of comments.
> +@chapter Recording programs running message and replay it
I don't understand this phrase. What did you want to say? what does
the word "message" have to do with this feature?
> +@kindex target record
> +@kindex record
> +@kindex rec
> +@item target record
> +This a standard command to start process record and replay target.
> +Process record and replay target can only debug a process that already
> +running. Therefore you need to first start the process @code{run},
> +and then start the recording @code{record}.
Is the command "target record" or just "record"? or is the latter an
alias of the former? We need to explain all this in the manual,
because the @item says "target record", while in the last sentence
above you say that the command is "record"
Other than that, this part of the patch is okay.
By the way, I think we need a NEWS entry for this feature.