From: Eli Zaretskii <eliz@gnu.org>
To: teawater <teawater@gmail.com>
Cc: gdb-patches@sourceware.org
Subject: Re: [RFA] Process record and replay, 10/10
Date: Fri, 07 Nov 2008 14:49:00 -0000 [thread overview]
Message-ID: <uskq3sibo.fsf@gnu.org> (raw)
In-Reply-To: <daef60380811052352r48807fc1sbbeeb0f34c035217@mail.gmail.com>
> 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.
next prev parent reply other threads:[~2008-11-07 14:49 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-11-06 7:53 teawater
2008-11-07 14:49 ` Eli Zaretskii [this message]
2008-11-08 4:40 ` teawater
2008-11-08 9:25 ` Eli Zaretskii
2008-11-11 5:43 ` teawater
2008-11-14 16:04 ` Eli Zaretskii
2008-11-14 16:29 ` teawater
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=uskq3sibo.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=gdb-patches@sourceware.org \
--cc=teawater@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox