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: [RFA] Process record and replay, 10/10


> 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.


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